Use directly a set comprehension...

... instead of a list comprehension to initialise a set.
populse · Nov 19, 2023 · 55f1149 · 55f1149
1 parent 4f62bd3
commit 55f1149
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 19 deletions.
diff --git a/capsul/pipeline/test/test_custom_nodes.py b/capsul/pipeline/test/test_custom_nodes.py
@@ -275,12 +275,10 @@ def pipeline_definition(self):
         self.add_iterative_process(
             "test_it",
             "capsul.pipeline.test.test_custom_nodes.CVtest",
-            non_iterative_plugs=set(
-                [
-                    "model",
-                    "base",
-                ]
-            ),
+            non_iterative_plugs={
+                "model",
+                "base",
+            },
             make_optional=["out1"],
         )
         self.add_custom_node(

diff --git a/capsul/pipeline/test/test_iterative_process.py b/capsul/pipeline/test/test_iterative_process.py
@@ -279,7 +279,7 @@ def test_iterative_big_pipeline_workflow(self):
             subject = workflow.parameters_values[proxy[1]]
             subjects.add(subject)
             self.assertIn(subject, ["toto", "tutu", "tata", "titi", "tete"])
-        self.assertEqual(subjects, set(["toto", "tutu", "tata", "titi", "tete"]))
+        self.assertEqual(subjects, {"toto", "tutu", "tata", "titi", "tete"})
 
     def test_iterative_pipeline_workflow_run(self):
         self.small_pipeline.output_image = [

diff --git a/capsul/qt_gui/widgets/settings_editor.py b/capsul/qt_gui/widgets/settings_editor.py
@@ -47,16 +47,14 @@ def update_gui(self):
         self.tab_wid.clear()
         self.module_tabs = {}
         resource = self.resource_combo.currentText()
-        non_modules = set(
-            [
-                "dataset",
-                "config_modules",
-                "python_modules",
-                "database",
-                "persistent",
-                "start_workers",
-            ]
-        )
+        non_modules = {
+            "dataset",
+            "config_modules",
+            "python_modules",
+            "database",
+            "persistent",
+            "start_workers",
+        }
         mod_map = [
             f.name
             for f in getattr(self.config, resource).fields()

diff --git a/capsul/sphinxext/capsul_pipeline_rst.py b/capsul/sphinxext/capsul_pipeline_rst.py
@@ -124,7 +124,7 @@
 ):
     # From the modules full path 'm1.m2.pipeline/process' get the module
     # name 'm2'
-    module_names = set([x.split(".")[1] for x in modules])
+    module_names = {x.split(".")[1] for x in modules}
 
     # Sort each item according to its module name.
     # The result is a dict of the form 'd[m2] = [pipeline/process1, ...]'.

diff --git a/capsul/sphinxext/capsul_usecases_rst.py b/capsul/sphinxext/capsul_usecases_rst.py
@@ -83,7 +83,7 @@
 
 # Sort all the pilots
 # > from the pilots full path 'm1.m2.pipeline' get the module name 'm2'
-module_names = set([x.split(".")[1] for x in pilots])
+module_names = {x.split(".")[1] for x in pilots}
 # > sort each pilot according to its module name.
 # > the result is a dict of the form 'd[m2] = [pilot1, ...]'
 sorted_pilots = {}