#11521: Memleak when resolving the action of Integers on an Elliptic Curve
------------------------+---------------------------------------------------
Reporter: jpflori | Owner: robertwb
Type: defect | Status: needs_info
Priority: major | Milestone: sage-4.8
Component: coercion | Keywords:
Work_issues: | Upstream: N/A
Reviewer: | Author:
Merged: | Dependencies:
------------------------+---------------------------------------------------
Comment(by SimonKing):
Hi Paul,
Replying to [comment:42 zimmerma]:
> Simon, you can also use the test suggested by Jean-Pierre Flori (see
comment [comment:18] for an
> example).
Yes, that looks good. With my patch, the test would be like
{{{
sage: for p in prime_range(10^5):
....: K = GF(p)
....: a = K(0)
....:
sage: import gc
sage: gc.collect()
1881
sage: from sage.rings.finite_rings.finite_field_prime_modn import
FiniteField_prime_modn as FF
sage: L = [x for x in gc.get_objects() if isinstance(x, FF)]
sage: len(L), L[0], L[len(L)-1]
(2, Finite Field of size 2, Finite Field of size 99991)
}}}
The people at sage-devel somehow seem to agree that objects of a category
should be instances of `CategoryObject` (which elements aren't!), and that
we should thus drop the `Hom(2/3,8/1)` test.
In addition to that, I suggest to provide a better error message,
something like
{{{
sage: Hom(2/3, 8/1)
Traceback (most recent call last):
...
TypeError: Objects of categories must be instances of <type
'sage.structure.category_object.CategoryObject'>, but 2/3 isn't.
}}}
Cheers,
Simon
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/11521#comment:43>
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.
