#8611: speed up cached_function and cached_method
---------------------------------------+------------------------------------
   Reporter:  jason                    |       Owner:  tbd            
       Type:  enhancement              |      Status:  positive_review
   Priority:  major                    |   Milestone:  sage-4.6.2     
  Component:  misc                     |    Keywords:  cached method  
     Author:  Jason Grout, Simon King  |    Upstream:  N/A            
   Reviewer:  David Loeffler           |      Merged:                 
Work_issues:                           |  
---------------------------------------+------------------------------------
Changes (by davidloeffler):

  * status:  needs_review => positive_review
  * reviewer:  => David Loeffler


Comment:

 Applies fine and passes doctests both using 4.6.2.alpha0 on my system, and
 using 4.6.1 with patchbot. I ran some tests and in the case of a method
 with no arguments it's something like 15 times faster than the old code
 and only a tiny bit slower than a hand-rolled cache. Code looks fine by
 eye, and it's great that you added these modules to the ref manual as
 well. Positive review.

 I'm looking forward to using %prun in future sage versions and not finding
 that 20% of the execution time has been spent in "fix_to_pos", which is
 frequently the case at the moment...

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8611#comment:17>
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