Changes between Version 112 and Version 126 of Ticket #11521


Ignore:
Timestamp:
03/21/12 10:21:03 (9 years ago)
Author:
jpflori
Comment:

Here is a reviewer patch added a little doc.

I hope you will find it satisfactory.

If so, feel free to set the ticket to positive review, I personnally feel happy with youre code.

My above rant, if it should be addressed, should be elsewhere.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #11521

    • Property Summary changed from Memleak when resolving the action of Integers on an Elliptic Curve to Use weak references to cache homsets
  • Ticket #11521 – Description

    v112 v126  
    11Originally, this ticket was about the following memory leak when computing with elliptic curves:
     2
    23{{{
    34sage: K = GF(1<<55,'t')
     
    67....:     E = EllipticCurve(j=a); P = E.random_point(); 2*P;
    78}}}
     9This example is in fact solved by #715. However, while working on that ticket, another leak has been found, namely
    810
    9 This example is in fact solved by #715. However, while working on that ticket, another leak has been found, namely
    1011{{{
    1112sage: for p in prime_range(10^5):
     
    17180
    1819}}}
    19 
    2020So, I suggest to start with #715 and solve the second memory leak on top of it. It seems that a strong cache for homsets is to blame. I suggest to use the weak `TripleDict` instead, which were introduced in #715.
    21 
    2221
    2322'''Apply'''
    2423
    25 [attachment:trac11521_triple_homset.patch]
     24* [attachment:trac11521_triple_homset.patch]
     25* [attachment:trac_11521-reviewer.patch]