Skip to content

Fix handling of scalar Zarr v3 arrays with None dimension_names #897

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
larsbuntemeyer wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Fix handling of scalar Zarr v3 arrays with None dimension_names #897

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
039c34a
handle dimension_names is None
larsbuntemeyer Mar 2, 2026
8f74c63
update releases.md
larsbuntemeyer Mar 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions docs/releases.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,9 @@

### Bug fixes

- Fix handling of scalar Zarr V3 arrays with None dimension_names.
([#897](https://github.com/zarr-developers/VirtualiZarr/pull/897)).
By [Lars Buntemeyer](https://github.com/larsbuntemeyer).
- Fix setting `fill_value` for Zarr V2 arrays if data type is a subtype of integer or float.
([#845](https://github.com/zarr-developers/VirtualiZarr/pull/845)).
By [Hauke Schulz](https://github.com/observingClouds).
Expand Down
2 changes: 1 addition & 1 deletion virtualizarr/manifests/array.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -280,7 +280,7 @@ def to_virtual_variable(self) -> xr.Variable:

# The xarray data model stores dimension names and arbitrary extra metadata outside of the wrapped array class,
# so to avoid that information being duplicated we strip it from the ManifestArray before wrapping it.
dims = self.metadata.dimension_names
dims = self.metadata.dimension_names or ()
attrs = self.metadata.attributes
stripped_metadata = utils.copy_and_replace_metadata(
self.metadata, new_dimension_names=None, new_attributes={}
Expand Down
Loading