-
Notifications
You must be signed in to change notification settings - Fork 132
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
T2w and acq-hippo_T2w mixed up in report #1303
Comments
I believe #1317 may have resolved this -- if not completely, it definitely has a better approach to identify BIDS entities in the input file so a solution to this problem could be easy. Thanks for the test dataset -- will check ASAP. |
I reran it with 24.0.1 on our HPC. It crashed. I'm attaching the slurm log and mriqc logs in case they are useful. |
I reran with 24.0.2 and it is much better: |
I'm quite overloaded so I'm not sure I'll be able to replicate this myself. Could you post screen captures of what you see? |
@oesteban Here's a tar file with the results in it: https://www.dropbox.com/scl/fi/xkkia8d9ad2rktpbx4qhb/mriqc_dev.tar?rlkey=t3vlxew717q8kexcucjzc6z7t&st=lr4dmpcs&dl=0 Two T2w reports were generated: sub-CAM003_ses-01brain_T2w.html and sub-CAM003_ses-01brain_acq-hippo_T2w.html. That seems perfect. The sub-CAM003_ses-01brain_acq-hippo_T2w.html is exactly what one would hope (it contains only results and images for this partial-volume T2w image). However, the sub-CAM003_ses-01brain_T2w.html report contains data for both the T2w image AND the T2w-hippo image (i.e., twice as much data as should have been produced). Obviously, this is not a devastating issue as it just means the T2w-hippo data is replicated in 2 places. I've attached a screenshot from sub-CAM003_ses-01brain_T2w.html to illustrate. |
What happened?
I ran MRIQC on data that has two T2w images: one is a partial volume hippocampus image; the other is a full volume T2w image.
What command did you use?
What version of the software are you running?
Running MRIQC version 24.1.0.dev0+g3fe90466.d20240417
How are you running this software?
Singularity
Is your data BIDS valid?
Yes
Are you reusing any previously computed results?
No
Please copy and paste any relevant log output.
Additional information / screenshots
It seems that separate svgs are being generated in the figures directory BUT the actual report for both the T2w and the acq-hippo_T2w are including the acq-hippo_T2w images and ignoring the regular T2w images.
A suitable test BIDS dataset with both types of T2w image is available here.
The text was updated successfully, but these errors were encountered: