-
Notifications
You must be signed in to change notification settings - Fork 47
Test binaries #1637
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
Test binaries #1637
Changes from all commits
8469261
4d03247
52d97a7
8b56cbf
20d14d9
417c742
d8f201f
41fd17c
d8bf04c
e461602
80b9365
4d8c565
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -63,6 +63,12 @@ n_colour_bits = 4 | |
| error_on_non_spynnaker_pynn = True | ||
| @error_on_non_spynnaker_pynn = Whether to error or just warn on non-spynnaker-compatible PyNN | ||
|
|
||
| n_synapse_cores = None | ||
| @n_synapse_cores = Testing Option! | ||
| A better way is to call set_model_n_synapse_cores on the model. | ||
| Mapping of a AbstractPyNNNeuronModel to n_synapse_cores to be set | ||
| In the format model:n_synapse_cores,model:n_synapse_cores | ||
|
|
||
|
Comment on lines
+66
to
+71
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. See other comments |
||
| [Recording] | ||
| @ = Section for the sending of live spikes. | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -120,6 +120,7 @@ def live_neuron_voltage() -> None: | |||||
| translator_2 = Translator(devices_2) | ||||||
| model_2 = p.external_devices.ExternalDeviceLifControl( | ||||||
| devices_2, create_edges, translator_2) | ||||||
| model_2.set_model_n_synapse_cores(1) | ||||||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
Might be better to advise against calling a class method on an instance?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. |
||||||
| conn = p.external_devices.SpynnakerLiveSpikesConnection( | ||||||
| receive_labels=["stim"], local_port=None) | ||||||
| conn.add_receive_callback("stim", spike_receiver) | ||||||
|
|
@@ -160,6 +161,8 @@ class TestLiveNeuronVoltage(BaseTestCase): | |||||
|
|
||||||
| def test_live_neuron_voltage(self) -> None: | ||||||
| self.runsafe(live_neuron_voltage) | ||||||
| self.check_binaries_used(["external_device_lif_control.aplx", | ||||||
| "external_device_lif_control_neuron.aplx"]) | ||||||
|
|
||||||
|
|
||||||
| if __name__ == '__main__': | ||||||
|
|
||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason this can't just be done in the test script itself?