-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pushbot Protocol mixup #1520
Comments
PushBotEthernetRetinaDevice( init was declared to take a MunichIoEthernetProtocol https://github.com/SpiNNakerManchester/PyNNExamples/blob/master/examples/external_devices_examples/pushbot_ethernet_example.py |
So in general, the Pushbot is represented by an ExternalDeviceLifControl. This is a Population of neurons which, instead of spiking, send multicast messages periodically based on their membrane voltage. The "Devices" of that Population control what each of the neurons does - that is, the multicast key they send, whether they send the voltage directly, minimum and maximum values and how often they send the message (could be every timestep, but could be less often to not overload) If the Pushbot is connected to the SpiNNaker link, the devices are each themselves virtual vertices, and the ExternalDeviceLifControl then has edges from it to each of them. If the Pushbot is connected via Ethernet, the ExternalDeviceLifControl connects to a LivePacketGather instead. But note that the same commands are sent. On the host, these are received by a Translator which decodes them and sends messages to the PushBot using a different protocol, using the WiFi connection directly. Basically the laptop becomes the equivalent of the Munich IO board - it translates the multicast packets into Ethernet packets. |
It looks like the code and examples are correct even if a little confusing. The few cases where the type in the doc differs from what the super class, code or example uses will be fixed in the typing pr. |
A number of the PushBot Ethernet classes are using MunichIoSpiNNakerLinkProtocol and not MunichIoEthernetProtocol
a simple attmept to just change the class failed.
See #1519
And especially the mypy report.
This needs a proper review ideally by someone with a Pushbot
The text was updated successfully, but these errors were encountered: