-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Ensure attributes & elements can have sinular/plural equivalents generated #46
Comments
Added ion_channel to the NameTable to ensure correct naming of attribute of ChannelDensity: ChannelDensity(cond_density='50 mS_per_cm2', ion_channel='NaF', erev='55.0 mV', ion='na') This has the unfortunate consequence that the array in doc is singular: doc.ion_channel.append(chan) See #46
This is due to this line:
needed to ensure attribute ionChannel in ChannelDensity becomes ion_channel |
Could you explain this issue Padraig? It should either be singular or On 21 February 2014 18:10, Padraig Gleeson [email protected] wrote:
|
I see Note: I think making this change will change the signature of the NeuroMLDocument constructor etc., so this will be a backwards incompatible breaking change which will need to be clearly announced with an appropriate version bump + changelog entry etc.. |
e.g. see https://github.com/NeuralEnsemble/libNeuroML/blob/development/neuroml/examples/ion_channel_generation.py
Should be
doc.ion_channels.append(chan)
not
doc.ion_channel.append(chan)
The text was updated successfully, but these errors were encountered: