Changes between Initial Version and Version 1 of Ticket #20324


Ignore:
Timestamp:
04/05/16 20:13:49 (6 years ago)
Author:
mkoeppe
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #20324

    • Property Summary changed from MIP backends: Unify add_col, add_variable to MIP backends: Unify add_col, add_variable; and clarify defaulting behavior of binary, continuous, integer
  • Ticket #20324 – Description

    initial v1  
    33Note that `add_col` is not used by `MixedIntegerLinearProgram`; it is only used in doctests of the backends. 
    44(Also compare with `add_linear_constraint`, which takes a zipped index/coefficient list, whereas `add_col` takes two parallel lists.)
     5
     6While making this change, should also clarify the defaulting behavior of the `binary`, `continuous`, `integer` arguments of `add_variable`. The argument list is:
     7{{{
     8cpdef int add_variable(self, lower_bound=0.0, upper_bound=None, binary=False, continuous=False, integer=False, obj=0.0, name=None)
     9}}}
     10Whereas the documentation says:
     11        - `binary` - `True` if the variable is binary (default: `False`).
     12        - `continuous` - `True` if the variable is binary (default: ``True`).
     13        - `integer` - `True` if the variable is binary (default: `False`).
     14Probably the defaults in the arglist should be `None`, and defaulting to creating continuous variables should be explained separately.