#12220: Updated CBC spkg
--------------------------------------+-------------------------------------
Reporter: ncohen | Owner: ncohen
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-5.0
Component: linear programming | Resolution:
Keywords: sd35.5, Cernay2012 | Work issues:
Report Upstream: N/A | Reviewers:
Authors: John Perry | Merged in:
Dependencies: | Stopgaps:
--------------------------------------+-------------------------------------
Comment (by john_perry):
Replying to [comment:76 ncohen]:
> Yep, looks like that was the problem. Hopefully I did not know (or
either forgot) that the default value was specified there or I would not
have thought of changing it ! `:-p`
I wouldn't have thought of it. I'm quite certain that worked once.
> > Are you sure the model is automatically deleted? Currently, our code
deletes it (new line 692). I wouldn't want to introduce a new memory leak.
Otherwise, I don't object at all.
>
> Well, when you add a constraint to the LP you call
sage_free(self.solutions), so what about replacing that by del self.model
? If there is no way to do that without introducing a memory leak, we will
just refrain from doing it `:-D`
I'm okay with `del self.model`, as long as it's in there. `:-)`
> Ahah. Well, today is a *good* day. With my interview behind me, I just
have useful work to do, and it feels great ! Of course I also spent some
time tracking nasty bugs `:-p`
I hope the interview has a good result! and now I'd better get to writing
that email...
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12220#comment:77>
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.
