#11900: Serious regression caused by #9138
---------------------------+------------------------------------------------
   Reporter:  SimonKing    |          Owner:  tbd                  
       Type:  defect       |         Status:  new                  
   Priority:  blocker      |      Milestone:  sage-4.7.2           
  Component:  performance  |       Keywords:  categories regression
Work_issues:               |       Upstream:  N/A                  
   Reviewer:               |         Author:                       
     Merged:               |   Dependencies:                       
---------------------------+------------------------------------------------

Comment(by SimonKing):

 When I use strong references for caching matrix spaces (in addition to the
 previous changes), I get
 {{{
 sage: E = J0(46).endomorphism_ring()
 sage: %time g = E.gens()
 CPU times: user 6.15 s, sys: 0.16 s, total: 6.31 s
 Wall time: 6.33 s
 }}}

 Of course, one wouldn't like to have all the matrix spaces hanging around.
 But perhaps it is an option to temporarily turn garbage collection off.

 However, the computation of `E.gens()` actually requires the construction
 of many ''different'' matrix spaces. So, making matrix space creation
 faster certainly is the best option.

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