Luc Grosheintz
Luc Grosheintz
@ohm314 do you want to resolve the conflicts and then review/merge; or do we close this?
Fails as follows (sometimes): ``` x86_64/corenrn/mod2c/variable_names.cpp:155:20: error: redeclaration of ‘double* coreneuron::variable_names_Instance::pi’ 155 | double* pi{}; | ^ ``` Likely a better error message would be good.
Further comments on: https://github.com/BlueBrain/nmodl-references/pull/20
One should probably check if multiple tables are permitted and work. The reason is that we do use both `sig` and `sigmoid` to mangle names. Therefore, the question is can...
@pramodk Do we need the property that the visitor can be reapplied, even though its output is invalid input?
This might be helpful: https://github.com/neuronsimulator/nrn/pull/2768 otherwise you'll not have a way to get the Node from a Prop.
Yes, because NEURON + nocmodl allow it. The issue is with the string we generate that we then send to SymPy. We should add something to internally rename `is` to...
@pramodk this looks useful. It contains unrelated code which is now outdated. Should this be revived and merged? Alternatively we can close it as too old.
It certainly doesn't help with https://github.com/BlueBrain/nmodl/issues/1217 but if it's useful to you, we can merge it.
This `try ... except` block has caused me to take longer to notice that the installation of NMODL was broken. We should have helper in coding-conventions to make sanitizers and...