earth-mover / icechunk

Open-source, cloud-native transactional tensor storage engine
https://icechunk.io
Apache License 2.0
305 stars 18 forks source link

Fix non-conformant "attributes" metadata #303

Closed LDeakin closed 1 month ago

LDeakin commented 1 month ago

I believe "attributes": null in array/group metadata is not conformant to the Zarr V3 specification. If empty, it must be omitted entirely or be "attributes": {}.

https://zarr-specs.readthedocs.io/en/latest/v3/core/v3.0.html#attributes

The following members are optional:

attributes The value must be an object. The object may contain any key/value pairs, where the key must be a string and the value can be an arbitrary JSON literal. Intended to allow storage of arbitrary user metadata.