project-rig / rig

A collection of tools for developing SpiNNaker applications.
GNU General Public License v2.0
4 stars 0 forks source link

Fix multiply-overlapping SameChipConstraint #223

Closed mossblaser closed 8 years ago

mossblaser commented 8 years ago

As spotted by @neworderofjamie, if a pair of SameChipConstraints overlap by more than one vertex, the apply_same_chip_constraints utility function would crash. This commit fixes this issue and updates the tests to explicitly test this behaviour.

mossblaser commented 8 years ago

@neworderofjamie approves; awaiting travis...

mossblaser commented 8 years ago

The coverage issue here is an ongoing issue with coveralls, coverage has not actually dropped and this issue will be fixed separately.