Skip to content

Conversation

d-v-b
Copy link
Contributor

@d-v-b d-v-b commented May 22, 2025

This pr adds the array's fill value to the output of info.

@d-v-b d-v-b requested a review from dstansby May 22, 2025 09:49
Comment on lines 214 to 215
No. bytes stored : 9696520
Storage ratio : 41.3
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this looks off-target from my PR, curious how this change slipped in

Copy link
Contributor

Choose a reason for hiding this comment

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

Seeing the same thing in my PR

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, this is possibly because of the yanked numcodecs?

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah I was unaware numcodecs had been yanked, ok that's probably it then.

@dstansby
Copy link
Contributor

As suspected, re-running the tests now numcodecs 0.16.1 is out fails - could you re-(re?)-fix the doctest, otherwise this looks good to me 👍

@dstansby dstansby merged commit 2361cd7 into zarr-developers:main May 25, 2025
30 checks passed
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.

3 participants