FNNDSC / ami

AMI Medical Imaging (AMI) JS ToolKit
https://fnndsc.github.io/ami/#viewers_upload
MIT License
717 stars 213 forks source link

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

Open ppageon opened 4 years ago

ppageon commented 4 years ago

The function core.utils.ijk2LPS seems wrong in Nifti scenario, the original code as follow: image when I test a case with Nifti image with spacing like [0.5, 1.0, 0.75], the rendered images are distortion. image It looked fine when I swap the spacing.y with spacing.x, like: image 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.