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

Should we reorient this template as RPI? #16

Open
jcohenadad opened this issue Mar 18, 2021 · 2 comments
Open

Should we reorient this template as RPI? #16

jcohenadad opened this issue Mar 18, 2021 · 2 comments

Comments

@jcohenadad
Copy link
Member

Currently, the template is oriented as LPI ("Neurological" convention). This has highlighted a problem when registering to a template spinalcordtoolbox/spinalcordtoolbox#3300.

Regardless this problem (which should be solved whatever orientation the template is), most templates are in fact RPI ("Radiological" convention), not LPI. Example:

  • SCT PAM50
  • MNI template

So, it would be more consistent to keep with the convention.

Also see: http://www.grahamwideman.com/gw/brain/orientation/orientterms.htm

@charleygros
Copy link
Member

I think this has been fixed in a merged PR:

chgroc@rosenberg:~/exvivo-template$ sct_image -i template/exvivo_t2.nii.gz -getorient

--
Spinal Cord Toolbox (git-master-af2d4de531f66cd062af6fefa87c3f84185875ff)

sct_image -i template/exvivo_t2.nii.gz -getorient
--

RPI

Closing this issue.

@jcohenadad
Copy link
Member Author

Reopening because fslhd says LPI (and I trust fslhd more than sct_image 😅 ):

 fslhd exvivo_t2.nii.gz 
[...]
qform_xorient	Left-to-Right
qform_yorient	Posterior-to-Anterior
qform_zorient	Inferior-to-Superior

version of the dataset: 6a595f9

There might be a but in sct_image. Issue opened: spinalcordtoolbox/spinalcordtoolbox#3311

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

No branches or pull requests

2 participants