#11900: Serious regression caused by #9138
---------------------------+------------------------------------------------
Reporter: SimonKing | Owner: tbd
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.7.3
Component: performance | Keywords: categories regression
Work_issues: | Upstream: N/A
Reviewer: | Author: Simon King
Merged: | Dependencies: #9138 #11911
---------------------------+------------------------------------------------
Comment(by SimonKing):
A very pleasant result:
I was running the doc tests in sage/schemes (which seem to be the most
critical). With sage-4.7.2.alpha2 (hence, without #9138), I obtain:
{{{
All tests passed!
Total time for all tests: 663.5 seconds
}}}
With sage-4.7.2.alpha3 (hence, with #9138) and the patches from #11911 and
here, I obtain
{{{
All tests passed!
Total time for all tests: 597.0 seconds
}}}
In other words: The regression from #9138 apparently turned into a speed-
up!
At Jeroen: You mentioned that you would not review it, because you are not
so familiar with coercion. However, my own impression is that coercion
plays almost no role in my patches (note that "new coercion for polynomial
rings" is ''not'' part of the patches - it will be moved to a different
ticket).
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11900#comment:78>
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.
