Skip to content

Converted dataset attributes - #191

Merged
bogovicj merged 2 commits into
developmentfrom
convertedDatasetAttributes
Jan 15, 2026
Merged

Converted dataset attributes#191
bogovicj merged 2 commits into
developmentfrom
convertedDatasetAttributes

Conversation

@cmhulbert

Copy link
Copy Markdown
Contributor

No description provided.

@cmhulbert cmhulbert added this to the 4.0.0 Release milestone Jan 9, 2026
@cmhulbert
cmhulbert changed the base branch from master to development January 9, 2026 19:51
cmhulbert added a commit to saalfeldlab/n5-zarr that referenced this pull request Jan 9, 2026


refactor[BREAKING]: remove constructor parameters for `mapAttributes`, `dimensionSeparator`, `mergeN5Attributes`. Map and Merge are no longer supported (they weren't really for ZarrV3 ever, but the constructors were inherited from ZarrV2). Dimension separator now has a getter/setter

Signed-off-by: Caleb Hulbert <cmhulbert@gmail.com>
@cmhulbert cmhulbert closed this Jan 12, 2026
@cmhulbert cmhulbert reopened this Jan 12, 2026
@cmhulbert
cmhulbert deleted the branch development January 12, 2026 16:52
@cmhulbert cmhulbert closed this Jan 12, 2026
@cmhulbert cmhulbert reopened this Jan 12, 2026
@cmhulbert
cmhulbert force-pushed the convertedDatasetAttributes branch from 3526907 to d34f342 Compare January 12, 2026 16:56
@cmhulbert
cmhulbert marked this pull request as ready for review January 12, 2026 19:37
cmhulbert added a commit to saalfeldlab/n5-zarr that referenced this pull request Jan 12, 2026


refactor[BREAKING]: remove constructor parameters for `mapAttributes`, `dimensionSeparator`, `mergeN5Attributes`. Map and Merge are no longer supported (they weren't really for ZarrV3 ever, but the constructors were inherited from ZarrV2). Dimension separator now has a getter/setter

Signed-off-by: Caleb Hulbert <cmhulbert@gmail.com>
@bogovicj
bogovicj merged commit 1c6b3ad into development Jan 15, 2026
2 checks passed
@bogovicj bogovicj self-assigned this Feb 2, 2026
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.

2 participants