Luc Grosheintz
Luc Grosheintz
To check if recent improvements are sufficient (now all merged), I used the this version of Model DB: https://github.com/nrnhines/266806/commit/c57abdba4833919dad878e556e667d5811214ec5 When compiled with: ``` cmake -DCMAKE_INSTALL_PREFIX=build/install -DNRN_ENABLE_TESTS=ON -DNRN_ENABLE_PYTHON=ON -DNRN_ENABLE_DOCS=OFF -DNRN_ENABLE_RX3D=OFF -DNRN_ENABLE_CORENEURON=ON...
I don't know how we can easily add "false" instances of a mechanism to achieve the required padding. I'll let you know if I think of something. There's three reasons...
We tried NEURON+NMODL (and with a small tweak we get): ``` ncores: [ 2 , 4 , 8 , 16 ] time [s]: [ 34.2, 20.0, 13.8, 23.4 ] [...
@nrnhines I believe we've discussed this issue in a different context. Would something like this make sense?
It's a slightly more involved process, since some code that should be simplified is still used with `Layout::AoS`.
Second attempt here: #2895.
Thank you! If nobody else needs it right away, I'd propose I first implement the code in NMODL that needs this. Then when that's ready for merging we also merge...
@nrnhines I'll merge this now, because we use it in #3066 to fix issue #3059. In NMODL I use it to solve similar issues: https://github.com/BlueBrain/nmodl/pull/1414
No, probably the order will be something like: * Finish https://github.com/neuronsimulator/nrn/pull/2895 * https://github.com/neuronsimulator/nrn/issues/2894 and then this.
The following is a shorter reproducer, found by checking the backtrace of the reproducer provided above: ``` import numpy as np from neuron import h, gui from neuron.units import ms...