Skip to content

deprecate / remove support for creating v2 array metadata with null fill value #3084

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

Open
d-v-b opened this issue May 23, 2025 · 0 comments
Open
Milestone

Comments

@d-v-b
Copy link
Contributor

d-v-b commented May 23, 2025

Indeed! To be honest, I think null fill values in Zarr V2 were a bit of a misstep anyway, mainly because undefined values in partially written chunks cannot be distinguished from real data. This could be a good opportunity for zarr-python to stop writing null fill values altogether in Zarr V2 metadata and just write the default fill value. What do you think?

Originally posted by @LDeakin in #2819 (comment)

Following up on this idea, I think we should consider altering our user-facing array creation routines to disallow creating a null fill value in v2 array metadata. Instead, we should treat fill_value=None to mean "use the default fill value for the dtype".

Users who want to create a zarr v2 array with a fill value that is null should be directed to lower-level routines for this.

@d-v-b d-v-b added this to the 3.1.0 milestone May 23, 2025
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

No branches or pull requests

1 participant