#13922: Avoid a regression in the creation of homsets
-------------------------------+--------------------------------------------
       Reporter:  SimonKing    |         Owner:  tbd         
           Type:  defect       |        Status:  needs_review
       Priority:  critical     |     Milestone:  sage-5.6    
      Component:  performance  |    Resolution:              
       Keywords:               |   Work issues:              
Report Upstream:  N/A          |     Reviewers:              
        Authors:  Simon King   |     Merged in:              
   Dependencies:  #715         |      Stopgaps:              
-------------------------------+--------------------------------------------
Changes (by {'newvalue': u'Simon King', 'oldvalue': ''}):

  * status:  new => needs_review
  * author:  => Simon King


Comment:

 The attached patch is preliminary, as it is not tested, and I don't know
 if I was over-eager to cdefine stuff.

 Anyway. Without the patch, it is
 {{{
 sage: %timeit a = QQ.gcd(3.14159,2*3.14159)
 125 loops, best of 3: 4.58 ms per loop
 sage: var('t')
 t
 sage: %time p = plot(lambda t:
 (100/(100+(t-pi/2)^8))*(2-sin(7*t)-cos(30*t)/2), -pi/4, 3*pi/2,
 color="red",plot_points=1000)
 CPU times: user 87.06 s, sys: 0.69 s, total: 87.75 s
 Wall time: 88.15 s
 }}}

 With the patch, it is
 {{{
 sage: %timeit a = QQ.gcd(3.14159,2*3.14159)
 125 loops, best of 3: 2.33 ms per loop
 sage: var('t')
 t
 sage: %time p = plot(lambda t:
 (100/(100+(t-pi/2)^8))*(2-sin(7*t)-cos(30*t)/2), -pi/4, 3*pi/2,
 color="red",plot_points=1000)
 CPU times: user 55.41 s, sys: 0.00 s, total: 55.41 s
 Wall time: 55.48 s
 }}}

 Would that be enough to fix the regression completely?

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