Merge pull request #1026 from oesteban/fix/DefaultValueBreaksMake

FIX: Certain default values (i.e. np.array) crash make check-before-commit

Author: satra

CHANGES

@@ -1,5 +1,6 @@
 Next release
 ============
+* FIX: Bug crashed 'make api' (https://github.com/nipy/nipype/pull/1026)
 * ENH: Updated antsIntroduction to handle RA and RI registrations (https://github.com/nipy/nipype/pull/1009)
 * ENH: Updated N4BiasCorrection input spec to include weight image and spline order. Made 
 	   argument formatting consistent. Cleaned ants.segmentation according to PEP8.

nipype/interfaces/ants/legacy.py

@@ -96,7 +96,7 @@ def _list_outputs(self):
         outputs = self._outputs().get()
         transmodel = self.inputs.transformation_model
 
-        # When transform is set as 'RI'/'RA', wrap fields should not be expected 
+        # When transform is set as 'RI'/'RA', wrap fields should not be expected
         # The default transformation is GR, which outputs the wrap fields
         if not isdefined(transmodel) or (isdefined(transmodel) and transmodel not in ['RI', 'RA']):
             outputs['warp_field'] = os.path.join(os.getcwd(),

nipype/interfaces/base.py

@@ -757,10 +757,14 @@ def _get_trait_desc(self, inputs, name, spec):
 
         manhelpstr = ['\t%s' % name]
 
-        type_info = spec.full_info(inputs, name, None) 
-        default = ', nipype default value: %s' % spec.default_value()[1]
-        line = "(%s%s)" % (type_info, default if spec.usedefault else '')
-        manhelpstr = wrap(line, 70, 
+        type_info = spec.full_info(inputs, name, None)
+        default = ''
+
+        if spec.usedefault:
+            default = ', nipype default value: %s' % str(spec.default_value()[1])
+
+        line = "(%s%s)" % (type_info, default)
+        manhelpstr = wrap(line, 70,
                           initial_indent=manhelpstr[0]+': ',
                           subsequent_indent='\t\t ')