Skip to content

Commit

Permalink
typing fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
Christian-B committed Dec 19, 2023
1 parent f502615 commit 7f388dd
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -163,5 +163,5 @@ def machine_vertices_for_recording(
return []

@overrides(AbstractSplitterCommon.reset_called)
def reset_called(self) -> bool:
def reset_called(self) -> None:
pass
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
# limitations under the License.

import logging
from typing import List, Optional
from typing import cast, List, Optional
from spinn_utilities.overrides import overrides
from spinn_utilities.log import FormatAdapter
from pacman.exceptions import PacmanConfigurationException
Expand Down Expand Up @@ -48,9 +48,10 @@ def set_governed_app_vertex(self, app_vertex: ApplicationVertex):
f"{self} is not a LegacyPartitionerAPI")
super().set_governed_app_vertex(app_vertex)
self._vertex_slice = Slice(0, self.governed_app_vertex.n_atoms - 1)
self._sdram = self.governed_app_vertex.get_sdram_used_by_atoms(
lp = cast(LegacyPartitionerAPI, app_vertex)
self._sdram = lp.get_sdram_used_by_atoms(
self._vertex_slice)
self._machine_vertex = self.governed_app_vertex.create_machine_vertex(
self._machine_vertex = lp.create_machine_vertex(
vertex_slice=self._vertex_slice,
sdram=self._sdram, label=None)
self.governed_app_vertex.remember_machine_vertex(self._machine_vertex)
Expand Down

0 comments on commit 7f388dd

Please sign in to comment.