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

[BUG] ACPC in surface RAS, should be in Scanner RAS #989

Closed
alexrockhill opened this issue Mar 29, 2022 · 0 comments · Fixed by #990
Closed

[BUG] ACPC in surface RAS, should be in Scanner RAS #989

alexrockhill opened this issue Mar 29, 2022 · 0 comments · Fixed by #990

Comments

@alexrockhill
Copy link
Contributor

I realized that in https://mne.tools/mne-bids/dev/auto_examples/convert_ieeg_to_bids.html the ACPC coordinates are in surface RAS but in https://mne.tools/stable/auto_tutorials/clinical/10_ieeg_localize.html we tell users (correctly) to align scanner RAS to ACPC. I can make a PR, it might be nice to use the Freesurfer T1 in a convenience function that converts a montage to scanner RAS. Although someone could share a T1 that isn't 256 x 256 x 256 and then the Freesurfer T1 wouldn't be the same size, but, I think, since RAS is in mm/m, it should be compensated for by the affine and should be fine to use the Freesurfer T1 even in that instance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant