#11734: sage_wraps should only read the sources of wrapped functions when 
needed.
--------------------------+-------------------------------------------------
   Reporter:  SimonKing   |          Owner:  jason                              
       
       Type:  defect      |         Status:  needs_review                       
       
   Priority:  blocker     |      Milestone:  sage-4.7.2                         
       
  Component:  misc        |       Keywords:  sage_wraps sources gentoo 
startuptime sd32
Work_issues:              |       Upstream:  N/A                                
       
   Reviewer:  Simon King  |         Author:  Julian Rueth                       
       
     Merged:              |   Dependencies:                                     
       
--------------------------+-------------------------------------------------

Comment(by saraedum):

 I spend quite a bit of time looking at that speedup yesterday. While
 everything you say is certainly correct, a big part comes from calling the
 constructor I mentioned less frequently. It turned out that removing the
 decorator from ''any'' interacts control would almost yield the same
 speedup - even though the that control is never referenced in the
 --startuptime call.
 If I trust cProfile then there is a single call to the ArgSpec constructor
 that is slow after a certain number of calls. Maybe cProfile is misleading
 here, and actually something else is happening...anyway, it's faster now,
 I don't think it introduces any new bug, so I'm happy with the result ;)

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