genn-team / ml_genn

A library for deep learning with Spiking Neural Networks (SNN).
https://ml-genn.readthedocs.io
GNU Lesser General Public License v2.1
24 stars 7 forks source link

Several bug fixes and additional tests #96

Closed neworderofjamie closed 4 months ago

neworderofjamie commented 5 months ago

Clearly I was in a bit of a rush releasing the new version of mlGeNN corresponding with GeNN 5 before Capocaccia because Balazs found several stupid bugs in spike recording. This PR fixes those and a slightly subtler one that broke regularisation with EventProp due to lambda values being cast to integer. I have also added a test of spike recording functionality.

codecov-commenter commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 65.92%. Comparing base (26badad) to head (fa80dc3).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #96 +/- ## =========================================== + Coverage 0.00% 65.92% +65.92% =========================================== Files 100 100 Lines 4026 4024 -2 =========================================== + Hits 0 2653 +2653 + Misses 4026 1371 -2655 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.