#11900: Serious regression caused by #9138
------------------------------+---------------------------------------------
Reporter: SimonKing | Owner: tbd
Type: defect | Status: needs_work
Priority: critical | Milestone: sage-4.8
Component: performance | Keywords: categories regression
Work_issues: | Upstream: N/A
Reviewer: Jeroen Demeyer | Author: Simon King
Merged: | Dependencies: #9138 #11911
------------------------------+---------------------------------------------
Comment(by SimonKing):
And the very good news is:
{{{
sage: R = Rings()
sage: MS = MatrixSpace(QQ,2)
sage: MS.is_ring.__module__
'sage.categories.rings'
sage: %timeit MS.is_ring()
625 loops, best of 3: 3.94 µs per loop
sage: %timeit MS in R
625 loops, best of 3: 1.41 µs per loop
}}}
So, the parent method is_ring that I introduced for speed reasons is now
not the fastest thing to do. However, we still have
{{{
sage: P.<x,y,z> = QQ[]
sage: %timeit P.is_ring() # that's a cython method defined for
sage.rings.ring.Ring
625 loops, best of 3: 400 ns per loop
}}}
and
{{{
sage: %timeit MS in Rings() # Adding the time needed to call Rings()
625 loops, best of 3: 5.09 µs per loop
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11900#comment:108>
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.
