-
Notifications
You must be signed in to change notification settings - Fork 44
Add xcube support #2917
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
base: main
Are you sure you want to change the base?
Add xcube support #2917
Conversation
| - xcube-cci | ||
| - yamale | ||
| - zarr >3 | ||
| - zarr >2 |
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.
zarr3 is perfectly able to read zarr2 datasets, bud
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.
xcube requires zarr==2
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.
well that's a bummer - that means it can't read Zarr3 spec?
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.
also Zarr2 is borderline archaic - good luck to us trying to maintain such an evironment
|
this is shaping up nicely - couple quick questions: why not use the standard xarray - to iris via ncdata path? Is XCube really needed to load the ESA-CCI Zarr files, and if not, do we know what database they are in so we can bolt on an eg intake-esm functionality? |
That is exactly what is used: ESMValCore/esmvalcore/io/xcube.py Line 104 in 428ae04
ESMValCore/esmvalcore/iris_helpers.py Line 617 in 428ae04
No, but it is convenient
It looks like the data is available here: https://github.com/esa-cci/xcube-cci/blob/fe8ac26405bd36b0176e1a0cae30238f52009a10/xcube_cci/zarraccess.py#L47
That might be possible, but I'm not sure if that will be easier to maintain or less work. xcube supports an interesting range of data sources from ESA, Copernicus, Climate Data Store, and even Zenodo, so I think that having support for xcube will be an interesting feature for our users. Of course, that shouldn't stop us from adding support for intake-esm as well. |
Description
Add support for loading data with xcube
Related to #2584
To use the data source, run
Example recipe, run it with
--max-parallel-tasks 1for now:This code currently skips all fixes and CMOR checks.
Link to documentation: https://esmvaltool--2917.org.readthedocs.build/projects/ESMValCore/en/2917/api/esmvalcore.io.xcube.html
Before you get started
Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.
To help with the number pull requests: