SpiNNakerManchester / PACMAN

Partition and Configuration Manager for SpiNNaker
Apache License 2.0
9 stars 7 forks source link

ConnectiveBasedPlacer fails with SameChipConstraint #406

Closed Christian-B closed 2 years ago

Christian-B commented 3 years ago

I added ConnectiveBasedPlacer to https://github.com/SpiNNakerManchester/PACMAN/blob/master/unittests/operations_tests/placer_algorithms_tests/test_same_chip_constraint.py

this throws a PacmanAlreadyPlacedError

Options are.

  1. Nuke ConnectiveBasedPlacer
  2. Have ConnectiveBasedPlacer raise an unsupported constraint exception
  3. Fix ConnectiveBasedPlacer
Christian-B commented 3 years ago

https://github.com/SpiNNakerManchester/PACMAN/blob/master/unittests/operations_tests/placer_algorithms_tests/test_sdram_edge_placement.py also fails with the ConnectiveBasedPlacer