#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:70 ncohen]:
> Well.... If instead of calling self.si.clone() we call self.si.clone(1),
the problem is solved.
Oh, wow. Seriously: '''wow'''. I thought that took care of itself; after
all, the C++ signature is
{{{
virtual OsiSolverInterface * clone(bool copyData = true) const = 0;
}}}
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? (maybe the new version? the
new Cython has several regressions from my point of view, though the
developers call them bug fixes. `;-)`) Seriously, this worked for me once.
I wonder if it worked when I put in `1` & then thought it was something
else...
Replying to [comment:73 ncohen]:
> Tell me John : instead of copying the whole solution inside of the
Python structure, what would you think of just storing the "model" object
? As you do, it could be automatically destroyed when the LP is changed,
actually all that you do with .solutions could be done with .model, and
then we would not have to copy solutions over and over ? What do you think
? `:-)`
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.
> I am glad that this updated spkg at least passes all tests..... `:-)`
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.
Meanwhile, I've discovered a new GLPK bug; I'll tell you about it in
email, though.
Hey, I just noticed the new trac puts a preview of the message beneath the
text box. '''Awesome.'''
On the other hand, Firefox just crashed, too. Argh...
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12220#comment:74>
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.
