#12973: Cannot copy() instances of GurobiBackend.
--------------------------------------------------------------------+-------
Reporter: e.vaughan |
Owner: jason, jkantor
Type: defect |
Status: needs_review
Priority: minor |
Milestone: sage-5.0.1
Component: linear programming |
Resolution:
Keywords: Gurobi, GurobiBackend, MixedIntegerLinearProgram | Work
issues:
Report Upstream: N/A |
Reviewers:
Authors: |
Merged in:
Dependencies: |
Stopgaps:
--------------------------------------------------------------------+-------
Comment (by ncohen):
Helloooooo !!!
> Thanks Nathann! I am a bit reluctant to upgrade to Sage 5.x to test the
patch due to the issues with Core2 Duo Macs....
Ahaha. That's understandable `:-D`
So the bug happens even when you compile from source ? Scary mac..
> Also, in my original patch, I *did* have a
>
> {{{cpdef GurobiBackend copy(self)}}}
>
> line.... Maybe that bit of the patch didn't apply properly in 5.1?
OOpps. Yes, probably. I looked at the list of rejected lines and saw so
many GUROBI -> Gurobi, I probably missed that one.
> One question: why have you changed copy to be a cdef instead of a cpdef?
(I used cpdef because that's what GLPKBackend uses (or did in 4.8).)
Oops, you are right. Actually, cpef functions are functions that can be
called at either C-level or Python level. Well, it more or less changes
nothing in this situation, as the code onl calls the copy method from the
backends at C level. It's actually totally up to you `:-)`
Nathann
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12973#comment:7>
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.
