search

SpiNNakerManchester / PACMAN

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

merge graph into ApplicationGraph #431

Closed Christian-B closed 2 years ago

Christian-B commented 2 years ago

Now that there is no MachineGraph no need for a common Graph object.

Decided to keep the class name ApplicationGraph to show it can only take Application Objects

Note: Also removed ConstrainedObject as a super class as we can not even remember why or how a Graph could be Constrained.

dkfellows commented 2 years ago

Also removed ConstrainedObject as a super class as we can not even remember why or how a Graph could be Constrained.

👍 👍 👍

Christian-B commented 2 years ago

Better solution found