Skip to content
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.

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

Improper intermediate downscale spacing #655

Closed
tbirdso opened this issue May 18, 2023 · 2 comments · Fixed by #671
Closed

Improper intermediate downscale spacing #655

tbirdso opened this issue May 18, 2023 · 2 comments · Fixed by #671
Assignees

Comments

@tbirdso
Copy link

tbirdso commented May 18, 2023

Overview

Observed an issue where itkwidgets.view(image) with three levels shows incorrect spacing for Level 1, while Level 0 and Level 2 appear correctly.

Level 0 (original image)

scale0

Level 1 (improper scale, pancake image)

scale1

Level 2 (apparently correct scale, lowest resolution)

scale2

Versions

itkwidgets[all]==1.0a32
itk==v5.3.0

Steps to Reproduce

3D image file: https://drive.google.com/file/d/1rKB7a65EIPvtNYclea1C7cvOhGhbCG_0/view?usp=share_link

itkwidgets.view(image)

Additional Notes

Unclear whether the image itself is downsampled with the wrong spacing or if the issue is limited to viewing. The fact that the subsequent level is correctly scaled seems to imply a viewing issue?

cc @thewtex @PaulHax

@tbirdso
Copy link
Author

tbirdso commented May 23, 2023

Note that this behavior appeared to be visible in ngff-zarr in Exercise 1(?) during the GYBT multiscale tutorial:

https://github.com/InsightSoftwareConsortium/GetYourBrainTogether/blob/main/HCK02_2023_Allen_Institute_Hybrid/Tutorials/LargeScaleRegistration/Multiscale_Spatial_Metadata_and_Distributed_Processing.ipynb

EDIT: Behavior may have been due to the size of the matplotlib pyplot view rather than improper image size in space. May be
worth investigating to confirm.

@tbirdso
Copy link
Author

tbirdso commented May 30, 2023

I was able to recreate the issue in neuroglancer, it appears that the origin is in ngff-zarr rather than itkwidgets. I have cross-posted the issue there with neuroglancer details: thewtex/ngff-zarr#33

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 a pull request may close this issue.

3 participants