Fix ruff/flake8-bugbear (B020) issues

DimitriPapadopoulos · Mar 4, 2024 · 9a695e2 · 9a695e2
1 parent 44b091e
commit 9a695e2
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/capsul/execution_context.py b/capsul/execution_context.py
@@ -772,15 +772,15 @@ def find_temporary_to_generate(executable):
         field.generate_temporary = False
         if isinstance(node, ProcessIteration):
             node.process.field(field.name).generate_temporary = field.generate_temporary
-        for node, parameter in executable.get_linked_items(
+        for snode, parameter in executable.get_linked_items(
             node, field.name, direction="links_from", in_outer_pipelines=True
         ):
             if isinstance(node, ProcessIteration):
-                stack.append((node.process, node.process.field(parameter)))
-                # print('!temporaries!   + ', node.process.full_name, ':', parameter)
+                stack.append((snode.process, snode.process.field(parameter)))
+                # print('!temporaries!   + ', snode.process.full_name, ':', parameter)
             else:
-                stack.append((node, node.field(parameter)))
-                # print('!temporaries!   + ', node.full_name, ':', parameter)
+                stack.append((snode, snode.field(parameter)))
+                # print('!temporaries!   + ', snode.full_name, ':', parameter)
 
     # print('!temporaries!  parameters with temporary')
     # for n, p in temporaries:

diff --git a/capsul/pipeline/pipeline.py b/capsul/pipeline/pipeline.py
@@ -2460,10 +2460,10 @@ def __setitem__(self, path, value):
     def dispatch_value(self, node, name, value):
         """Propagate the value from a pipeline plug through links"""
         # print(f"!dispatch! {node.name}.{name} = {value}")
-        for node, plug in self.dispatch_plugs(node, name):
-            # print(f"!dispatch! -> {node.name}.{plug}")
-            if getattr(node, plug, undefined) != value:
-                setattr(node, plug, value)
+        for snode, plug in self.dispatch_plugs(node, name):
+            # print(f"!dispatch! -> {snode.name}.{plug}")
+            if getattr(snode, plug, undefined) != value:
+                setattr(snode, plug, value)
 
     def dispatch_plugs(self, node, name):
         """generator through linked plugs"""

diff --git a/capsul/pipeline/python_export.py b/capsul/pipeline/python_export.py
@@ -139,7 +139,7 @@ def _write_process(process, pyf, name, enabled, skip_invalid):
 
             if isinstance(snode, Pipeline):
                 sself_str = '%s.nodes["%s"]' % (self_str, "%s")
-                for node_name, snode in snode.nodes.items():
+                for node_name, _ in snode.nodes.items():
                     scnode = cnode.nodes[node_name]
 
                     if node_name == "":