#12616: The LP are not deallocated because of cyclic references !
----------------------------------+-----------------------------------------
Reporter: ncohen | Owner: ncohen
Type: defect | Status: positive_review
Priority: major | Milestone: sage-5.0
Component: linear programming | Keywords:
Work_issues: | Upstream: N/A
Reviewer: Simon King | Author: Nathann Cohen
Merged: | Dependencies: #11606
----------------------------------+-----------------------------------------
Description changed by jdemeyer:
Old description:
> Helloooooooooo !!!!
>
> Because of a cyclic reference (the LP pointed toward the LP variables,
> and the LP variables toward the LP), the LP objects were not immediately
> deallocated at the end of functions using them. That produced memory
> problems that have been reported there :
>
> http://ask.sagemath.org/question/1170/memory-blowup-with-milp
> http://ask.sagemath.org/question/1191/memory-blowup-2
> http://groups.google.com/forum/#!topic/sage-devel/4jJbIC5TqVs/discussion
>
> Thanks to the wonderful people one can find among Sage developpers who
> always have a perfect answer ready for you, here is a patch to fix it.
> It's actually quite straightforward (one of the two references was not
> even necessary, nor even used) when you know where it comes from, but
> without the good hint you can search for a long time `:-D`
>
> Nathann
>
> APPLY:
>
> * [attachment:trac_12616_reviewer]
> * [attachment:trac_12616_reviewer.patch]
New description:
Helloooooooooo !!!!
Because of a cyclic reference (the LP pointed toward the LP variables, and
the LP variables toward the LP), the LP objects were not immediately
deallocated at the end of functions using them. That produced memory
problems that have been reported there :
http://ask.sagemath.org/question/1170/memory-blowup-with-milp
http://ask.sagemath.org/question/1191/memory-blowup-2
http://groups.google.com/forum/#!topic/sage-devel/4jJbIC5TqVs/discussion
Thanks to the wonderful people one can find among Sage developpers who
always have a perfect answer ready for you, here is a patch to fix it.
It's actually quite straightforward (one of the two references was not
even necessary, nor even used) when you know where it comes from, but
without the good hint you can search for a long time `:-D`
Nathann
APPLY:
* [attachment:trac_12616.patch]
* [attachment:trac_12616_reviewer.patch]
--
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12616#comment:19>
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.
