Implement support for structured and struct zarr-extension defined dtypes

[BrianMichell]

Updates for consistency with the zarr-extension for structured types.

Implements legacy read-only for structured type and support for new struct definition.

TODO:

• Add unit tests and/or doctests in docstrings
• Add docstrings and API docs for any new/modified user-facing classes and functions
• New/modified features documented in docs/user-guide/*.md
• Changes documented as a new file in changes/
• GitHub Actions have all passed
• Test coverage is 100% (Codecov passes)

[d-v-b]

this looks great, thanks for this. Instead of modifying the existing structured data type, can we make struct as a subclass of that data type, and register it? That means people who need the legacy dtype can still use it (by overriding the registry)

[BrianMichell]

this looks great, thanks for this. Instead of modifying the existing structured data type, can we make struct as a subclass of that data type, and register it? That means people who need the legacy dtype can still use it (by overriding the registry)

Should be good to go now!

[BrianMichell]

@d-v-b Is there anything I can do to move this PR forward?

[codecov]

Codecov Report

❌ Patch coverage is 93.10345% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.88%. Comparing base (c9b534a) to head (5580194).

Files with missing lines	Patch %	Lines
src/zarr/codecs/bytes.py	60.00%	4 Missing ⚠️
src/zarr/core/dtype/npy/structured.py	97.29%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3781      +/-   ##
==========================================
- Coverage   92.89%   92.88%   -0.02%     
==========================================
  Files          86       86              
  Lines       10689    10759      +70     
==========================================
+ Hits         9930     9993      +63     
- Misses        759      766       +7     

Files with missing lines	Coverage Δ
src/zarr/core/array.py	97.92% <100.00%> (ø)
src/zarr/core/dtype/__init__.py	100.00% <100.00%> (ø)
src/zarr/dtype.py	100.00% <ø> (ø)
src/zarr/core/dtype/npy/structured.py	97.40% <97.29%> (-1.54%)	⬇️
src/zarr/codecs/bytes.py	92.75% <60.00%> (-5.58%)	⬇️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[d-v-b]

@d-v-b Is there anything I can do to move this PR forward?

hi @BrianMichell thanks for the ping, I will get a review in soon. since numpy is handling the actual data type, I think we will ask much less of you than the tensorstore devs :)