#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):
Hi Jeroen,
I understand your results that cyclomat.sage, elltest2.sage and
J0_46_disc.sage are fine, while ellbsd.sage and elltest1.sage remain
problematic.
I did five runs for each of the problematic tests, with sage-4.7.2.alpha2
and sage-4.7.2.alpha3+patches.
__ellbsd.sage__
4.7.2.alpha2
{{{
king@mpc622:/mnt/local/king/SAGE/debug/sage-4.7.2.alpha2$ time ./sage
ellbsd.sage
real 0m6.344s
user 0m5.180s
sys 0m0.380s
king@mpc622:/mnt/local/king/SAGE/debug/sage-4.7.2.alpha2$ time ./sage
ellbsd.sage
real 0m5.331s
user 0m4.932s
sys 0m0.308s
king@mpc622:/mnt/local/king/SAGE/debug/sage-4.7.2.alpha2$ time ./sage
ellbsd.sage
real 0m5.260s
user 0m4.812s
sys 0m0.356s
king@mpc622:/mnt/local/king/SAGE/debug/sage-4.7.2.alpha2$ time ./sage
ellbsd.sage
real 0m5.270s
user 0m4.848s
sys 0m0.336s
king@mpc622:/mnt/local/king/SAGE/debug/sage-4.7.2.alpha2$ time ./sage
ellbsd.sage
real 0m5.215s
user 0m4.836s
sys 0m0.288s
}}}
patched 4.7.2.alpha3
{{{
king@mpc622:/mnt/local/king/SAGE/rebase/sage-4.7.2.alpha3$ time ./sage
ellbsd.sage
real 0m6.392s
user 0m5.756s
sys 0m0.496s
king@mpc622:/mnt/local/king/SAGE/rebase/sage-4.7.2.alpha3$ time ./sage
ellbsd.sage
real 0m5.928s
user 0m5.448s
sys 0m0.384s
king@mpc622:/mnt/local/king/SAGE/rebase/sage-4.7.2.alpha3$ time ./sage
ellbsd.sage
real 0m5.654s
user 0m5.236s
sys 0m0.368s
king@mpc622:/mnt/local/king/SAGE/rebase/sage-4.7.2.alpha3$ time ./sage
ellbsd.sage
real 0m5.576s
user 0m5.204s
sys 0m0.340s
king@mpc622:/mnt/local/king/SAGE/rebase/sage-4.7.2.alpha3$ time ./sage
ellbsd.sage
real 0m5.599s
user 0m5.156s
sys 0m0.364s
}}}
__elltest1.sage__
4.7.2.alpha2
{{{
king@mpc622:/mnt/local/king/SAGE/debug/sage-4.7.2.alpha2$ time ./sage
elltest1.sage
real 0m19.189s
user 0m18.669s
sys 0m0.352s
king@mpc622:/mnt/local/king/SAGE/debug/sage-4.7.2.alpha2$ time ./sage
elltest1.sage
real 0m18.909s
user 0m18.417s
sys 0m0.380s
king@mpc622:/mnt/local/king/SAGE/debug/sage-4.7.2.alpha2$ time ./sage
elltest1.sage
real 0m19.463s
user 0m19.085s
sys 0m0.260s
king@mpc622:/mnt/local/king/SAGE/debug/sage-4.7.2.alpha2$ time ./sage
elltest1.sage
real 0m19.607s
user 0m19.205s
sys 0m0.276s
king@mpc622:/mnt/local/king/SAGE/debug/sage-4.7.2.alpha2$ time ./sage
elltest1.sage
real 0m18.897s
user 0m18.445s
sys 0m0.328s
}}}
4.7.2.alpha3+patches
{{{
king@mpc622:/mnt/local/king/SAGE/rebase/sage-4.7.2.alpha3$ time ./sage
elltest1.sage
real 0m18.420s
user 0m17.989s
sys 0m0.308s
king@mpc622:/mnt/local/king/SAGE/rebase/sage-4.7.2.alpha3$ time ./sage
elltest1.sage
real 0m18.273s
user 0m17.833s
sys 0m0.372s
king@mpc622:/mnt/local/king/SAGE/rebase/sage-4.7.2.alpha3$ time ./sage
elltest1.sage
real 0m18.461s
user 0m17.997s
sys 0m0.380s
king@mpc622:/mnt/local/king/SAGE/rebase/sage-4.7.2.alpha3$ time ./sage
elltest1.sage
real 0m18.416s
user 0m18.005s
sys 0m0.348s
king@mpc622:/mnt/local/king/SAGE/rebase/sage-4.7.2.alpha3$ time ./sage
elltest1.sage
real 0m19.687s
user 0m18.997s
sys 0m0.588s
}}}
'''__Conclusion__'''
I see a rather small slow-down in the first test, when comparing
sage-4.7.2.alpha2 with "baseline+#9138+#11900"; it is smaller than the
regression that you observe.
I can not confirm the slow-down that you observe in the second test: Here,
I actually find a slight speed-up.
My setting is different, as I replace "baseline+#9138+#11900" with
"sage-4.7.2.alpha3+#11900". Just to be on the safe side, here you see what
I was working with:
{{{
king@mpc622:/mnt/local/king/SAGE/debug/sage-4.7.2.alpha2/devel/sage$ hg
qtop
Keine Patches angewendet
}}}
versus
{{{
king@mpc622:/mnt/local/king/SAGE/rebase/sage-4.7.2.alpha3/devel/sage$ hg
qapplied
trac11911_libsingular_pickling.patch
trac11900_no_categories_for_matrices.patch
trac11900_category_speedup.patch
trac11900_further_tweaks.patch
}}}
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11900#comment:85>
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.
