STY: Apply ruff/flake8-comprehensions rule C414

C414 Unnecessary `list` call

Author: DimitriPapadopoulos

nipype/interfaces/base/core.py

@@ -486,7 +486,7 @@ def load_inputs_from_json(self, json_file, overwrite=True):
         if not overwrite:
             def_inputs = list(self.inputs.get_traitsfree().keys())
 
-        new_inputs = list(set(list(inputs_dict.keys())) - set(def_inputs))
+        new_inputs = list(set(inputs_dict.keys()) - set(def_inputs))
         for key in new_inputs:
             if hasattr(self.inputs, key):
                 setattr(self.inputs, key, inputs_dict[key])

nipype/pipeline/engine/nodes.py

@@ -240,7 +240,7 @@ def needed_outputs(self):
     @needed_outputs.setter
     def needed_outputs(self, new_outputs):
         """Needed outputs changes the hash, refresh if changed"""
-        new_outputs = sorted(list(set(new_outputs or [])))
+        new_outputs = sorted(set(new_outputs or []))
         if new_outputs != self._needed_outputs:
             # Reset hash
             self._hashvalue = None

nipype/scripts/instance.py

@@ -39,7 +39,7 @@ def list_interfaces(module):
     the given module.
     """
     iface_names = []
-    for k, v in sorted(list(module.__dict__.items())):
+    for k, v in sorted(module.__dict__.items()):
         if inspect.isclass(v) and issubclass(v, Interface):
             iface_names.append(k)
     return iface_names

nipype/utils/nipype_cmd.py

@@ -13,7 +13,7 @@ def listClasses(module=None):
         __import__(module)
         pkg = sys.modules[module]
         print("Available Interfaces:")
-        for k, v in sorted(list(pkg.__dict__.items())):
+        for k, v in sorted(pkg.__dict__.items()):
             if inspect.isclass(v) and issubclass(v, Interface):
                 print("\t%s" % k)