Repository navigation
Declare zarr and dask as dependencies, require xarray >= 2025.6 - #16
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What this PR fixes
After
pip install xarray-ecmwf, opening any dataset fails:and once dask is installed:
# ImportError: The zarr package is required for working with Zarr stores but could not be imported. ...The engine needs both, for every client (
cdsapi,ecmwf-opendata,polytope):chunks={}, and chunks the schema sample of each variable;to_zarr, and reads it back.Neither was ever declared, not even in the old conda
environment.yml. They were only added to the dev group in d962e26, to makemake all-testswork.With zarr 3, xarray must also be at least 2025.6. Older versions fail on every open:
Changes
dask>=2026.8.0andzarr>=3.1.6move from the dev group to the package dependencies, with the same minimums.xarray>=2025.6, the first version that works with zarr 3.daskandzarras unused: the package doesn't import them, xarray does.Tests
All CI jobs pass locally. A plain install of the built wheel now includes dask and zarr, and opens a dataset.
Notes
dask>=2026.8.0is the minimum the dev group already tested. It could be lowered, if older versions are tested too.tests/test_20_client_cdsapi.pycan be removed.