#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 ncohen):
> so shouldn't `copyData` be initialized to `true` if it isn't? or could
it be Cython that's sending a default value of 0?
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`
> 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`
> Yeah, thanks for that. I seriously hadn't been able to get to that; I
had hoped to take care of it yesterday myself, but I got so bogged down in
trying to find a simple example of cycling in simplex that I never had the
chance.
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`
Nathann
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12220#comment:76>
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.
