#715: Parents probably not reclaimed due to too much caching
--------------------------------+-------------------------------------------
Reporter: robertwb | Owner: somebody
Type: defect | Status: needs_info
Priority: major | Milestone: sage-4.8
Component: coercion | Keywords: weak cache coercion
Work_issues: avoid regression | Upstream: N/A
Reviewer: | Author: Simon King
Merged: | Dependencies: #9138, #11900
--------------------------------+-------------------------------------------
Changes (by SimonKing):
* status: needs_work => needs_info
Comment:
With the two patches applied, I get some doctest errors that seem trivial
to fix, and it takes 10905 seconds in total. Now, I am not sure:
Originally, I had much less time with unpatched Sage.
But perhaps my computer was in a different state at that time? Jean-
Pierre, if I understood correctly, you did not find any significant
slowdown, right?
The first (i.e., the "official") patch is enough to fix the leak for the
original example. According to Jean-Pierre, the timings are fine, it does
not matter whether we have no patch, the official patch only, or the first
experimental patch. And according to my own test, it does not matter
whether we have the first or the second experimental patch.
So, the further proceeding depends on the following questions:
* The experimental patches provide two different approaches to fix a
potential problem, namely actions being deallocated when they are still
needed. However, is this potential problem a ''real'' problem? Only then
would it make sense to consider the experimental patches!
* Do we also want to fix the leak in the more difficult example, namely
when the j-invariant varies? In this case, we need to find out why the
actions are registered in ZZ. It is not clear yet whether one really needs
one of the experimental patches to get rid of it.
What is your answer to the questions?
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/715#comment:90>
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.
