FIX: Fixes the 'out_file' and out_datatype inputs for MRIsConvert.

If the 'out_file' input was set in MRIsConvert it would cause and error because
the 'out_file' would not match the auto generated 'out_file' name. This patch
allows 'out_file' to be set.

Author: David Ellis

nipype/interfaces/freesurfer/tests/test_auto_MRIsConvert.py

@@ -30,11 +30,12 @@ def test_MRIsConvert_inputs():
     ),
     origname=dict(argstr='-o %s',
     ),
-    out_datatype=dict(mandatory=True,
+    out_datatype=dict(xor=['out_file'],
     ),
-    out_file=dict(argstr='./%s',
+    out_file=dict(argstr='%s',
     genfile=True,
     position=-1,
+    xor=['out_datatype'],
     ),
     parcstats_file=dict(argstr='--parcstats %s',
     ),

nipype/interfaces/freesurfer/utils.py

@@ -814,10 +814,10 @@ class MRIsConvertInputSpec(FSTraitedSpec):
     origname = traits.String(argstr="-o %s", desc="read orig positions")
 
     in_file = File(exists=True, mandatory=True, position=-2, argstr='%s', desc='File to read/convert')
-    out_file = File(argstr='./%s', position=-1, genfile=True, desc='output filename or True to generate one')
-    # Not really sure why the ./ is necessary but the module fails without it
+    out_file = File(argstr='%s', position=-1, genfile=True, xor=['out_datatype'],
+                    desc='output filename or True to generate one')
 
-    out_datatype = traits.Enum("ico", "tri", "stl", "vtk", "gii", "mgh", "mgz", mandatory=True,
+    out_datatype = traits.Enum("ico", "tri", "stl", "vtk", "gii", "mgh", "mgz", xor=['out_file'],
                                desc="These file formats are supported:  ASCII:       .asc"
                                "ICO: .ico, .tri GEO: .geo STL: .stl VTK: .vtk GIFTI: .gii MGH surface-encoded 'volume': .mgh, .mgz")
 
@@ -853,12 +853,14 @@ def _list_outputs(self):
 
     def _gen_filename(self, name):
         if name is 'out_file':
-            return self._gen_outfilename()
+            return os.path.abspath(self._gen_outfilename())
         else:
             return None
 
     def _gen_outfilename(self):
-        if isdefined(self.inputs.annot_file):
+        if isdefined(self.inputs.out_file):
+            return self.inputs.out_file
+        elif isdefined(self.inputs.annot_file):
             _, name, ext = split_filename(self.inputs.annot_file)
         elif isdefined(self.inputs.parcstats_file):
             _, name, ext = split_filename(self.inputs.parcstats_file)