STY: Apply ruff/flake8-bugbear rule B010

B010 Do not call `setattr` with a constant attribute value.
     It is not any safer than normal property access.

Author: DimitriPapadopoulos

Diff for: nipype/algorithms/tests/test_modelgen.py

@@ -122,7 +122,7 @@ def test_modelgen_spm_concat(tmpdir):
     s = SpecifySPMModel()
     s.inputs.input_units = "secs"
     s.inputs.concatenate_runs = True
-    setattr(s.inputs, "output_units", "secs")
+    s.inputs.output_units = "secs"
     assert s.inputs.output_units == "secs"
     s.inputs.functional_runs = [filename1, filename2]
     s.inputs.time_repetition = 6
@@ -147,7 +147,7 @@ def test_modelgen_spm_concat(tmpdir):
     )
 
     # Test case of scans as output units instead of seconds
-    setattr(s.inputs, "output_units", "scans")
+    s.inputs.output_units = "scans"
     assert s.inputs.output_units == "scans"
     s.inputs.subject_info = deepcopy(info)
     res = s.run()

Diff for: nipype/interfaces/afni/utils.py

@@ -3244,7 +3244,7 @@ def _run_interface(self, runtime):
             for line in runtime.stdout.split("\n")
             if line.strip().startswith("GCOR = ")
         ][-1]
-        setattr(self, "_gcor", float(gcor_line[len("GCOR = ") :]))
+        self._gcor = float(gcor_line[len("GCOR = ") :])
         return runtime
 
     def _list_outputs(self):

Diff for: nipype/interfaces/elastix/utils.py

@@ -173,9 +173,9 @@ def _get_outfile(self):
             return val
 
         if isdefined(self.inputs.output_file):
-            setattr(self, "_out_file", self.inputs.output_file)
+            self._out_file = self.inputs.output_file
             return self.inputs.output_file
 
         out_file = op.abspath(op.basename(self.inputs.transform_file))
-        setattr(self, "_out_file", out_file)
+        self._out_file = out_file
         return out_file

Diff for: nipype/interfaces/fsl/utils.py

@@ -929,7 +929,7 @@ def _run_interface(self, runtime):
                 float(r) for r in out["translations"].strip().split(" ")
             ]
 
-        setattr(self, "_results", outputs)
+        self._results = outputs
         return runtime
 
     def _list_outputs(self):
@@ -2513,8 +2513,8 @@ def _format_arg(self, name, trait_spec, value):
 
     def _parse_inputs(self, skip=None):
         fname, ext = op.splitext(self.inputs.in_coords)
-        setattr(self, "_in_file", fname)
-        setattr(self, "_outformat", ext[1:])
+        self._in_file = fname
+        self._outformat = ext[1:]
         first_args = super()._parse_inputs(skip=["in_coords", "out_file"])
 
         second_args = fname + ".txt"

Diff for: nipype/interfaces/spm/model.py

@@ -849,30 +849,22 @@ def _make_matlab_command(self, _):
 
     def aggregate_outputs(self, runtime=None):
         outputs = self._outputs()
-        setattr(outputs, "thresholded_map", self._gen_thresholded_map_filename())
-        setattr(outputs, "pre_topo_fdr_map", self._gen_pre_topo_map_filename())
+        outputs.thresholded_map = self._gen_thresholded_map_filename()
+        outputs.pre_topo_fdr_map = self._gen_pre_topo_map_filename()
         for line in runtime.stdout.split("\n"):
             if line.startswith("activation_forced = "):
-                setattr(
-                    outputs,
-                    "activation_forced",
-                    line[len("activation_forced = ") :].strip() == "1",
+                outputs.activation_forced = (
+                    line[len("activation_forced = ") :].strip() == "1"
                 )
             elif line.startswith("n_clusters = "):
-                setattr(
-                    outputs, "n_clusters", int(line[len("n_clusters = ") :].strip())
-                )
+                outputs.n_clusters = int(line[len("n_clusters = ") :].strip())
             elif line.startswith("pre_topo_n_clusters = "):
-                setattr(
-                    outputs,
-                    "pre_topo_n_clusters",
-                    int(line[len("pre_topo_n_clusters = ") :].strip()),
+                outputs.pre_topo_n_clusters = int(
+                    line[len("pre_topo_n_clusters = ") :].strip()
                 )
             elif line.startswith("cluster_forming_thr = "):
-                setattr(
-                    outputs,
-                    "cluster_forming_thr",
-                    float(line[len("cluster_forming_thr = ") :].strip()),
+                outputs.cluster_forming_thr = float(
+                    line[len("cluster_forming_thr = ") :].strip()
                 )
         return outputs
 

Diff for: nipype/interfaces/tests/test_io.py

@@ -517,7 +517,7 @@ def test_datasink_copydir_2(_temp_analyze_files, tmpdir):
         base_directory=tmpdir.mkdir("basedir").strpath, parameterization=False
     )
     ds.inputs.remove_dest_dir = True
-    setattr(ds.inputs, "outdir", pth)
+    ds.inputs.outdir = pth
     ds.run()
     sep = os.path.sep
     assert not tmpdir.join("basedir", pth.split(sep)[-1], fname).check()

Diff for: nipype/pipeline/engine/tests/test_engine.py

@@ -25,7 +25,7 @@
 def test_1mod(iterables, expected):
     pipe = pe.Workflow(name="pipe")
     mod1 = pe.Node(interface=EngineTestInterface(), name="mod1")
-    setattr(mod1, "iterables", iterables["1"])
+    mod1.iterables = iterables["1"]
     pipe.add_nodes([mod1])
     pipe._flatgraph = pipe._create_flat_graph()
     pipe._execgraph = pe.generate_expanded_graph(deepcopy(pipe._flatgraph))
@@ -49,7 +49,7 @@ def test_2mods(iterables, expected):
     mod1 = pe.Node(interface=EngineTestInterface(), name="mod1")
     mod2 = pe.Node(interface=EngineTestInterface(), name="mod2")
     for nr in ["1", "2"]:
-        setattr(eval("mod" + nr), "iterables", iterables[nr])
+        eval("mod" + nr).iterables = iterables[nr]
     pipe.connect([(mod1, mod2, [("output1", "input2")])])
     pipe._flatgraph = pipe._create_flat_graph()
     pipe._execgraph = pe.generate_expanded_graph(deepcopy(pipe._flatgraph))
@@ -87,7 +87,7 @@ def test_3mods(iterables, expected, connect):
     mod2 = pe.Node(interface=EngineTestInterface(), name="mod2")
     mod3 = pe.Node(interface=EngineTestInterface(), name="mod3")
     for nr in ["1", "2", "3"]:
-        setattr(eval("mod" + nr), "iterables", iterables[nr])
+        eval("mod" + nr).iterables = iterables[nr]
     if connect == ("1-2", "2-3"):
         pipe.connect(
             [

Diff for: nipype/utils/config.py

@@ -353,7 +353,7 @@ def _mock():
 
             # Older versions of xvfbwrapper used vdisplay_num
             if not hasattr(self._display, "new_display"):
-                setattr(self._display, "new_display", self._display.vdisplay_num)
+                self._display.new_display = self._display.vdisplay_num
             return self.get_display()
 
     def stop_display(self):