Skip to content

Conversation

LDeakin
Copy link
Member

@LDeakin LDeakin commented Jul 13, 2025

This simplifies a lot, removes the StoreManager and lets us properly support a broader range of arrays where codec behaviour is dependent on non-codec metadata (e.g. data_type, order).

I added a warning that indicates to a user why an array is not supported by zarrs when constructing a ZarrsCodecPipeline. This catches a few things and reveals some holes in zarrs support for Zarr V2 data which can be supported eventually. Features that aren't supported by zarrs-python, like variable sized data types, still just silently fallback at runtime.

Base automatically changed from ig/zarr_python_new_dtypes to main July 15, 2025 12:18
@LDeakin LDeakin marked this pull request as ready for review July 15, 2025 12:47
@LDeakin LDeakin merged commit 53c2224 into main Jul 16, 2025
16 checks passed
@LDeakin LDeakin deleted the ld/array_metadata_init branch July 16, 2025 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants