Skip to content

Conversation

@eshort0401
Copy link

Slight documentation amendments following discussion in #8749 and #11006.

  • I add a few lines to communicate that Xarray checks all three supported conventions; first Zarr v3, then Xarray flavoured Zarr v2, and finally NCZarr, raising an error if no dimension names are found.
  • I make a few additional word choice edits for consistency and clarity, e.g. replace "Zarr arrays" with "Zarr groups" in the "Compatibility and Reading" section.
  • I have also added an additional code comment to xarray.backends.zarr._get_zarr_dims_and_attrs providing the link to the (very helpful) Xarray encoding specification - I missed this document when I first looked at this code!

Additional Notes

Copy link
Member

@TomNicholas TomNicholas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thank you!

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lack of resilience towards missing _ARRAY_DIMENSIONS xarray's special zarr attribute #280

2 participants