-
-
Notifications
You must be signed in to change notification settings - Fork 282
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
base64 encode fill value for some dtypes with zarr_format=2
#2286
base64 encode fill value for some dtypes with zarr_format=2
#2286
Conversation
This should be ready to go (the CI failure is from appveyor). |
zarr_format=2
zarr_format=2
were we not doing this in the v2 codebase? I couldn't find code for it. |
I found it at Line 215 in 6b9ab9e
|
Thanks Tom! 🙏 JFYI we have a |
Thanks John. I think this version (rather than numcodec's) is OK. It ends up using the same actual implementation of |
Closes #2282
Needs more tests.