pangeo-data/rechunker

Rechunker 0.4 requires Dask and Prefect

tomwhite opened this issue · 6 comments

It looks like the changes in #77 mean that Prefect (and Dask) are now required, which seems like an inadvertent change.

I noticed this in https://github.com/pystatgen/sgkit/issues/530.

Agreed, that wasn't intentional.

Do we just need to make the prefect import at https://github.com/pangeo-data/rechunker/pull/77/files#diff-3dfce265445fa41c7ca0059e55990be70becc0eacead2d147ee39840fba1db27R1-R3 lazy / put in a try / except block?

Hi folks, I did this release just before vacation. Sorry for the glitch. I'll push a fix and do a new release asap.

Rechunker 0.4.1 is now on pypi. @tomwhite, do you want to confirm that it actually resolves your original problem? #85 was a quick fix and doesn't actually explicitly test that you can import rechunker without the optional dependencies.

Unfortunately #85 didn't fix the issue. I've opened #87 which fixes the issue.

#85 was a quick fix and doesn't actually explicitly test that you can import rechunker without the optional dependencies.

It would be good to add such a test if someone has time.

Ok thanks for checking. Should we also revert #85?

Ok hopefully 0.4.2 does the trick 😉 ! Thanks for your patience with this error and for your help fixing my mistake.

FWIW, dask[array] has been a required dependency for a while:

install_requires = ["dask[array]", "zarr", "xarray"]