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

ENH: Improve support for infant MRI templates #581

Open
2 tasks
larsoner opened this issue Aug 1, 2022 · 1 comment
Open
2 tasks

ENH: Improve support for infant MRI templates #581

larsoner opened this issue Aug 1, 2022 · 1 comment
Assignees
Milestone

Comments

@larsoner
Copy link
Member

larsoner commented Aug 1, 2022

As part of #574, I've created a single subject with a mne coreg warp of the ANTS6-0Months3T infant template provided by fetch_infant_template. There is currently no "nice" way for me to include this AFAIK in the dataset. I see two options (maybe we want to support both?):

1. Include as freesurfer derivative

I could add the warped MRI as derivatives/freesurfer/<subject>, but mne-bids-pipeline doesn't seem to have the logic to look for already-existing BEM solutions. I could add this support and upload to openneuro with the warped MRI as a FreeSurfer derivative, then I think everything could "just work".

2. Add support for non-fsaverage templates

This would be a potentially cleaner solution since it would only require me shipping one small text file rather than a whole FreeSurfer subject directory. It would require something like

  • Add support for non-fsaverage templates in MNE-BIDS-Pipeline (at least the ANTS* templates).
  • Include in our BIDS derivatives (somewhere?) the -trans.fif plus the MRI scaling parameters.cfg file that tells mne.coreg.scale_mri how to scale ANTS6-0Months3T to make the template subject. This could be done in a backward compatible way for fsaverage where, if not found, uses the current trans (and uniform scaling?) method.

From this the necessary FreeSurfer subjects could be created.

Notably, in either of these cases, the BEM code would need to be updated a bit -- the data are MEG-only, which triggers MNE-BIDS-Pipeline to want to use 1-layer BEMs, but the ones for ANTS* templates are all 3-layer (and I think it should be okay/acceptable to use 3-layer BEMs with MEG data if you want and they are already available).

We could have a quick video chat about this at some point to hash out the ideas @hoechenberger @agramfort

@agramfort
Copy link
Member

agramfort commented Aug 2, 2022 via email

@larsoner larsoner added this to the 0.2 milestone Nov 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

3 participants