#12823: Allow constants for objective function & deletion of rows in
MixedIntegerLinearProgram
--------------------------------------+-------------------------------------
       Reporter:  john_perry          |         Owner:  ncohen          
           Type:  defect              |        Status:  needs_review    
       Priority:  major               |     Milestone:  sage-5.0        
      Component:  linear programming  |    Resolution:                  
       Keywords:  solver objective    |   Work issues:  failing doctests
Report Upstream:  N/A                 |     Reviewers:                  
        Authors:  john_perry, ncohen  |     Merged in:                  
   Dependencies:  12833               |      Stopgaps:                  
--------------------------------------+-------------------------------------

Comment (by john_perry):

 I've examined the code for the patch. Only one question: in the Gurobi
 backend, for `remove_constraints`, why do you loop and call
 `remove_constraint` on each index? The signature for `GRBdelconstrs()`
 seems to allow sending an index, just like GLPK and Coin. I'm not looking
 at the documentation for it, though, so maybe that's okay.

 It's pretty clear that CPLEX is different, so I understand the loop there.

 Anyway, I'd like to test the patch ASAP, and give it a positive review,
 but -- I don't have either CPLEX or Gurobi installed. I understand those
 aren't free like Coin and GLPK. How should I test the doctests? Must I
 download them, or can I take you at your word that you've doctested them
 and the doctests check out?

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/12823#comment:16>
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.

Reply via email to