Make BlueCellulab NEURON 9.0+ compatible#67
Conversation
- void** --> IvocVect** in DetAMPANMDA,
DetGABAAB, ProbGABAAB_EMS, GluSynapse, and VecStim MOD files
(both tests/mechanisms/ and examples/mechanisms/)
- RNG-related void** patterns kept as-is (explicit casts, valid C++)
- Replace deprecated neuron.run() with neuron.h.continuerun() in
test_ballstick.py
Update test tolerances for NEURON 9.0 numerical differences
- Eigen library replacing Meschach introduces small FP differences
(tightened to rel=5e-4 to abs=5e-3 for affected assertions)
- RNG implementation changes produce different random sequences
(relaxed tolerances for RNG-dependent test assertions)
Test reference files:
- Update 10 h5 reference files with NEURON 9.0 Eigen-based outputs
Codecov Report✅ All modified and coverable lines are covered by tests.
🚀 New features to boost your workflow:
|
AurelienJaquier
left a comment
There was a problem hiding this comment.
Very nice! Have you tried to run the examples with NEURON 9 ?
|
I just ran the examples manually to see the figures generated in each notebook. They look fine at first glance. The tests would have also run the examples. |
WeinaJi
left a comment
There was a problem hiding this comment.
Are these mod files in example/mechanisms copied from neurodamus-models repo?
If yes, I am wondering if they can be copied on the fly during testing. That would be make them consistent
Good idea. The README In the |
Yes, that plan sounds good. Probably you just do I don't have other comments for this PR. I will let @jamesgking comment. |
jamesgking
left a comment
There was a problem hiding this comment.
Changes look straight forward to me. Yes, I agree with idea to have a future PR use directly the models from other repo to avoid duplication and out of sync files
|
Thank you, @WeinaJi and @jamesgking ! I created the new issue for mechanisms. |
Main Changes
Based on NEURON 9.0 release notes: https://github.com/neuronsimulator/nrn/releases/tag/9.0.0, the following main changes were made:
Update test tolerances for NEURON 9.0 numerical differences
NEURON 9.0 release notes mention:
Test reference files: