lava icon indicating copy to clipboard operation
lava copied to clipboard

Add rf and rf_iz neurons to lava

Open Michaeljurado42 opened this issue 3 years ago • 3 comments

Objective of issue: Add rf and rf_iz neurons to lava

Lava version:

  • [ ] 0.5.0 (feature release)
  • [ ] 0.4.1 (bug fixes)
  • [x] 0.4.0 (current version)
  • [ ] 0.3.0
  • [ ] 0.1.2

I'm submitting a ...

  • [ ] bug report
  • [x] feature request
  • [ ] documentation request

Current behavior:

  • No support for rf neurons in lava

Expected behavior:

  • There should be a floating point and loihi bit accurate implementation of rf neurons in lava. This implementation should match the lava-dl rf neuron implementations.

Related code:

from lava.proc.rf_iz.models import RF
from lava.proc.rf_iz.models import RF_IZ
...
neuron = RF(shape=(n_neurons,), ..., vth = vth) 
neuron = RF_IZ(shape=(n_neurons,), ... , vth = vth) 

Other information:

This issue is a result of an ongoing lava-dl discussion in which I ask whether or not rf neurons will be added to lava.

Currently I am working on a fork that implements rf and rf_iz versions based on the implementation of their lava-dl counterparts. A proof of concept example can be found here.

Michaeljurado42 avatar Sep 21 '22 18:09 Michaeljurado42

@Michaeljurado24 thanks for taking this initiative. Let's start a PR (keep it as a draft). Then I can look at the PR and provide more feedback.

bamsumit avatar Sep 26 '22 17:09 bamsumit

No problem. The GT community and I are very excited about this capability so I am happy to contribute. I can make a PR tomorrow.

Michaeljurado42 avatar Sep 26 '22 21:09 Michaeljurado42

draft pull request here

Michaeljurado42 avatar Sep 27 '22 13:09 Michaeljurado42