-
-
Notifications
You must be signed in to change notification settings - Fork 491
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
LPBackendDictionary - a debugging view of a MIP backend connected to interactive_simplex_method #18804
Comments
This comment has been minimized.
This comment has been minimized.
Branch: u/zwang/temp |
Author: Aedi Wang |
This comment has been minimized.
This comment has been minimized.
Commit: |
New commits:
|
Branch pushed to git repo; I updated commit sha1. Last 10 new commits:
|
Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:
|
Changed branch from u/zwang/temp to u/mkoeppe/temp |
Branch pushed to git repo; I updated commit sha1. New commits:
|
Changed branch from u/mkoeppe/temp to u/zwang/temp |
New commits:
|
Changed branch from u/zwang/temp to u/mkoeppe/temp |
Branch pushed to git repo; I updated commit sha1. New commits:
|
comment:18
Merged current version of branch on #20203 |
comment:19
This is rebased on top of 9.1.beta1 |
Branch pushed to git repo; I updated commit sha1. New commits:
|
Work Issues: Add methods add_row, eq so that TestSuite passes |
comment:25
Development continues on https://github.com/mkoeppe/sage-numerical-interactive-mip The present ticket can be closed. |
Reviewer: Dima Pasechnik |
This is a follow-up on #18734.
This ticket creates:
LPBackendDictionary
(inheriting fromLPAbstractDictionary
), would avoid going throughInteractiveLP
and rather be tied directly to the MILP backend, using tableau data query functions such asgetBInvArow
etc.(It would be in a separate file perhaps
interactive_backend_dictionary.py
, so as not to add clutter to the didactical implementation ininteractive_simplex_method.py
.)#18732 added most of the missing functions for retrieving the data of the dictionary (for the GLPK backend) such as
getBInvARow
etc. The remaining functions are provided by #19472.#18763 and #19471 do the same for the COIN backend.
See also: #18733
Doctests could then test that:
LPBackendDictionary
(in this ticket) and theLPDictionary
(created in Construct an interactive_simplex_method.LPDictionary from a MixedIntegerLinearProgram #18734) are the same.CC: @nathanncohen @yuan-zhou @uduse @novoselt @dimpase
Component: numerical
Work Issues: Add methods add_row, eq so that TestSuite passes
Author: Aedi Wang
Branch/Commit: u/mkoeppe/zeyi-wang--lpbackenddictionary-trac18804--lpcleandictionary-trac20203--rebased-to-9.1.beeta1 @
06ec022
Reviewer: Dima Pasechnik
Issue created by migration from https://trac.sagemath.org/ticket/18804
The text was updated successfully, but these errors were encountered: