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

Query for fMRIprep and MRIQC for T1w images is differnet #866

Closed
jpurcel8 opened this issue Oct 14, 2020 · 5 comments · Fixed by #942
Closed

Query for fMRIprep and MRIQC for T1w images is differnet #866

jpurcel8 opened this issue Oct 14, 2020 · 5 comments · Fixed by #942

Comments

@jpurcel8
Copy link

Hello,
fMRIPrep looks for T1w images with the query get(suffix='T1w', dataype='anat'), while MRIQC appears to use a different mechanism. Can the MRIQC mechanism be set to the same one that fMRIPrep uses?
Thanks,

@oesteban
Copy link
Member

Can you point me to the particular section of MRIQC code you think we could make more consistent with fMRIPrep?

@jpurcel8
Copy link
Author

fMRIPrep looks for T1w images with the query get(suffix='T1w', dataype='anat'), while MRIQC appears to use a different mechanism. I don't know what this other mechanism is per se, but the consequence is that when I have non-standard T1w images, then MRIQC does not run. (e.g. T1w images that only slice through the hippocampus). While, of course, it runs fine with fMRIprep.
Thanks!

@mgxd
Copy link
Contributor

mgxd commented Oct 15, 2020

The relevant bits are here:
https://github.com/poldracklab/mriqc/blob/5898396026817ce114bbc90bac0dd5d7c48618e0/mriqc/cli/parser.py#L449-L461

https://github.com/poldracklab/mriqc/blob/ee433f28d0c4d84eea2d050c54002ca823c86531/mriqc/utils/bids.py#L9-L32

but the consequence is that when I have non-standard T1w images, then MRIQC does not run.

@jpurcel8 could you share this T1w's filename, and what options you are passing in to MRIQC?

@jpurcel8
Copy link
Author

jpurcel8 commented Oct 15, 2020 via email

@effigies
Copy link
Member

Linking in the neurostars issue.

@jpurcel8 has some wonky T1w images in misc/*_T1w.nii.gz. fMRIPrep avoids these because we request datatype='anat', while MRIQC only queries for suffix.

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

Successfully merging a pull request may close this issue.

4 participants