search

zarr-developers / VirtualiZarr

Create virtual Zarr stores from archival data files using xarray syntax
https://virtualizarr.readthedocs.io/en/stable/api.html
Apache License 2.0
124 stars 24 forks source link

Avoid loading scalar values automatically? #270

Open TomNicholas opened 1 month ago

TomNicholas commented 1 month ago

unless there is a good reason not to load scalar values automatically.

I guess the only reason not to is that once https://github.com/zarr-developers/zarr-python/issues/1977 is fixed (as it should be, else its a breaking change between v2 and v3) we will no longer need to load scalar values automatically, so we would be adding something to virtualizarr that we know we should remove later.

I think that's okay though - if you're happy to add a PR @ghidalgo3 we can think about what scenarios should cause a warning (or deprecation warning).

Originally posted by @TomNicholas in https://github.com/zarr-developers/VirtualiZarr/issues/194#issuecomment-2253161789

TomNicholas commented 1 month ago

Raising because https://github.com/zarr-developers/zarr-python/issues/1977 was merged.