FIX: Drop Bunch.keys, use existing dictcopy method

Author: effigies

nipype/interfaces/base/core.py

@@ -500,7 +500,7 @@ def run(self, cwd=None, ignore_exception=None, **inputs):
             platform=platform.platform(),
             hostname=platform.node(),
             version=self.version)
-        runtime_attrs = set(runtime.keys())
+        runtime_attrs = set(runtime.dictcopy())
 
         mon_sp = None
         if enable_rm:
@@ -541,7 +541,7 @@ def run(self, cwd=None, ignore_exception=None, **inputs):
             if not ignore_exception:
                 raise
         finally:
-            if runtime is None or runtime_attrs - set(runtime.keys()):
+            if runtime is None or runtime_attrs - set(runtime.dictcopy()):
                 raise RuntimeError("{} interface failed to return valid "
                                    "runtime object".format(
                                        interface.__class__.__name__))

nipype/interfaces/base/support.py

@@ -68,9 +68,6 @@ def update(self, *args, **kwargs):
         Note: update is very much like HasTraits.set"""
         self.__dict__.update(*args, **kwargs)
 
-    def keys(self):
-        return list(self.__dict__.keys())
-
     def items(self):
         """iterates over bunch attributes as key, value pairs"""
         return list(self.__dict__.items())