#9138: Categories for all rings
--------------------------+-------------------------------------------------
   Reporter:  jbandlow    |       Owner:  nthiery                            
       Type:  defect      |      Status:  needs_review                       
   Priority:  major       |   Milestone:  sage-4.7                           
  Component:  categories  |    Keywords:  introspection, categories for rings
     Author:  Simon King  |    Upstream:  N/A                                
   Reviewer:              |      Merged:                                     
Work_issues:              |  
--------------------------+-------------------------------------------------

Comment(by SimonKing):

 To be on the safe side, I repeated timings with the latest patch from
 here, applied only on top of #9944:
 {{{
 $ ./sage -startuptime
 ...
 == Slowest (including children) ==
 1.396 sage.all (None)
 0.398 sage.misc.all (sage.all)
 0.285 functional (sage.misc.all)
 0.282 sage.schemes.all (sage.all)
 0.271 sage.rings.complex_double (functional)
 ...
 }}}
 and
 {{{
 sage: B.<t> = PolynomialRing(Integers(125))
 sage: R = monsky_washnitzer.SpecialCubicQuotientRing(t^3 - t + B(1/4))
 sage: P = R.poly_ring()
 sage: x, T = R.gens()
 sage: timeit('x*T')
 625 loops, best of 3: 869 µs per loop
 }}}

 Without all these patches, I get
 {{{
 $ ./sage -startuptime
 ...
 == Slowest (including children) ==
 1.282 sage.all (None)
 0.284 sage.schemes.all (sage.all)
 0.241 sage.misc.all (sage.all)
 0.184 twisted.persisted.styles (sage.all)
 0.165 elliptic_curves.all (sage.schemes.all)
 ...
 }}}
 and
 {{{
 sage: B.<t> = PolynomialRing(Integers(125))
 sage: R = monsky_washnitzer.SpecialCubicQuotientRing(t^3 - t + B(1/4))
 sage: P = R.poly_ring()
 sage: x, T = R.gens()
 sage: timeit('x*T')
 625 loops, best of 3: 501 µs per loop
 }}}

 I think the timing of the startup time is within the error margin. But
 there is a slow-down of the Monsky-Washnitzer code. I have shown how to
 solve the latter problem, based on #11115.

 I leave the decision to the reviewer whether or not #11115, together with
 a patch that puts a cached_method decorator in front of the modulus
 method, should be a dependency for this ticket.

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