#715: Parents probably not reclaimed due to too much caching
----------------------------------+-----------------------------------------
Reporter: robertwb | Owner: somebody
Type: defect | Status: needs_work
Priority: major | Milestone: sage-4.8
Component: coercion | Keywords: weak cache coercion
Work_issues: Avoid a regression | Upstream: N/A
Reviewer: | Author: Simon King
Merged: | Dependencies: #9138, #11900
----------------------------------+-----------------------------------------
Changes (by SimonKing):
* status: needs_review => needs_work
* work_issues: => Avoid a regression
Comment:
We have a big regression.
I considered the doctests of sage/modules/free_module.py and took each
timing twice, in order to be on the safe side.
Vanilla 5.0.prealpha0
{{{
sage -t "devel/sage-main/sage/modules/free_module.py"
[11.9 s]
sage -t "devel/sage-main/sage/modules/free_module.py"
[10.3 s]
}}}
With the first patch from here:
{{{
sage -t "devel/sage-main/sage/modules/free_module.py"
[24.1 s]
sage -t "devel/sage-main/sage/modules/free_module.py"
[25.7 s]
}}}
With both patches from here:
{{{
sage -t "devel/sage-main/sage/modules/free_module.py"
[26.0 s]
sage -t "devel/sage-main/sage/modules/free_module.py"
[25.8 s]
}}}
I think such a huge regression can't be accepted. Thus, it is "needs
work".
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/715#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.
