-
-
Notifications
You must be signed in to change notification settings - Fork 15k
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
python312Packages.zarr: 2.18.3 -> 3.0.4 #373248
base: master
Are you sure you want to change the base?
Conversation
3a338df
to
88f4197
Compare
88f4197
to
e6e5922
Compare
e6e5922
to
6b93bb9
Compare
6b93bb9
to
ffd6efa
Compare
I can't find an upstream issue for that. Mind opening one? |
This issue might still be observed though: zarr-developers/zarr-python#2732
Done, also for tifffile, and updated the top comment of the PR. Also, I pushed a few more commits related to the main topic of the PR. I also added there |
ffd6efa
to
6477f52
Compare
After working a bit with the patch here and with zarr v3 directly, I noticed that many features of it are missing in comparison to version 2.x. Even if the dependencies mentioned in the PR's top comment will adapt to v3, I think we should still have a |
Currently draft since at least the following packages need to be updated upstream to the Zarr 3.x. API:
python312Packages.tifffile
: Zarr v3 compatibility? cgohlke/tifffile#282python312Packages.multiscale-spatial-image
: zarr 3 spatial-image/multiscale-spatial-image#108python312Packages.ome-zarr
: Zarr v3 ome/ome-zarr-py#404python312Packages.reproject
Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.