-
Notifications
You must be signed in to change notification settings - Fork 39
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.
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
Add support for downloading CMIP6 data with Synda #699
Conversation
Hi, just wanted to chime in that I used this branch to successfully download some CMIP6 data with Synda for a recipe yesterday! 👍 |
Not a full review, but I tried to use this branch to download the data required for ESMValGroup/ESMValTool#1391, and found the following. It works for most datasets, but I also get time-out errors and dataset not found problems, e.g.:
Not sure if it's related to this PR, but I thought a real use case test could be relevant. |
@valeriupredoi Would you have time to review this pull request, so it can still be included in v2.1?
@Peter9192 I'm afraid that's not something we can fix in the ESMValTool. |
I am testing it with a cmip6-only
Forget about this, sometimes I get the output, sometimes not. It will need a bit more investiggation to ahve something that looks well in all cases |
It is working (almost) well, with the almost being fault of synda I think. I find a case in which synda search finds a file but then can not download it. I think we already face this kind of things with CMIP5, but I am nor sure |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice addition! I will create an issue to remind us that we may want to take a look in adding support for obs4mips
and ana4mips
Downloading CMIP6 data with Synda did not work because the query is different from CMIP5. This pull request adds the correct query.
Tasks