-
Notifications
You must be signed in to change notification settings - Fork 7
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
Kernel learning #106
base: master
Are you sure you want to change the base?
Kernel learning #106
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #106 +/- ##
==========================================
- Coverage 64.80% 64.64% -0.16%
==========================================
Files 101 101
Lines 4259 4271 +12
==========================================
+ Hits 2760 2761 +1
- Misses 1499 1510 +11 ☔ View full report in Codecov by Sentry. |
1cb7244
to
8bd13dd
Compare
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.
That looks surprisingly simple. I assume you have run some sort of test.
97263fd
to
62aa216
Compare
Using the temporary functionality for modifying
KERNEL_G
variables implemented in genn-team/genn#631, this re-instates learning of convolutions with EventProp