zarr-python icon indicating copy to clipboard operation
zarr-python copied to clipboard

[v3] Array constructor API compatibility

Open jhamman opened this issue 1 year ago • 0 comments

In addition to the top-level API compatibility layer, we are also seeing that some applications are interacting with the Array class constructor directly. For example, see this from Dask.

z = zarr.Array(url, read_only=True, path=component, **kwargs)

But our current constructor doesn't support this.

https://github.com/zarr-developers/zarr-python/blob/661acb37f08a77cc1a86c0da55288e89f2388801/src/zarr/array.py#L572-L574

To be honest, I've been assuming no one was using Array.__init__ but that seems to have been a mistake.

Thanks @will-moore for reporting this (https://github.com/zarr-developers/zarr-python/pull/1884#issuecomment-2152253454).

cc @jrbourbeau

jhamman avatar Jun 06 '24 23:06 jhamman