VirtualiZarr icon indicating copy to clipboard operation
VirtualiZarr copied to clipboard

Check chunk grid shape consistent upon ManifestArray init

Open TomNicholas opened this issue 5 months ago • 0 comments

Addresses this TODO:

https://github.com/zarr-developers/VirtualiZarr/blob/2e0b22822b51f1a9c8cbe08c791289e1fce895c3/virtualizarr/manifests/array.py#L67

I noticed that this should be disallowed while debugging https://github.com/zarr-developers/VirtualiZarr/issues/728#issuecomment-3110221034.

Funnily though this breaks some other tests, which potentially need to be changed to not create invalid ManifestArrays.

  • [x] Tests added
  • [ ] Tests passing
  • [x] Full type hint coverage
  • [ ] Changes are documented in docs/releases.rst
  • [ ] ~~New functions/methods are listed in api.rst~~
  • [ ] New functionality has documentation

TomNicholas avatar Jul 23 '25 22:07 TomNicholas