unblocked anyway...
thanks Julien! you made me blush that I have missed them in my
quick review of the changes.
NB I am CCing upstream who might like to straighten those up
On Wed, 22 Aug 2012, Julien Cristau wrote:
def setup(timestep=0.1, min_delay=0.1, max_delay=0.1,
debug=False,**extra_params):
+
+ logger.debug("setup() called, extra_params =
"+str(extra_params))
this is broken, surely it should be below the docstring?
ideally -- yes -- but should be harmless
"""
Should be called at the very beginning of a script.
extra_params contains any keyword arguments that are required
by a given
simulator but not by others.
"""
also the sys.path.append('/usr/lib/python2.x/site-packages/oldxml')
seems like a horrible, horrible hack.
indeed... with not 100% clear motive (besides suggesting wanting
an older xml module, which was just bluntly manually copied on
that system).
Andrew -- could you clarify?:
src/neuroml.py:sys.path.append('/usr/lib/python%s/site-packages/
oldxml' % sys.version[:3]) # needed for Ubuntu
src/neuroml2.py:sys.path.append('/usr/lib/python%s/site-packages/
oldxml' % sys.version[:3]) # needed for Ubuntu
may be those are just left-overs which could be safely dropped?