use list to guarantee order

SpiNNakerManchester · Sep 9, 2024 · c8e2812 · c8e2812
1 parent 63201d1
commit c8e2812
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/spinn_front_end_common/interface/buffer_management/buffer_manager.py b/spinn_front_end_common/interface/buffer_management/buffer_manager.py
@@ -359,12 +359,9 @@ def extract_data(self) -> None:
         """
         with BufferDatabase() as db:
             db.start_new_extraction()
-        recording_placements = set(
+        recording_placements = list(
             FecDataView.iterate_placements_by_vertex_type(
-                AbstractReceiveBuffersToHost))
-        recording_placements.update(
-            FecDataView.iterate_placements_by_vertex_type(
-                AbstractReceiveRegionsToHost))
+                (AbstractReceiveBuffersToHost, AbstractReceiveRegionsToHost)))
         if self._java_caller is not None:
             logger.info("Starting buffer extraction using Java")
             self._java_caller.set_placements(recording_placements)
@@ -375,7 +372,7 @@ def extract_data(self) -> None:
             self.__python_extract_no_monitors(recording_placements)
 
     def __python_extract_with_monitors(
-            self, recording_placements: Set[Placement]):
+            self, recording_placements: List[Placement]):
         """
         :param list(~pacman.model.placements.Placement) recording_placements:
             Where to get the data from.
@@ -394,7 +391,7 @@ def __python_extract_with_monitors(
             self.__python_extract_no_monitors(recording_placements)
 
     def __python_extract_no_monitors(
-            self, recording_placements: Set[Placement]):
+            self, recording_placements: List[Placement]):
         """
         :param list(~pacman.model.placements.Placement) recording_placements:
             Where to get the data from.