Ankur Sinha

Citing generateDS

+1 for this feature (although I don't know how much work this is to implement). At the moment, our classes all look like this: Izhikevich2007Cell(neuro_lex_id=None, id=None, metaid=None, notes=None, properties=None, annotation=None, C=None, v0=None, k=None, vr=None, vt=None, vpeak=None, a=None, b=None, c=None, d=None, gds_collector_=None, **kwargs_) which means users don't really get any information about what each argument to the constructor should be---they need to go look at our schema documentation...

Hi Dave, I think I figured out what was going on. In the generated nml.py file, in the __str__ method of GeneratedsSuper, we're initialising a new output variable as an object of the StringIO class. At the moment, we simply from io import StringIO, which while works for Python 3, isn't the right StringIO to use for Python 2. In Python 2, for strings, we must use StringIO.StringIO because there, io.StringIO is for unicode strings. So, I think the generateds.py code should be tweaked as per this diff:...

Hi Dave, I haven't been able to pin point the cause of the error yet but I did find a workaround that fixes the issue for us: https://github.com/NeuralEnsemble/libNeuroML/pull/110/commits/aa025c608a678697d14acaca1cf8d7ed0a0a7fd6 The error is caused when we use nml.py with other bits, like reading from HDF5 files, so I'm thinking it's caused by some code in our HDF5 related methods where we haven't been careful enough to take both Python 2 and 3 into account. So we'll just carry this tweak for the...

Hi Dave, Thanks for that. I'll go through the failing test and see if I can narrow it down. If you're not seeing the error, it could be something else in the library that's using nml.py perhaps. I'll report back once I have something. Thanks again, Ankur

Python2.7: TypeError: unicode argument expected, got 'str'

Thanks very much, Dave. I can confirm that this is now fixed. Please close this ticket as "fixed" :)

Thanks very much, and sorry for the late reply. I've attached the XSD and other files we use for generation to the issue. You can also see them in our repository here: https://github.com/NeuralEnsemble/libNeuroML/tree/development/neuroml/nml The schema is currently at version 2.2