STY: Apply ruff/flake8-comprehensions rule C409

DimitriPapadopoulos · DimitriPapadopoulos · commit f9ac9d9e0d2f · 2024-10-06T17:49:06.000+02:00
C409 Unnecessary `list` literal passed to `tuple()` (rewrite as a `tuple` literal)
diff --git a/nipype/interfaces/cmtk/cmtk.py b/nipype/interfaces/cmtk/cmtk.py
@@ -248,7 +248,7 @@ def cmat(
                     axis=1,
                 )
             )
-            G.nodes[int(u)]["dn_position"] = tuple([xyz[0], xyz[2], -xyz[1]])
+            G.nodes[int(u)]["dn_position"] = (xyz[0], xyz[2], -xyz[1])
 
     if intersections:
         iflogger.info("Filtering tractography from intersections")
@@ -1070,7 +1070,7 @@ def create_nodes(roi_file, resolution_network_file, out_filename):
                 np.where(np.flipud(roiData) == int(d["dn_correspondence_id"])), axis=1
             )
         )
-        G.nodes[int(u)]["dn_position"] = tuple([xyz[0], xyz[2], -xyz[1]])
+        G.nodes[int(u)]["dn_position"] = (xyz[0], xyz[2], -xyz[1])
     with open(out_filename, 'wb') as f:
         pickle.dump(G, f, pickle.HIGHEST_PROTOCOL)
     return out_filename

Original file line number	Diff line number	Diff line change
`@@ -248,7 +248,7 @@ def cmat(`
`248`	`248`	`axis=1,`
`249`	`249`	`)`
`250`	`250`	`)`
`251`		`- G.nodes[int(u)]["dn_position"] = tuple([xyz[0], xyz[2], -xyz[1]])`
	`251`	`+ G.nodes[int(u)]["dn_position"] = (xyz[0], xyz[2], -xyz[1])`
`252`	`252`
`253`	`253`	`if intersections:`
`254`	`254`	`iflogger.info("Filtering tractography from intersections")`
`@@ -1070,7 +1070,7 @@ def create_nodes(roi_file, resolution_network_file, out_filename):`
`1070`	`1070`	`np.where(np.flipud(roiData) == int(d["dn_correspondence_id"])), axis=1`
`1071`	`1071`	`)`
`1072`	`1072`	`)`
`1073`		`- G.nodes[int(u)]["dn_position"] = tuple([xyz[0], xyz[2], -xyz[1]])`
	`1073`	`+ G.nodes[int(u)]["dn_position"] = (xyz[0], xyz[2], -xyz[1])`
`1074`	`1074`	`with open(out_filename, 'wb') as f:`
`1075`	`1075`	`pickle.dump(G, f, pickle.HIGHEST_PROTOCOL)`
`1076`	`1076`	`return out_filename`