• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

openbraininstitute / neurodamus / 13816229923 / 2 – unit
90%
main: 90%

Build:
DEFAULT BRANCH: main
Ran 12 Mar 2025 04:15PM UTC
Files 48
Run time 4s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

12 Mar 2025 04:13PM UTC coverage: 65.856% (+0.4%) from 65.501%
unit – 13816229923.2

push

github

web-flow
synapse bluecellulab checks (#144)

## Context

After talking with bluecellulab we decided to be inspired by some of
their tests (copy). This PR adds some checks to the synapses and the
ways you can override values with the `simulation_config.json` that I
was not aware of.

## Fix #142 

## Scope

- [x] `nrrp` was already checked. Minor improvements: it should be a
positive integer
- [x] `"%s.NMDA_ratio = 1.22 %s.tau_r_NMDA = 3.9 %s.tau_d_NMDA =
148.5"`.
    - [x] `NMDA_ratio`
- [x] `tau_r_NMDA` and `tau_d_NMDA` are global variables. Added test to
override these
- [x] add test to verify synapse location
- [x] test in particular modoverride for `GluSynapse`
- [x] move/merge `test_connection.py` into unit tests
- [x] check synapse override from
[conditions](https://sonata-extension.readthedocs.io/en/latest/sonata_simulation.html#parameters-required-for-modifications)
and how it interacts with the one from
`connection_overrides.synapse_configure`. Spoiler:
``connection_overrides.synapse_configure` wins

## Additional improvements, not stemming from bluecellulab

- [x] `np.isclose(10001.1, 10001.2) == True`. We should use smaller
values. default rtol is 1e-5
- [x] modoverride test was not really overriding (it was already ampa).
Switch to gaba in the test
- [x] fix pytest assert reporting if `tests.utils` is imported in
`conftest.py`
- [x] create issue for inconsistend behavior for synapse global variable
override #147
- [x] fix
[docs](https://github.com/openbraininstitute/sonata-extension/pull/2)

5933 of 9009 relevant lines covered (65.86%)

0.66 hits per line

Source Files on job unit - 13816229923.2
  • Tree
  • List 48
  • Changed 5
  • Source Changed 0
  • Coverage Changed 5
Coverage ∆ File Lines Relevant Covered Missed Hits/Line
  • Back to Build 13816229923
  • 08278fd6 on github
  • Prev Job for on main (#13813033482.2)
  • Next Job for on main (#13967358348.2)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc