Christian Y. Brenninkmeijer

Part of https://github.com/SpiNNakerManchester/SpiNNUtils/pull/193 _reset_has_read_neuron_parameters_this_run is an underscore method as not designed for users.

part of https://github.com/SpiNNakerManchester/PACMAN/pull/461

Part of https://github.com/SpiNNakerManchester/PACMAN/pull/455

related to https://github.com/SpiNNakerManchester/PACMAN/issues/457 But can be done separate as just changes a comment.

part of https://github.com/SpiNNakerManchester/SpiNNFrontEndCommon/pull/967 A few weird cases where handled by external_devices: As the main init imports the exernal_devices the cleanest fix was to push the simulator into external_device when setup....

Fixes https://github.com/SpiNNakerManchester/sPyNNaker/issues/1188 Instead of Projections rounding delays the SynapseDynamics does it. This is at a minor cost that SynapseDynamics these must be created after sim.setup The user available set_delays has...

Part of https://github.com/SpiNNakerManchester/SpiNNFrontEndCommon/pull/969 Note: the github actions cfg has been changed from an invalid one pointing at spinn-4.cs.man.ac.uk to a virtual machine There is a minor slowdown in github action...

I notice that SpikeInjectorVertex is a SimplePopulationSettable As is not/was not a AbstractChangableAfterRun Population used to block set calls after run based on AbstractChangableAfterRun not being implemented. The questions are...

SynapticMatrixApp __delay_index set but never used

1

Strip if out?

Remove Abstract from classes that are not Abstract

11

An Abstract class should have an Abstract method or Property An Abstract class should NEVER be created directly. (Except maybe in a test) For example: AbstractPopulationVertex SplitterAbstractPopulationVertexFixed