Skip to content

Added support to provide nocache parameter in Config - #555

Merged
DarshanSP19 merged 2 commits into
mainfrom
support-nocache-param
Jul 30, 2026
Merged

Added support to provide nocache parameter in Config#555
DarshanSP19 merged 2 commits into
mainfrom
support-nocache-param

Conversation

@DarshanSP19

@DarshanSP19 DarshanSP19 commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Add support to provide a nocache to fetch new data instead of a cached one. (This is currently specific to the CDS API). The value can only be true or false. The error will be thrown in case of unsupported value.
Examples for supported values

[selection]
no_cache=true
[selection]
no_cache=false

@DarshanSP19
DarshanSP19 requested a review from mahrsee1997 July 29, 2026 08:55
@DarshanSP19 DarshanSP19 self-assigned this Jul 29, 2026

@mahrsee1997 mahrsee1997 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple of changes ...
(1) We need to do similar changes for weather-dl v2.
(2) Bump appropriate setup version

Comment thread weather_dl/download_pipeline/config.py Outdated
@DarshanSP19
DarshanSP19 force-pushed the support-nocache-param branch from 1d3d0ad to ceb398b Compare July 29, 2026 11:54
@DarshanSP19
DarshanSP19 requested a review from mahrsee1997 July 29, 2026 11:54

@mahrsee1997 mahrsee1997 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@DarshanSP19
DarshanSP19 merged commit 6feb38b into main Jul 30, 2026
14 checks passed
@DarshanSP19
DarshanSP19 deleted the support-nocache-param branch July 30, 2026 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants