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

[DAR-1590][External] Resize Axial annotation to isotropic #826

Conversation

shernshiou
Copy link
Contributor

@shernshiou shernshiou commented Apr 24, 2024

Problem

The volumetric_extractor at busybee, resize each AnatomicalPlane to isotropic 2D. Because of that, the annotation when imported must be resize to the specific dimension based on the pixdims. Without resizing, the mask will be shifted.

Solution

This PR implements the isotropic rescaling logic from volumetric_extractor so that the mask is resized matching the size of the Axial png from volumetric_extractor

Changelog

  • Fix the NifTI mask import to the size matching the one extracted by busybee.

@shernshiou shernshiou changed the title [DAR-1590][external] Resize Axial annotation to isotropic [DAR-1590][External] Resize Axial annotation to isotropic Apr 24, 2024
@shernshiou shernshiou force-pushed the dar-1590-bug-nifti-annotations-appear-slightly-displaced-in-the branch 2 times, most recently from e081adf to ebea28e Compare April 24, 2024 13:17
@shernshiou shernshiou self-assigned this Apr 24, 2024
@shernshiou shernshiou marked this pull request as ready for review April 24, 2024 13:22
@shernshiou shernshiou force-pushed the dar-1590-bug-nifti-annotations-appear-slightly-displaced-in-the branch from ebea28e to 828269a Compare April 25, 2024 06:47
@shernshiou shernshiou force-pushed the dar-1590-bug-nifti-annotations-appear-slightly-displaced-in-the branch 2 times, most recently from 3708a57 to bdb8e19 Compare April 25, 2024 13:04
@shernshiou shernshiou merged commit bc91e5e into master Apr 30, 2024
32 checks passed
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