#11521: Use weak references to cache homsets
--------------------------------------------------+-------------------------
Reporter: jpflori | Owner: robertwb
Type: defect | Status: needs_work
Priority: major | Milestone: sage-5.5
Component: coercion | Resolution:
Keywords: sd35 | Work issues:
Report Upstream: N/A | Reviewers: Jean-Pierre
Flori, Nils Bruin
Authors: Simon King | Merged in:
Dependencies: #12969; to be merged with #715 | Stopgaps:
--------------------------------------------------+-------------------------
Comment (by SimonKing):
Replying to [comment:179 jdemeyer]:
> Sorry to spoil the party, but with sage-5.4.rc0 + #715 + #11521, I get
> {{{
> sage -t "devel/sage/sage/libs/singular/ring.pyx"
> **********************************************************************
> File
"/release/merger/sage-5.5.beta0/devel/sage/sage/libs/singular/ring.pyx",
line 490:
> sage: ring_ptr in ring_refcount_dict
> Expected:
> False
> Got:
> True
}}}
I guess this is expected, because of the newly introduced strong cache for
polynomial rings at the latest version of #715.
> **********************************************************************
> File
"/release/merger/sage-5.5.beta0/devel/sage/sage/rings/polynomial/multi_polynomial_libsingular.pyx",
line 423:
> sage: len(ring_refcount_dict) == n
> Expected:
> True
> Got:
> False
Same as above: This test was supposed to show that garbage collection for
polynomial rings happens, but it doesn't, with a strong cache.
> **********************************************************************
> sage -t
"devel/sage/sage/rings/polynomial/multi_polynomial_libsingular.pyx"
> Exception AttributeError:
AttributeError('PolynomialRing_field_with_category' object has no
attribute '_modulus',) in ignored
> Exception AttributeError:
AttributeError('PolynomialRing_field_with_category' object has no
attribute '_modulus',) in ignored
I guess that is more serious. IIRC, it comes from code in
polynomial_template.
Anyway. If that kind of problems persist, then I guess we need #13447.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11521#comment:181>
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.
