#20559: InteractiveLPProblem, dictionaries: add_constraint / add_row methods
-------------------------------------+-------------------------------------
Reporter: mkoeppe | Owner:
Type: enhancement | Status: needs_work
Priority: major | Milestone: sage-7.2
Component: linear | Resolution:
programming | Merged in:
Keywords: | Reviewers: Andrey Novoseltsev
Authors: Peijun Xiao | Work issues: rebasing
Report Upstream: N/A | Commit:
Branch: | f73fc57d12596752224ad3b5ff456574efab178d
u/pjxiao/interactivelpproblem__dictionaries__add_constraint___add_row_methods|
Stopgaps:
Dependencies: #20500 |
-------------------------------------+-------------------------------------
Comment (by pjxiao):
I fixed the problems your comments 1, 2, 4, and 5.
Replying to [comment:10 novoselt]:
> 1. "a 1 by n matrix of the new constraint coefficients" is a bit weird,
especially since examples show using a list rather than a matrix. I'd
write just "coefficients of the new constraint"
> 2. `new_row` does not reflect very well the meaning, how about
`coefficients`?..
> ...
> 4. When constructing a new problem, as much of the old one as possible
should be preserved. The current version does not keep even the type of
the problem!!!
> 5. It should not be necessary to explicitly name a new slack variable.
Please take a look at the last three commits listed in comment:15.
Thanks!
--
Ticket URL: <http://trac.sagemath.org/ticket/20559#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 unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/sage-trac.
For more options, visit https://groups.google.com/d/optout.
