-
Notifications
You must be signed in to change notification settings - Fork 50
Update to zarr v3 #735
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
Update to zarr v3 #735
Conversation
… behaviour too I think
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.
Only comment here is to add this in the docs under the catalog spec section, other than that, this looks great! 🎉
https://intake-esm.readthedocs.io/en/stable/reference/esm-catalog-spec.html#
…. I think we can do better & take care of this for the user
…tion in the data format
@mgrover1 not quite sure what you meant by that so I might have gone a bit overboard - added distinct Some tests that I added in a recent PR apparently didn't get merged & got dragged in with this PR. I'll see if I can get codecov running again so I don't make any more mistakes like that... |
@martindurant does intake have a codecov account? I'd like to get the code coverage CI step running again if possible. |
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.
Thank you for specifying zarr2 vs zarr3 here which will help with backwards compatibility 🎉
Sorry, no. |
* Tests all passing? Deps must have updated since I last tried * Missing environment bits * Apparently the bugfix was in zarr v3.0.10 * Fix requirements in ci/env*.yml * Allow zarr <3.0 or >=3.0.10 * Change async in test based on zarr version - we can make this default behaviour too I think * Move `_zarr_async` to utils module * Add asynchronous/synchronous note to docs under Assets object section. I think we can do better & take care of this for the user * @mgrover1 comment, add some extra tests, allow zarr version specification in the data format * Misformatted esm-catalog-spec * Re-pin minimum zarr version * Formatting
Change Summary
'asynchronous': True
totest_open_dataset_kerchunk
- fails otherwise. We probably want to add this to the documentation?Related issue number
#727
Checklist