On Jun 30, 12:22 am, William Stein <wst...@gmail.com> wrote:
> On Tue, Jun 30, 2009 at 3:11 AM, Dan Drake<dr...@kaist.edu> wrote:
> > On Mon, 29 Jun 2009 at 05:21PM -0700, John H Palmieri wrote:
> >> How about #6429: sagedoc: make search_src and friends less OS
> >> dependent?  This should fix the doctest failure for sagedoc.py on ppc
> >> OS X 10.4 (I hope).
>
> > I'd also like to see some testing of that patch on slower computers. The
> > patch uses os.walk and splitlines to search every line in the Sage
> > library...on my work computer it's plenty fast, but it's possible that
> > on some computers it's quite slow. But I have no idea how slow that
> > might be.
>
> It would be very nice if it built a reverse-lookup database, so that after the
> first search it is really fast.   When Alex Clemesha wrote a full text
> search system for Sage a long time ago he did that (I think via
> sqlite), and it was nice.
> The drawback is that when the Sage library changes, one has to know to
> not use the cache (or to properly update it).
>
>
>
> > I don't think being slow should hold up the patch, since "works, is
> > portable, but slow" should always beat "fast but broken/unreliable",
> > right? :)
>
> It depends on how slow it is.  If it is a hundred times slower, I wouldn't be
> so sure.

On my Intel iMac:
Before the patch:

sage: time s = search_src('matrix', interact=False)
CPU times: user 1.53 s, sys: 2.46 s, total: 4.00 s
Wall time: 4.51 s

After the patch:

sage: time s = search_src('matrix', interact=False)
CPU times: user 2.15 s, sys: 0.16 s, total: 2.31 s
Wall time: 2.31 s

It would be nice to see timings on other systems.

  John

--~--~---------~--~----~------------~-------~--~----~
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
-~----------~----~----~----~------~----~------~--~---

Reply via email to