Bugfix for failing to deepcopy named columns.

kyleaoman · kyleaoman · commit 29aae3e37b97 · 2025-04-09T23:58:27.000+01:00
diff --git a/swiftgalaxy/iterator.py b/swiftgalaxy/iterator.py
@@ -239,7 +239,6 @@ def __init__(
             )
         else:
             num_targets = len(self.halo_catalogue._region_centre)
-            print(num_targets)
         # before evaluating optimized solutions:
         if num_targets == 0:
             # if we have 0 targets short-circuit
diff --git a/swiftgalaxy/reader.py b/swiftgalaxy/reader.py
@@ -1910,7 +1910,9 @@ def _data_copy(
                         )
                         if data is not None:
                             setattr(
-                                new_named_columns_helper, f"_{named_column}", data[mask]
+                                new_named_columns_helper._named_column_dataset,
+                                f"_{named_column}",
+                                data[mask],
                             )
                 else:
                     data = getattr(

Original file line number	Diff line number	Diff line change
`@@ -239,7 +239,6 @@ def __init__(`
`239`	`239`	`)`
`240`	`240`	`else:`
`241`	`241`	`num_targets = len(self.halo_catalogue._region_centre)`
`242`		`- print(num_targets)`
`243`	`242`	`# before evaluating optimized solutions:`
`244`	`243`	`if num_targets == 0:`
`245`	`244`	`# if we have 0 targets short-circuit`
Original file line number	Diff line number	Diff line change
`@@ -1910,7 +1910,9 @@ def _data_copy(`
`1910`	`1910`	`)`
`1911`	`1911`	`if data is not None:`
`1912`	`1912`	`setattr(`
`1913`		`- new_named_columns_helper, f"_{named_column}", data[mask]`
	`1913`	`+ new_named_columns_helper._named_column_dataset,`
	`1914`	`+ f"_{named_column}",`
	`1915`	`+ data[mask],`
`1914`	`1916`	`)`
`1915`	`1917`	`else:`
`1916`	`1918`	`data = getattr(`