Skip to content

Draft: [MCCS 1286] Refactor aavs_tile_simulator

Alistair Child requested to merge mccs-1286 into main

Deleted Classes:

  • DynamicTpmSimulator
  • BaseTpmSimulator
  • StaticTpmSimulator
  • StaticTileSimulator
  • _TpmSimulatorComponentManager
  • DynamicTpmSimulatorComponentManager
  • StaticTpmSimulatorPatchedReadWrite

Added Classes:

  • AavsTileSimulator
  • AavsDynamicTileSimulator

This merge will add some technical debt:

  • The new aavs_tile_simulator is still just skeleton code and needs filling out.
  • The new aavs_tile_simulator has not been type hinted.
  • Some tests have been marked as xfail.
  • Some tests/fixtures may need revisiting.

This merge will add some significant changes to the architecture of the tile simulators:

  • The simulators are driven in the same way as the pyaavs Tile (i.e. driven by the TPMDriver).
  • Existing simulators have been removed(moved/copied) meaning that the UML diagram has changed. This is shown by 2 UML diagrams Gianni created showing the New proposition.
  • The "Tile" is injected from the component manager into the TPMDriver. This "Tile" can be either a AavsDynamicTileSimulator, AavsTileSimulator or the Tile(the real hardware tile).

Previous UML: Old UML Proposed UML: New Proposed UML

Edited by Alistair Child

Merge request reports