search

arvoelke / nengolib

Nengo library of additional extensions
Other
29 stars 6 forks source link

[FIX] Bug in general synapse mapping #94

Closed arvoelke closed 7 years ago

arvoelke commented 7 years ago

This was a pretty serious bug, since it meant the mapping wasn't being exploited to full effect. Fortunately this could only be seen when len(sys) < len(synapse), since otherwise accessing elements out of bounds in a numpy polynomial returns 0 by default (which was good).

codecov-io commented 7 years ago

Codecov Report

Merging #94 into master will not change coverage. The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #94   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          25     25           
  Lines        1065   1065           
  Branches      125    125           
=====================================
  Hits         1065   1065
Impacted Files Coverage Δ
nengolib/synapses/mapping.py 100% <100%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 3a7482e...7e204e0. Read the comment docs.