reloading data uses quantity path string instead of column id - Githubissues

NeuroML / pyNeuroML

A single package in Python unifying scripts and modules for reading, writing, simulating and analysing NeuroML2/LEMS models.

https://docs.neuroml.org/Userdocs/Software/pyNeuroML.html

GNU Lesser General Public License v3.0

34 stars 30 forks source link

reloading data uses quantity path string instead of column id #225

Closed sanjayankur31 closed 5 months ago

sanjayankur31 commented 1 year ago

e.g.:

dict_keys(['t', 'TestPop[0]/v', 'TestPop[0]/i', 'TestPop[0]/synapses:probAMPANMDASyn:0/i_AMPA', 'TestPop[0]/synapses:probAMPANMDASyn:0/i_NMDA', 'TestPop[0]/synapses:probAMPANMDASyn:0/g_AMPA', 'TestPop[0]/synapses:probAMPANMDASyn:0/g_NMDA'])

is this expected, or should it be using column id and this is a bug in our method
if it's expected, should it use column id instead given that these are generally rather long and cumbersome? (note---changing will be a breaking change, so perhaps we leave it as is by default but extend the function to allow using column ids instead too)