lava-nc / lava-dl

Deep Learning library for Lava
https://lava-nc.org
BSD 3-Clause "New" or "Revised" License
149 stars 71 forks source link

fixing sign of surrogate gradient for graded spikes #286

Closed ccaccavella closed 6 months ago

ccaccavella commented 6 months ago

Issue Number: There is no issue open. The change has been discussed with Sumit Shrestha.

Objective of pull request: The changes has been made because the implementation of the surrogate gradient in the case of a graded spike activity was wrong. Specifically, the the sign inside the exponential function was reversed.

Pull request checklist

Your PR fulfills the following requirements:

Pull request type

Please check your PR type:

What is the current behavior?

-

What is the new behavior?

-

Does this introduce a breaking change?

Supplemental information