#14058: Weakly reference binary operation codomains
-------------------------------------+-------------------------------------
Reporter: robertwb | Owner: rlm
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-6.4
Component: memleak | Resolution:
Keywords: | Merged in:
Authors: Robert Bradshaw, | Reviewers: Simon King, Frédéric
Nils Bruin | Chapoton, Jean-Pierre Flori
Report Upstream: N/A | Work issues:
Branch: | Commit:
public/ticket/14058 | 8713960717f916d459e0b6a0dcef391dc992d9b2
Dependencies: #12313 | Stopgaps:
-------------------------------------+-------------------------------------
Comment (by slabbe):
On sage-6.9.beta1, I get:
{{{
$ sage nbruin.sage
Time: CPU 7.95 s, Wall: 7.95 s
Time: CPU 7.68 s, Wall: 7.68 s
Time: CPU 76.76 s, Wall: 76.94 s
}}}
On sage-6.9.beta1 + #14058, I get a very good improvement:
{{{
$ sage nbruin.sage
Time: CPU 2.48 s, Wall: 2.48 s
Time: CPU 2.07 s, Wall: 2.07 s
Time: CPU 32.38 s, Wall: 32.42 s
}}}
Note: I originally thought it [https://groups.google.com/d/msg/sage-
devel/n7_21tCO4zI/cuCAlO01DwAJ was a regression in sage] since I was
getting this earlier this morning with sage-6.8 + #14058. But thanksfully,
it seems to be a improvement obtained from this ticket.
The file `nbruin.sage` is from
[http://trac.sagemath.org/ticket/14058#comment:4 comment 4]:
{{{
$ cat nbruin.sage
a=ZZ(1)
b=QQ(1)
c=ZZ['x'](1)
d=b+c
def test(x,y):
for i in xrange(10^6):
_=x+y
time test(a,b)
time test(a,c)
time test(b,c)
}}}
--
Ticket URL: <http://trac.sagemath.org/ticket/14058#comment:63>
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/d/optout.
