#7543: Add S-adic to the word generator
-----------------------------+----------------------------------------------
   Reporter:  slabbe         |       Owner:  slabbe           
       Type:  enhancement    |      Status:  positive_review  
   Priority:  major          |   Milestone:  sage-4.3.1       
  Component:  combinatorics  |    Keywords:                   
Work_issues:                 |      Author:  Sebastien Labbe  
   Upstream:  N/A            |    Reviewer:  Vincent Delecroix
     Merged:                 |  
-----------------------------+----------------------------------------------
Changes (by vdelecroix):

  * status:  needs_review => positive_review


Comment:

 Knowing that
 {{{
 sage: timeit('W.size_of_alphabet() not in ZZ')
 625 loops, best of 3: 24 µs per loop
 sage: timeit('W.size_of_alphabet() is Infinity')
 625 loops, best of 3: 3.43 µs per loop
 }}}

 We can win 21 micro seconds (at least on my computer) ! As it not
 important I switch to postive review...

 Nice patch this sadic one !

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/7543#comment:15>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB
-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/sage-trac?hl=en.


Reply via email to