On May 14, 2009, at 1:18 AM, Rado wrote: > Last question if I used ./sage -clone myvrr and made the changes in > myver, how do I tell sage to run the tests there (if this even makes > sense?)
You can do ./sage -t devel/sage-myvrr/sage/graphs/... # or -tp 10 - Robert > > Rado > > On May 14, 3:13 am, mabshoff <mabsh...@googlemail.com> wrote: >> On May 14, 1:08 am, Rado <rki...@gmail.com> wrote: >> >>> The bug is almost trivial. The code >> >>> verts = data.keys() >>> .... >>> for u in data: >>> verts.union([v for v in data[u] if v not in verts]) >> >>> is slowing down because in python searching in lists is slow. If we >>> use "verts = set(data.keys())" the code speeds up tremendously. >> >> Nice catch. No idea if this is the fix, but we will see I guess ;). >> >>> sage: D={} >>> sage: for i in xrange(10^3): >>> ....: D[i]=[i+1,i-1] >>> ....: >>> sage: timeit('g=Graph(D)') >>> 5 loops, best of 3: 79.6 ms per loop >> >>> Before I submit a patch how do I run the the graph theory >>> doctests to >>> make sure nothing else is broken by changing verts from list to set? >> >> Run "make check" in $SAGE_ROOT or >> >> ./sage -tp 1 devel/sage >> >> where the number behind -tp is the number of tests run in parallel. >> >>> Rado >> >> Cheers, >> >> Michael > > --~--~---------~--~----~------------~-------~--~----~ To post to this group, send email to sage-devel@googlegroups.com To unsubscribe from this group, send email to sage-devel-unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URLs: http://www.sagemath.org -~----------~----~----~----~------~----~------~--~---