SpiNNakerManchester / PACMAN

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

Some Splitter init take a name others do not #459

Closed Christian-B closed 2 years ago

Christian-B commented 2 years ago

For example SplitterOneToOneLegacy can not be While SplitterFixedLegacy

Also the way Splitters name missing name is constant.

The best is: if splitter_name is None: splitter_name = type(self).name

Many other have a splitter name which is nearly always the Class name as Constant.

My suggestion is to move the name based on Type into the base Splitter and remove the if None check elsewhere