#13394: Write a WeakValueDictionary with safer key removal
-------------------------------------+-------------------------------------
Reporter: nbruin | Owner: rlm
Type: enhancement | Status: needs_review
Priority: major | Milestone: sage-5.13
Component: memleak | Resolution:
Keywords: | Merged in:
Authors: Simon King | Reviewers:
Report Upstream: None of the above | Work issues:
- read trac for reasoning. | Commit:
Branch: | 70a7b8accb2d2156f315cbae2511985936977670
u/SimonKing/ticket/13394 | Stopgaps:
Dependencies: |
-------------------------------------+-------------------------------------
Comment (by SimonKing):
With the latest commit, there is a context manager preventing the
dictionary from negative effects of deletions. The doctests show what the
context manager is capable of.
It is still "needs review". Perhaps one should test if the whole thing is
still faster than `weakref.WeakValueDictionar`. Even if it is (more or
less) equally fast, it is a progress, since
`sage.misc.weak_dict.WeakValueDictionary` now seems to be bullet proof
regarding garbage collection and explicit deletions.
Challange: Try to break it...
--
Ticket URL: <http://trac.sagemath.org/ticket/13394#comment:24>
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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/groups/opt_out.
