BlueBrain / nmodl

Code Generation Framework For NEURON MODeling Language
https://bluebrain.github.io/nmodl/
Apache License 2.0
55 stars 15 forks source link

Wrong order of ion variables declaration in mod file #436

Closed iomaganaris closed 3 years ago

iomaganaris commented 3 years ago

When compiling cal_mig.mod from https://github.com/Neurosim-lab/netpyne/blob/coreneuron/examples/coreneuron/m1/mod/cal_mig.mod the order of the cai and ica ion channels' variables is wrong in the generated .cpp file by NMODL. The issue is fixed if cai and ica definition as RANGE variables in line 44 is removed. We need to check again the order of the printing of the ion variables in NMODL to make sure it's the same as neuron nmodl.

cattabiani commented 3 years ago

I can take a look. How much priority this has?

pramodk commented 3 years ago

@cattabiani : me and Ioannis were debugging this issue yesterday. I am already looking at this (will create PR after double/float as a string PR is done).

cattabiani commented 3 years ago

I did not see the last comment