Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update weather-dl to make use of CDS beta #474

Open
mahrsee1997 opened this issue Sep 3, 2024 · 1 comment
Open

Update weather-dl to make use of CDS beta #474

mahrsee1997 opened this issue Sep 3, 2024 · 1 comment

Comments

@mahrsee1997
Copy link
Collaborator

mahrsee1997 commented Sep 3, 2024 •
edited
Loading

Ref: https://confluence.ecmwf.int/display/CKB/Please+read%3A+CDS+and+ADS+migrating+to+new+infrastructure%3A+Common+Data+Store+%28CDS%29+Engine

Looks like the current support for cds-beta in cdsapi library doesn't incline with our weather-dl codebase.

Part 1:
For temporary fix, fork the repo & make necessary changes as per our requirements.

Part 2:
And on September 26th, check and update cdsapi library to latest version; hoping they fully migrate their library to use cds-beta only.
@mahrsee1997 mahrsee1997 mentioned this issue Sep 3, 2024
Merged
dabhicusp added a commit that referenced this issue Sep 3, 2024
@dabhicusp
update cds to cds-beta in weather-dl. (#472)
f4405bd 
[TEMPORARY CODE CHANGE]

The Climate Data Store (CDS) has been moved to the CDS-Beta and will be deprecated on September 16th, as announced in the ECMWF Confluence page. The latest version of the cdsapi library is not compatible with the current code of the weather-dl tool.

Therefore, we forked their repo and made necessary changes as per our requirement.

On September 16th, we will check and update cdsapi library to latest version.

Bug for reference: #474
@dabhicusp
Copy link
Collaborator

dabhicusp commented Nov 12, 2024 •
edited
Loading

Part 1 is already done.

Part 2

  • The weather-dl code is incompatible with the latest cds-api version because cdsapi has not updated their code in a long time. As a temporary solution, we are relying on the fix implemented in the "update weather-dl to compatible with cds" (update cds to cds-beta in weather-dl. #472) pull request.
  • Can I closed this bug now? @mahrsee1997.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mahrsee1997 @dabhicusp