emer / axon

Axon adds axonal spiking to the Leabra framework for error-driven and associative learning
BSD 3-Clause "New" or "Revised" License
18 stars 8 forks source link

Update axon boa example to obelisk ALM, M1, VL connectivity #280

Closed rcoreilly closed 1 year ago

rcoreilly commented 1 year ago

This branch is named incorrectly but it is just a one-off to update this connectivity. should be merged after the previous PR on which it is based.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.05% :warning:

Comparison is base (c07c8ec) 30.14% compared to head (857de0a) 30.10%.

:exclamation: Current head 857de0a differs from pull request most recent head efefca1. Consider uploading reports for the commit efefca1 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #280 +/- ## ========================================== - Coverage 30.14% 30.10% -0.05% ========================================== Files 85 85 Lines 15345 15335 -10 ========================================== - Hits 4626 4616 -10 Misses 10455 10455 Partials 264 264 ``` | [Files Changed](https://app.codecov.io/gh/emer/axon/pull/280?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=emer) | Coverage Δ | | |---|---|---| | [examples/boa/boa.go](https://app.codecov.io/gh/emer/axon/pull/280?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=emer#diff-ZXhhbXBsZXMvYm9hL2JvYS5nbw==) | `82.97% <100.00%> (-0.19%)` | :arrow_down: |

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

rcoreilly commented 1 year ago

actually this ended up just being cleanup of commented-out connections -- additional notMaint -> VL produces later instability. investigate this later. will merge manually to get out of the way.