search

pangeo-data / rechunker

Disk-to-disk chunk transformation for chunked arrays.
https://rechunker.readthedocs.io/
MIT License
163 stars 25 forks source link

Clarify xarray dependency #63

Open rabernat opened 3 years ago

rabernat commented 3 years ago

It is not clear right now whether xarray is required dependency for rechunker and if so, what versions are required. See this Discourse thread for context: https://discourse.pangeo.io/t/issues-pip-installing-rechunker-onto-hpc-conda-environment/1034

A user reported

ImportError: cannot import name 'encode_zarr_attr_value' from 'xarray.backends.zarr' (/scratch/aeb783/penv/lib/python3.8/site-packages/xarray/backends/zarr.py)

due to an incompatible xarray version.

TomNicholas commented 1 year ago

Looking at the code it seems it would be pretty simple to demote xarray to be an optional dependency of rechunker?

tomwhite commented 1 year ago

I've opened a PR for this at #123.