#13518: Additions for sensitivity analysis in glpk_backend
-------------------------------------------------------------------------------+
Reporter: christiankuper
| Owner: jason, jkantor
Type: enhancement
| Status: needs_review
Priority: major
| Milestone: sage-5.6
Component: numerical
| Resolution:
Keywords: out-of-kilter, sensitivity analysis, lp, linear programming
| Work issues:
Report Upstream: N/A
| Reviewers:
Authors: Christian Kuper
| Merged in:
Dependencies:
| Stopgaps:
-------------------------------------------------------------------------------+
Comment (by christiankuper):
Replying to [comment:15 ncohen]:
> * There's a double '-' before your name at the beginning of the file
> * Thank you for the lp->mps fix.. Could you also change the filename,
which currently ends with .lp ?
> * It feels a bit weird to have a print_* method that does not print
anything but writes the output to a file. Of course that's how the GLPK
function is named, so it makes sense... What would you think of modifying
it just a bit so that it can also "PRINT" the information that is being
written to a file ? Something like, by default, write the information to a
file and print it, unless the user explicitely wants the information to be
written to a file, not displayed ? I can do that in another patch if you
like the idea and don't want to spend time on it.
> * What about just having ``get_row_dual`` and not
``get_constraint_shadow_price``, which is just an alias to it anyway ?
Besides, get_row_dual is how the GLPK function is named, and this method's
docstring can mention what a shadow price is at the same time ? And the
same goes for cols.
> * In the documentation of get_backend, why do you put ``instance``
inside of quotes ? We usually do this for python expressions.
Hello Nathann,
I have implmented your suggested changes and updated it for Sage 5.4.
Looking forward to your feedback!
Christian
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/13518#comment:18>
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.
