Skip to content

Add eprop work to main tools#1368

Draft
andrewgait wants to merge 144 commits into
masterfrom
eprop_adaptive_update
Draft

Add eprop work to main tools#1368
andrewgait wants to merge 144 commits into
masterfrom
eprop_adaptive_update

Conversation

@andrewgait
Copy link
Copy Markdown
Contributor

@andrewgait andrewgait commented Jul 6, 2023

Incorporate eprop models into the main tools.

This implementation effectively sits alongside the current "standard" neuron models and does not impinge too much on those; the current sticking points (which could be worked on) are:

  • It may be necessary to further silo the eprop implementation away from the standard models (for example, creating a spike_processing_eprop.c file and/or a synapses_eprop.c file)
  • In a similar vein, the standard Spike Source Poisson C code takes advantage of the multicast packet with payload to deal with cases where multiple spikes may be sent in the same timestep by sending them as one packet with a payload to indicate the number of spikes. This needed to be removed in the current eprop implementation because the communication of the learning signal into the eprop model population requires the use of the multicast packet with payload functionality. We could plausibly have two separate spike source poisson executables to denote these two scenarios.

oliverrhodes and others added 30 commits September 19, 2019 12:34
…ng for poisson sources to be update-able from this code, all poisson stuff has been removed, neuron index 1 and 2 has V stored as diff variables, mean and entropy of values comes from average V over the recall period
…s are corrected, init corrected to include store_recall and eprop properly, test script adjusted to creat edge for poisson updating, still not tested as waiting for erbp plasticity components
…e, created separate synapses.c for sinusoid to process neuron array separately, altered print outs, couple bug fixes
@coveralls
Copy link
Copy Markdown

coveralls commented Aug 8, 2023

Coverage Status

coverage: 60.979% (-0.3%) from 61.316% when pulling fa3478a on eprop_adaptive_update into 0f8e5b7 on master.

@Bekkozhan
Copy link
Copy Markdown

Hi all!

I'm working with Andrew Davison at the Institute of Neuroscience Paris-Saclay on extending PyNN to support spike-based learning algorithms such as e-prop and EventProp across neuromorphic backends. I've been reviewing this PR as part of a SpiNNaker feasibility assessment. The implementation looks complete, with two known integration sticking points noted in the description. Is there still active interest in resolving those and getting this merged into the main tools?

Bek

@andrewgait
Copy link
Copy Markdown
Contributor Author

Hello @Bekkozhan - this was worked on a number of years ago, and I am not sure how easy it will be to update and merge into the main tools - I don't work on this project any more (though I keep an eye out for things that happen). I'm tagging @rowleya and @Christian-B who are still working on it to see if they have any opinions about whether this branch/PR is worth reviving.

@Bekkozhan
Copy link
Copy Markdown

Thank you @andrewgait - will wait for @rowleya and @Christian-B to comment on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants