> 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?
--
Yaroslav O. Halchenko
Postdoctoral Fellow, Department of Psychological and Brain Sciences
Dartmouth College, 419 Moore Hall, Hinman Box 6207, Hanover, NH 03755
Phone: +1 (603) 646-9834 Fax: +1 (603) 646-1419
WWW: http://www.linkedin.com/in/yarik
--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
Archive: http://lists.debian.org/[email protected]
