Skip to content

Conversation

@joshmoore
Copy link
Member

@joshmoore joshmoore commented Feb 7, 2022

@jni
Copy link

jni commented Feb 7, 2022

Slight note, possibly not important, but I think it would make more sense re sparse writing to say it was added by Davis and me, rather than me and Davis. He did the hard work on the implementation, I changed a flag from True to False 😂.

@sanketverma1704
Copy link
Member

Sure @jni. 👍🏻

Copy link
Member Author

@joshmoore joshmoore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Few points (I pushed commits for them)

@sanketverma1704
Copy link
Member

All changes are done. Merging this PR now.

@sanketverma1704 sanketverma1704 merged commit 021cdf0 into main Feb 8, 2022
@joshmoore joshmoore deleted the release-2.11 branch February 8, 2022 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Blogpost for Zarr 2.11

4 participants