-
Notifications
You must be signed in to change notification settings - Fork 23
Zarr write + Zarr groups #915
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
|
Not a very large PR, this one! I don't know what to do about the recipe files linting thing ... |
As for reverting the recipes linter/isort auto-changes as discussed in-person, the git command I mentioned that is simplest would be |
|
I'll start reviewing now ignoring the linting changes to the recipes... |
sadielbartholomew
left a comment
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.
All good, with tests all passing and relevant packaging, docs and tests updates for the new Zarr read and write funcionality - though ideally drop the recipes linting auto-changes as discussed in person (see my comment above for recommended command to revert).
|
Thanks for the git command. I did |
Co-authored-by: Sadie L. Bartholomew <sadie.bartholomew@ncas.ac.uk>
Fixes #894 and #895