Merge pull request #3689 from DimitriPapadopoulos/E

STY: Apply ruff/pycodestyle rules (E)

Author: effigies

nipype/algorithms/modelgen.py

@@ -7,7 +7,9 @@
 experiments.
 """
 from copy import deepcopy
-import csv, math, os
+import csv
+import math
+import os
 
 from nibabel import load
 import numpy as np

nipype/interfaces/base/tests/test_support.py

@@ -35,7 +35,7 @@ def test_bunch_methods():
     assert b.get("a") == 3
     assert b.get("badkey", "otherthing") == "otherthing"
     assert b != newb
-    assert type(dict()) == type(newb)
+    assert type(newb) is dict
     assert newb["a"] == 3
 
 

nipype/interfaces/fsl/tests/test_base.py

@@ -37,7 +37,7 @@ def test_FSLCommand():
     # testing the one item that is not.
     cmd = fsl.FSLCommand(command="ls")
     res = cmd.run()
-    assert type(res) == InterfaceResult
+    assert type(res) is InterfaceResult
 
 
 @pytest.mark.skipif(no_fsl(), reason="fsl is not installed")

nipype/pipeline/engine/tests/test_workflows.py

@@ -20,7 +20,7 @@ def test_init():
     with pytest.raises(TypeError):
         pe.Workflow()
     pipe = pe.Workflow(name="pipe")
-    assert type(pipe._graph) == nx.DiGraph
+    assert type(pipe._graph) is nx.DiGraph
 
 
 def test_connect():

nipype/pipeline/plugins/dagman.py

@@ -78,7 +78,7 @@ def __init__(self, **kwargs):
         ):
             if (
                 "plugin_args" in kwargs
-                and not kwargs["plugin_args"] is None
+                and kwargs["plugin_args"] is not None
                 and id_ in kwargs["plugin_args"]
             ):
                 if id_ == "wrapper_cmd":
@@ -89,7 +89,7 @@ def __init__(self, **kwargs):
                     val = self._get_str_or_file(kwargs["plugin_args"][id_])
             setattr(self, var, val)
         # TODO remove after some time
-        if "plugin_args" in kwargs and not kwargs["plugin_args"] is None:
+        if "plugin_args" in kwargs and kwargs["plugin_args"] is not None:
             plugin_args = kwargs["plugin_args"]
             if "template" in plugin_args:
                 warn(

nipype/scripts/utils.py

@@ -71,12 +71,12 @@ def add_args_options(arg_parser, interface):
             if not spec.is_trait_type(traits.TraitCompound):
                 trait_type = type(spec.trait_type.default_value)
             if trait_type in (bytes, str, int, float):
-                if trait_type == bytes:
+                if trait_type is bytes:
                     trait_type = str
                 args["type"] = trait_type
         elif len(spec.inner_traits) == 1:
             trait_type = type(spec.inner_traits[0].trait_type.default_value)
-            if trait_type == bytes:
+            if trait_type is bytes:
                 trait_type = str
             if trait_type in (bytes, bool, str, int, float):
                 args["type"] = trait_type

tools/checkspecs.py

@@ -301,7 +301,7 @@ def test_specs(self, uri):
                         bad_specs.append(
                             [uri, c, "Inputs", traitname, "mandatory=False"]
                         )
-                    if key == "usedefault" and trait.__dict__[key] == False:
+                    if key == "usedefault" and trait.__dict__[key] is False:
                         bad_specs.append(
                             [uri, c, "Inputs", traitname, "usedefault=False"]
                         )