#12823: Allow constants for objective function & deletion of rows in
MixedIntegerLinearProgram
--------------------------------------+-------------------------------------
       Reporter:  john_perry          |         Owner:  ncohen          
           Type:  defect              |        Status:  needs_work      
       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):

 Apparently Nathann ran a doctest on everything '''except''' `mip.pyx`...
 LOL

 Replying to [comment:13 davidloeffler]:
 > Patchbot's not happy:
 > {{{
 > **********************************************************************
 > File
 
"/storage/masiao/sage-5.0.beta13-patchbot/devel/sage-12823/sage/numerical/mip.pyx",
 line 1101:
 >     sage: p.number_of_constraints()
 > Expected:
 >     3
 > Got:
 >     2
 > }}}

 Yes, `3` in the doctest should change to `2`.

 > {{{
 > **********************************************************************
 > File
 
"/storage/masiao/sage-5.0.beta13-patchbot/devel/sage-12823/sage/numerical/mip.pyx",
 line 1131:
 >     sage: p.remove_constraint([0, 1])
 > Exception raised:
 >
 > ...
 >
 > File
 
"/storage/masiao/sage-5.0.beta13-patchbot/devel/sage-12823/sage/numerical/mip.pyx",
 line 1132:
 >     sage: p.show()
 > Expected:
 >     Maximization:
 >     <BLANKLINE>
 >     Constraints:
 >       x_0 <= 4.0
 >     ...
 > Got:
 >     Maximization:
 >     <BLANKLINE>
 >     Constraints:
 >       x_0 + x_1 <= 10.0
 >       x_0 - x_1 <= 0.0
 >       x_0 <= 4.0
 >     Variables:
 >       x_0 is a continuous variable (min=0.0, max=+oo)
 >       x_1 is a continuous variable (min=0.0, max=+oo)
 > }}}

 These are both because line 1127, which used to have
 {{{
 sage: p.remove_constraints([2])
 }}}
 was changed to
 {{{
 sage: p.remove_constraint([0,1])
 }}}
 Note the missing '''s'''. Fixing that should fix these tests. I'll leave
 it to Nathann, since these come from his patch.

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