reduce number of SPM calls

Author: chrisgorgo

nipype/interfaces/base.py

@@ -925,34 +925,36 @@ def _check_version_requirements(self, trait_object, raise_exception=True):
         """ Raises an exception on version mismatch
         """
         unavailable_traits = []
-        version = LooseVersion(str(self.version))
-        if not version:
-            return
         # check minimum version
         check = dict(min_ver=lambda t: t is not None)
         names = trait_object.trait_names(**check)
-        for name in names:
-            min_ver = LooseVersion(str(trait_object.traits()[name].min_ver))
-            if min_ver > version:
-                unavailable_traits.append(name)
-                if not isdefined(getattr(trait_object, name)):
-                    continue
-                if raise_exception:
-                    raise Exception('Trait %s (%s) (version %s < required %s)' %
-                                    (name, self.__class__.__name__,
-                                     version, min_ver))
-        check = dict(max_ver=lambda t: t is not None)
-        names = trait_object.trait_names(**check)
-        for name in names:
-            max_ver = LooseVersion(str(trait_object.traits()[name].max_ver))
-            if max_ver < version:
-                unavailable_traits.append(name)
-                if not isdefined(getattr(trait_object, name)):
-                    continue
-                if raise_exception:
-                    raise Exception('Trait %s (%s) (version %s > required %s)' %
-                                    (name, self.__class__.__name__,
-                                     version, max_ver))
+        
+        if names:
+            version = LooseVersion(str(self.version))
+            if not version:
+                return
+            for name in names:
+                min_ver = LooseVersion(str(trait_object.traits()[name].min_ver))
+                if min_ver > version:
+                    unavailable_traits.append(name)
+                    if not isdefined(getattr(trait_object, name)):
+                        continue
+                    if raise_exception:
+                        raise Exception('Trait %s (%s) (version %s < required %s)' %
+                                        (name, self.__class__.__name__,
+                                         version, min_ver))
+            check = dict(max_ver=lambda t: t is not None)
+            names = trait_object.trait_names(**check)
+            for name in names:
+                max_ver = LooseVersion(str(trait_object.traits()[name].max_ver))
+                if max_ver < version:
+                    unavailable_traits.append(name)
+                    if not isdefined(getattr(trait_object, name)):
+                        continue
+                    if raise_exception:
+                        raise Exception('Trait %s (%s) (version %s > required %s)' %
+                                        (name, self.__class__.__name__,
+                                         version, max_ver))
         return unavailable_traits
 
     def _run_wrapper(self, runtime):

nipype/interfaces/spm/preprocess.py

@@ -765,16 +765,21 @@ class Segment(SPMCommand):
     >>> seg.run() # doctest: +SKIP
 
     """
-
-    if Info.version() and Info.version()['name'] == "SPM12":
-        _jobtype = 'tools'
-        _jobname = 'oldseg'
-    else:
-        _jobtype = 'spatial'
-        _jobname = 'preproc'
+    
 
     input_spec = SegmentInputSpec
     output_spec = SegmentOutputSpec
+    
+    def __init__(self, **inputs):
+        version = Info.version()
+        if version and version['name'] == "SPM12":
+            self._jobtype = 'tools'
+            self._jobname = 'oldseg'
+        else:
+            self._jobtype = 'spatial'
+            self._jobname = 'preproc'
+        
+        SPMCommand.__init__(self, **inputs)
 
     def _format_arg(self, opt, spec, val):
         """Convert input to appropriate format for spm
@@ -893,12 +898,17 @@ class NewSegment(SPMCommand):
 
     input_spec = NewSegmentInputSpec
     output_spec = NewSegmentOutputSpec
-    if Info.version() and Info.version()['name'] == "SPM12":
-        _jobtype = 'spatial'
-        _jobname = 'preproc'
-    else:
-        _jobtype = 'tools'
-        _jobname = 'preproc8'
+        
+    def __init__(self, **inputs):
+        version = Info.version()
+        if version and version['name'] == "SPM12":
+            self._jobtype = 'tools'
+            self._jobname = 'oldseg'
+        else:
+            self._jobtype = 'spatial'
+            self._jobname = 'preproc'
+        
+        SPMCommand.__init__(self, **inputs)
 
     def _format_arg(self, opt, spec, val):
         """Convert input to appropriate format for spm