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

v3 spec: no compression? #93

Closed
davidbrochart opened this issue Sep 8, 2020 · 3 comments
Closed

v3 spec: no compression? #93

davidbrochart opened this issue Sep 8, 2020 · 3 comments
Labels
core-protocol-v3.0 Issue relates to the core protocol version 3.0 spec

Comments

@davidbrochart
Copy link
Contributor

It looks like an array does not necessarily have a compressor ("An array may be associated with a compressor").
How is it encoded in the array metadata? If it is specified by the absence of a compressor key, the spec should be explicit about it.

@davidbrochart
Copy link
Contributor Author

Maybe we can have the following in the array metadata:

    "compressor": {
        "codec": "https://purl.org/zarr/spec/codec/binary/1.0",
        "configuration": {}
    },

@jakirkham
Copy link
Member

Based on discussion, we decided to simply not have a compressor key if the data is not compressed.

@Carreau Carreau added the core-protocol-v3.0 Issue relates to the core protocol version 3.0 spec label Sep 25, 2020
@Carreau
Copy link
Contributor

Carreau commented Sep 25, 2020

Closed by #94

@Carreau Carreau closed this as completed Sep 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core-protocol-v3.0 Issue relates to the core protocol version 3.0 spec
Projects
None yet
Development

No branches or pull requests

3 participants