-
Notifications
You must be signed in to change notification settings - Fork 15
⬆️ Add compatibility with zarr v3 #2711
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
Conversation
Ok, from the side of |
f91b779
to
b2c4f15
Compare
This reverts commit 4898219.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2711 +/- ##
==========================================
+ Coverage 91.94% 92.13% +0.19%
==========================================
Files 70 71 +1
Lines 10717 9629 -1088
==========================================
- Hits 9854 8872 -982
+ Misses 863 757 -106 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Deployment URL: https://5e31f432.lamindb.pages.dev |
This reverts commit ec969da.
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.
Looks great! 😄
Closes #2335
This updates the internals of our code to support
anndata>=0.12.0rc1
withzarr>=3.0.0
.lamindb
still hasanndata<=1.14.0
withzarr<3.0.0
in the requirements, but nothing breaks if neweranndata
orzarr
are installed.Also refactors some zarr-related code.