search

zarr-developers / zarr-specs

Zarr core protocol for storage and retrieval of N-dimensional typed arrays
https://zarr-specs.readthedocs.io/
Creative Commons Attribution 4.0 International
88 stars 28 forks source link

remove implicit groups #292

Closed d-v-b closed 3 months ago

d-v-b commented 7 months ago

this PR removes support for implicit groups (Zarr groups with no metadata documentation) from the spec. See #291 for the motivation. Besides changing the content of the text, I don't know what else to change, so I'm keeping this a draft for now.

A mergeable version of this PR would include a section explaining what implicit groups are, why they were added previously, and why they are now removed. I am happy to add this if it looks like this PR has legs.

Related discussion: #184

MSanKeys963 commented 6 months ago

@d-v-b, is this PR ready for review?

d-v-b commented 6 months ago

@MSanKeys963 yes!

d-v-b commented 4 months ago

how can we move forward with this? I think it's important to make these changes quickly, before we get skew between implementations.

jhamman commented 3 months ago

@zarr-developers/steering-council - this PR has sat without review for 4 months. Can we move it forward please?

joshmoore commented 3 months ago

@d-v-b: do you want to take a look at the conflict?

d-v-b commented 3 months ago

@d-v-b: do you want to take a look at the conflict?

done!