-
Notifications
You must be signed in to change notification settings - Fork 6
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
mriqc_clf
T1w classifier not working in mriqc latest release on docker
#8
Comments
This should have been fixed in 55269f8de6389efa4f60373eb40f269c1476e4cd (part of nipreps/mriqc#927). Can you try building your Singularity image on 21.0.0rc2? |
I ran into the same problem! Here is the error I got:
If I use version 0.16.0 instead, I can get a similar output file as Mengjia. However, here rises a question: since my IQMs were computed by version 21.0.0rc2, can I trust the prediction I got from version 0.16.0? |
The MRIQC classifier aged pretty badly. We have outsourced it to the nipreps/mriqc-learn package, where I'm also moving this issue for reference. |
Hello,
I am interesting in the T1w classifier function
mriqc_clf
which seems to be no longer supported in the latest releaseOn my system, container image files for both the latest and second latest version (0.16.1) of MRIQC were downloaded from docker hub.
When I ran the classifier on the latest version
singularity exec -H ~/mengjia_space -B ~/mengjia_space:/output ~/mriqc-21.0.0.sif mriqc_clf --load-classifier -X group_T1w.tsv -v
I got the error
However, when I ran the same command with the second latest (0.16.1) version
singularity exec -H ~/mengjia_space -B ~/mengjia_space:/output ~/mriqc_0.16.1.sif mriqc_clf --load-classifier -X group_T1w.tsv -v
, I was able to obtain the output file with prediction labels.Is it true that
mriqc_clf
is no longer supported? Thank you so much for your help and feedback!!The text was updated successfully, but these errors were encountered: