trendscenter / Coinstac_VBM_computation

This algorithm runs the Voxel Based Morphometry on structural T1 weighted MRI scans using SPMv12 and MATLAB Runtimev713.
3 stars 0 forks source link

Container does not work #33

Open everner opened 3 years ago

everner commented 3 years ago

Ross is trying to update the container for the new COINSTAC image.

When run in coinstac simulator, it produces this error:

"Cannot set the undefined 'mrf_weighting' attribute of a 'NewSegmentInputSpec' object."

everner commented 3 years ago

According to Helen:

BTW, mrf_weighting has to do with segmentation, mostly with SPM8. Not totally sure why it's in there at all, since we're using the New Segment.

Nipype has a default value for mrf_weighting (0.15), but Sandeep has initialized it to 1.0, and I don't know why

Not sure I can provide any more info on the error alone. She would need to run it to get more info.

everner commented 3 years ago

The problem was that the file named Dockerfile was not the Dockerfile that actually worked. The real one was Dockerfile_build_from_scratch, which included a patch that overwrote the NewSegment class, adding the mrf_weighting input parameter. This repo needs to be cleaned up to prevent these types of problems in the future.