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.

 -- William

>
> Dan
>
> --
> ---  Dan Drake <dr...@kaist.edu>
> -----  KAIST Department of Mathematical Sciences
> -------  http://mathsci.kaist.ac.kr/~drake
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.9 (GNU/Linux)
>
> iEYEARECAAYFAkpJZi8ACgkQr4V8SljC5Lp9xgCgpND0XMj9FKBHlgNVdNUoEw7+
> fT0AoLVhGkMzYv0lLA5+W9kLBOa/FvYh
> =mMkP
> -----END PGP SIGNATURE-----
>
>



-- 
William Stein
Associate Professor of Mathematics
University of Washington
http://wstein.org

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