-
Notifications
You must be signed in to change notification settings - Fork 97
Add option to install CAT12 #407
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
Comments
I'm also interested in this feature. |
Cat12 comes in an MCR version: http://141.35.69.218/cat12/ so modifying the SPM12 file would be a good start: https://github.com/ReproNim/neurodocker/blob/master/neurodocker/templates/spm12.yaml |
I previously tried building an image which using "cat12_latest_R2017b_MCR_Linux.zip" However when I tried running CAT12Segment from the nipype interface I got the following error shown below. Upon further inspection, it seemed to me like many of the matlab files included with SPM are not in the CAT12 standalone MCR. Granted, I know very little about Matlab or SPM / CAT12 so I may have made a mistake. /usr/local/lib/python3.8/dist-packages/nipype/interfaces/base/core.py in run(self, cwd, ignore_exception, **inputs) /usr/local/lib/python3.8/dist-packages/nipype/interfaces/spm/base.py in _run_interface(self, runtime) /usr/local/lib/python3.8/dist-packages/nipype/interfaces/base/core.py in run(self, cwd, ignore_exception, **inputs) /usr/local/lib/python3.8/dist-packages/nipype/interfaces/matlab.py in _run_interface(self, runtime) /usr/local/lib/python3.8/dist-packages/nipype/interfaces/base/core.py in raise_exception(self, runtime) RuntimeError: Command:
|
I build a container using the official CAT12 MCR release (http://141.35.69.218/cat12/CAT${CAT_VERSION}_${CAT_REVISION}_${MATLAB_VERSION}_MCR_Linux.zip ) and I also run into this problem. It seems that there is a custom function
that breaks it. Maybe @ChristianGaser has an idea what this function is and why it didn't get compiled for the MCR. I also tested @gllmflndn MCR SPM version with many toolboxes included (https://www.fil.ion.ucl.ac.uk/spm/download/restricted/utopia/dev/spm12_latest_BI_Linux_R2019b.zip) and run into another issue, but this time caused by the SnPM toolbox:
However, running SPM via Nipype works perfectly in the release version of SPM MCR (https://www.fil.ion.ucl.ac.uk/spm/download/restricted/bids/spm12_r7771_Linux_R2019b.zip) |
Yes I want to echo that building a container with nipype + SPM12 r7771 MCR R2019b works for me as well! There seems to be issues with nipype + CAT12 MCR release specifically. |
I will have a look at the issue with SnPM - this happens in the context of the development version of SPM so this is a side issue. I would prefer if the "official" standalone version of SPM was used for nipype + SPM12. |
Christian Gaser found the problem and fixed it in this build :) http://www.neuro.uni-jena.de/cat12/CAT12.8_r1933_R2017b_MCR_Linux.zip - I am just building a new image for Neurodesk and then I have a look at writing the build recipe for neurodocker :) |
thank you @stebo85 and everyone else! |
This issue is stale because it has been open for 30 days with no activity. |
Recently, A CAT12 interface was added to nipype by @mfmachado. This interface was tested using a bare-metal installation of SPM and CAT12. Furthermore, @m-wierzba has already created a singularity image that uses the standalone version from CAT12. So I guess it could be possible to bring those two together in NeuroDocker to be able to use Nipype+CAT12 in a containerized form?
The text was updated successfully, but these errors were encountered: