Skip to content

Conversation

@cgoina
Copy link
Collaborator

@cgoina cgoina commented Oct 28, 2025

@StephanPreibisch
This is an attempt to fix the OME-ZARR metadata transformations.

The fix was to also scale the translation with the initial voxel resolution

There's also a fix to use the proper multiscale dataset subpath in case the scale subpath is not numeric, e.g. in case it is 's0', 's1', 's2',...

@cgoina
Copy link
Collaborator Author

cgoina commented Oct 31, 2025

@StephanPreibisch I made changes to use the new average calibration.

dimensions.get( viewId.getViewSetupId() ),
//data.getSequenceDescription().getViewDescription( viewId ).getViewSetup().getVoxelSize().dimensionsAsDoubleArray(),
vx.dimensionsAsDoubleArray(), // resolutionS0
vx.unit(),
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this case it saves a single viewsetup so I think it is ok to use the voxel attributes from the view setup

@StephanPreibisch
Copy link
Contributor

StephanPreibisch commented Jan 9, 2026

@cgoina - did you test the fusion results thoroughly? That the fused zarrs display properly in neuroglancer and Fiji?

@cgoina
Copy link
Collaborator Author

cgoina commented Jan 9, 2026

@StephanPreibisch I only tested for one dataset - the tiff example dataset both in Fiji and neuroglancer. Initially in neuroglancer I got an error because "px" was not valid - so I chose micrometer as default. I have not tried to leave it empty. I will re-test with N5 and TIFF examples to make sure, because it's been a while.

@StephanPreibisch
Copy link
Contributor

Please try with and without "preserve anisotropy", and with "1x & 2x downsampling", and both combined since it changes the metadata.... thanks so much!

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