FNNDSC / FNNDSC/ami

spacing.y and spacing.x in function core.utils.ijk2LPS should be swapped in Nifti scenario

Open
#402 0 comments 2 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
745
Forks
212
PR merge metrics
No merged PRs in 30d

Description

The function core.utils.ijk2LPS seems wrong in Nifti scenario, the original code as follow:
![image](https://user-images.githubusercontent.com/21356208/70766003-6fbac600-1d97-11ea-952e-4bce78b0684e.png)
when I test a case with Nifti image with spacing like [0.5, 1.0, 0.75], the rendered images are distortion.
![image](https://user-images.githubusercontent.com/21356208/70766460-bceb6780-1d98-11ea-92de-2da6af818672.png)
It looked fine when I swap the spacing.y with spacing.x, like:
![image](https://user-images.githubusercontent.com/21356208/70766625-44d17180-1d99-11ea-9345-915249082412.png)
So I think it should be wrong to multiply x component of Cos with y component of spacing, at least it doesn't work for Nifti scenario.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.