openbraininstitute / libsonata / 26636586305

93%

master: 93%

Build Type

Pull #49

github

Committed by cattabiani

Commit Message

Move electrodes_file from run section to per-report LFP blocks

- Add electrodesFile field to SimulationConfig::Report struct
- Remove electrodesFile field from SimulationConfig::Run struct
- parseRun() now rejects electrodes_file with a helpful migration error
- parseReports() parses electrodes_file per LFP report block
- Make variable_name optional for LFP reports (ignored by LFP computation)
- Reject electrodes_file on non-LFP report types
- Update Python bindings (move electrodes_file from Run to Report)
- Update test data and add C++/Python tests for new behavior
- Add property tests for non-LFP validation

Pull Request Pull Request #49: Move electrodes_file to per-report LFP blocks

Coverage Stats

18 of 19 new or added lines in 1 file covered. (94.74%)

13 existing lines in 1 file now uncovered.

2525 of 2711 relevant lines covered (93.14%)

53.15 hits per line