Skip to content

stdlib json not up to json spec. #81

Open

Description

@Carreau

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.

Activity

  1. jstriebel commented on Nov 16, 2022

    @jstriebel
    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?

  2. joshmoore commented on Nov 17, 2022

    @joshmoore
    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

  3. jakirkham commented on Nov 17, 2022

    @jakirkham
    Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions