Repository: nipy/nipype

Issue:
JistIntensityMp2rageMasking interface incompatible with current CBS Tools release
For the command underlying the interface mipav.JistIntensityMp2rageMasking the parameter names have changed in the current CBS Tools release, which makes the interface fail. I meant to do a PR but then saw the mipav/developer.py file is autogenerated and should not be be edited. 

These would be the required changes:
https://github.com/nipy/nipype/compare/master...juhuntenburg:fix/cbstools_params



Discussion:

Resolving patch:
diff --git a/nipype/interfaces/mipav/developer.py b/nipype/interfaces/mipav/developer.py
--- a/nipype/interfaces/mipav/developer.py
+++ b/nipype/interfaces/mipav/developer.py
@@ -722,10 +722,10 @@ class JistIntensityMp2rageMaskingInputSpec(CommandLineInputSpec):
     inSkip = traits.Enum("true", "false", desc="Skip zero values", argstr="--inSkip %s")
     inMasking = traits.Enum("binary", "proba", desc="Whether to use a binary threshold or a weighted average based on the probability.", argstr="--inMasking %s")
     xPrefExt = traits.Enum("nrrd", desc="Output File Type", argstr="--xPrefExt %s")
-    outSignal = traits.Either(traits.Bool, File(), hash_files=False, desc="Signal Proba Image", argstr="--outSignal %s")
-    outSignal2 = traits.Either(traits.Bool, File(), hash_files=False, desc="Signal Mask Image", argstr="--outSignal2 %s")
-    outMasked = traits.Either(traits.Bool, File(), hash_files=False, desc="Masked T1 Map Image", argstr="--outMasked %s")
-    outMasked2 = traits.Either(traits.Bool, File(), hash_files=False, desc="Masked Iso Image", argstr="--outMasked2 %s")
+    outSignal = traits.Either(traits.Bool, File(), hash_files=False, desc="Signal Proba Image", argstr="--outSignal_Proba %s")
+    outSignal2 = traits.Either(traits.Bool, File(), hash_files=False, desc="Signal Mask Image", argstr="--outSignal_Mask %s")
+    outMasked = traits.Either(traits.Bool, File(), hash_files=False, desc="Masked T1 Map Image", argstr="--outMasked_T1_Map %s")
+    outMasked2 = traits.Either(traits.Bool, File(), hash_files=False, desc="Masked Iso Image", argstr="--outMasked_T1weighted %s")
     null = traits.Str(desc="Execution Time", argstr="--null %s")
     xDefaultMem = traits.Int(desc="Set default maximum heap size", argstr="-xDefaultMem %d")
     xMaxProcess = traits.Int(1, desc="Set default maximum number of processes.", argstr="-xMaxProcess %d", usedefault=True)


Test patch:
diff --git a/nipype/interfaces/mipav/tests/test_auto_JistIntensityMp2rageMasking.py b/nipype/interfaces/mipav/tests/test_auto_JistIntensityMp2rageMasking.py
--- a/nipype/interfaces/mipav/tests/test_auto_JistIntensityMp2rageMasking.py
+++ b/nipype/interfaces/mipav/tests/test_auto_JistIntensityMp2rageMasking.py
@@ -26,16 +26,16 @@ def test_JistIntensityMp2rageMasking_inputs():
     ),
     null=dict(argstr='--null %s',
     ),
-    outMasked=dict(argstr='--outMasked %s',
+    outMasked=dict(argstr='--outMasked_T1_Map %s',
     hash_files=False,
     ),
-    outMasked2=dict(argstr='--outMasked2 %s',
+    outMasked2=dict(argstr='--outMasked_T1weighted %s',
     hash_files=False,
     ),
-    outSignal=dict(argstr='--outSignal %s',
+    outSignal=dict(argstr='--outSignal_Proba %s',
     hash_files=False,
     ),
-    outSignal2=dict(argstr='--outSignal2 %s',
+    outSignal2=dict(argstr='--outSignal_Mask %s',
     hash_files=False,
     ),
     terminal_output=dict(nohash=True,