-
Notifications
You must be signed in to change notification settings - Fork 1
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
Complex ICA with GIFT #5
Comments
I might be able to base Nipype interfaces on https://github.com/trendscenter/coinstac_spatially_constrained_ica/tree/master/nipype-0.10.0/nipype/interfaces/gift |
Including the MATLAB runtime compiler in a Docker image apparently massively increases the size of the image (see nipreps/fmriprep#3308 (comment)), so this might not be doable. |
Maybe I can take https://github.com/afbujan/complex_ica and bring it up to date with sklearn's FastICA. |
@bpinsard do you recall exactly how much space MCR took up in your NORDIC image? I'm finding trying to translate complex-ICA to Python difficult enough that I'm tempted to just bundle GIFT into the image. |
The images I built using the matlab docker tools were about 3GB. |
Thanks @bpinsard! I think I might be willing to accept that kind of size for GIFT at least. I get why we want to avoid it for NORDIC in fMRIPrep though. |
Maybe there are smarter ways to package the MCR https://hub.docker.com/search?q=matlabruntime show some non-official images with smaller sizes. |
What would you like to see added in this software?
Would be cool to include complex ICA with GIFT. GIFT is a MATLAB toolbox, so this might be tough to include in a container.
Do you have any interest in helping implement the feature?
Yes
Additional information / screenshots
We might be able to base the Docker image on https://github.com/trendscenter/gift-bids and/or https://hub.docker.com/r/trends/gift-bids. Or maybe https://hub.docker.com/r/trendscenter/gift?
Relevant issue: trendscenter/gift#25
The text was updated successfully, but these errors were encountered: