Changes between Version 3 and Version 4 of Ticket #20311, comment 4


Ignore:
Timestamp:
03/30/16 03:59:13 (4 years ago)
Author:
mkoeppe
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #20311, comment 4

    v3 v4  
    1 I think the constant term should appear in the latex representation of the problem when nonzero.
    2 And it also needs to be passed through in `standard_form`.
    3 
    4 And I think there should be a doctest that illustrates how the constant term interacts with `is_negative`.
    5 
    61The standard-form transformation works well with #20296. Thanks very much! (I'll update the branch of #20296 when this ticket is merged.)
    72
    8 Passing `slack_variables` as a keyword argument to `standard_form` is a bit of an awkward interface, though, because `standard_form` may split equations and so the user would have to anticipate this. Perhaps it would be better to provide a list of "row_names" instead and modify that somehow when equations are split.
     3Some issues:
     4
     5- I think the constant term should appear in the latex representation of the problem when nonzero.
     6
     7- And it also needs to be passed through to `InteractiveLPProblemStandardForm` in `standard_form`. I have added a (failing) doctest.
     8
     9- And then `InteractiveLPProblemStandardForm` needs to pass it on to the dictionaries...
     10
     11- I think there should be a doctest that illustrates how the constant term interacts with `is_negative`.
     12
     13- Passing `slack_variables` as a keyword argument to `standard_form` is a bit of an awkward interface, though, because `standard_form` may split equations and so the user would have to anticipate this. Perhaps it would be better to provide a list of "row_names" instead and modify that somehow when equations are split.