Skip to content
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

Open
pgleeson opened this issue Feb 21, 2014 · 3 comments
Labels
T: bug Type: bug T: question Type: question

Comments

@pgleeson
Copy link
Member

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)

pgleeson added a commit that referenced this issue Feb 21, 2014
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
@pgleeson
Copy link
Member Author

This is due to this line:

NameTable['ionChannel'] = 'ion_channel'

needed to ensure attribute ionChannel in ChannelDensity becomes ion_channel

@vellamike
Copy link
Contributor

Could you explain this issue Padraig? It should either be singular or
plural, there shouldn't be the choice. Shouldn't 'ion_channel' just be
changed to 'ion_channels' ?

On 21 February 2014 18:10, Padraig Gleeson [email protected] wrote:

This is due to this line:

NameTable['ionChannel'] = 'ion_channel'

needed to ensure attribute ionChannel in ChannelDensity becomes ion_channel

Reply to this email directly or view it on GitHubhttps://github.com//issues/46#issuecomment-35756888
.

@sanjayankur31 sanjayankur31 self-assigned this Jun 18, 2021
@sanjayankur31 sanjayankur31 added T: bug Type: bug T: question Type: question labels Jun 18, 2021
@sanjayankur31
Copy link
Contributor

I see ion_channel but ion_channel_hhs etc. so I think this is still an issue.

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..

@sanjayankur31 sanjayankur31 removed their assignment Jul 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T: bug Type: bug T: question Type: question
Projects
None yet
Development

No branches or pull requests

3 participants