Skip to content

Commit

Permalink
fix add_monitor_core to all_cores
Browse files Browse the repository at this point in the history
  • Loading branch information
Christian-B committed Apr 17, 2024
1 parent b973ab2 commit a07c233
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions pacman/data/pacman_data_writer.py
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,7 @@ def add_edge(cls, edge: ApplicationEdge,
cls.__pacman_data._graph.add_edge(edge, outgoing_edge_partition_name)

def add_sample_monitor_vertex(
self, vertex: MachineVertex, all_cores: bool):
self, vertex: MachineVertex, all_chips: bool):
"""
Accepts a simple of the monitor cores to be added.
Expand All @@ -192,13 +192,13 @@ def add_sample_monitor_vertex(
:param ~pacman.model.graphs.machine.MachineVertex vertex:
One of the vertices added to each core, assumed to be typical of
all.
:param bool all_cores:
If True assumes that this Vertex will be placed on all cores
:param bool all_chips:
If True assumes that this Vertex will be placed on all chips
including Ethernet ones.
If False assumes that this Vertex type will only be placed on
Ethernet Vertices
"""
self.add_monitor_core(all_cores)
self.add_monitor_core(all_chips)
self.__pacman_data._ethernet_monitor_vertices.append(vertex)
if all_cores:
if all_chips:
self.__pacman_data._all_monitor_vertices.append(vertex)

0 comments on commit a07c233

Please sign in to comment.