Skip to content
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

stdlib json not up to json spec. #81

Open
Carreau opened this issue Jun 17, 2020 · 3 comments
Open

stdlib json not up to json spec. #81

Carreau opened this issue Jun 17, 2020 · 3 comments
Labels

Comments

@Carreau
Copy link
Contributor

Carreau commented Jun 17, 2020

A recent thread on Python idea pointed out that Json.dumps does not by default respect the json spec (at least on some number like NaN/Inf if should refuse to encode). we might want to be careful in v3 implmentation.

@jstriebel
Copy link
Member

Issue #141 is directly related. Besides changes to the spec specifically making it possible to encode such values somehow, this should probably be moved to zarr-python, do you agree @joshmoore?

@joshmoore
Copy link
Member

I'm not sure. I think this is something that all the v2 implementations will or do struggle with, no? Ok to mark this as v2 for the moment, and we perhaps reach out to the other implementations to see where they stand?

e.g. JuliaIO/Zarr.jl#97

@jakirkham
Copy link
Member

We already check arrays are loadable by different implementations. Maybe we extend this to metadata?

@jstriebel jstriebel added the v2 label Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants