Skip to content
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

Make _repr_ of MILP more informative #23681

Closed
jdemeyer opened this issue Aug 23, 2017 · 13 comments
Closed

Make _repr_ of MILP more informative #23681

jdemeyer opened this issue Aug 23, 2017 · 13 comments

Comments

@jdemeyer
Copy link

I am solving LP problems where I only care about feasibility and would prefer to see

Linear Program (no objective, 128 variables, 632 constraints)

instead of

Mixed Integer Program  ( maximization, 128 variables, 632 constraints )

Component: user interface

Keywords: days88, IMA coding sprints

Author: Jeroen Demeyer

Branch/Commit: b3e7322

Reviewer: Travis Scrimshaw

Issue created by migration from https://trac.sagemath.org/ticket/23681

@jdemeyer jdemeyer added this to the sage-8.1 milestone Aug 23, 2017
@jdemeyer
Copy link
Author

@jdemeyer
Copy link
Author

New commits:

aa5fa1dMake `_repr_` of MILP more informative

@jdemeyer

This comment has been minimized.

@jdemeyer
Copy link
Author

Commit: aa5fa1d

@tscrim
Copy link
Collaborator

tscrim commented Aug 23, 2017

Changed keywords from none to days88, IMA coding sprint

@tscrim
Copy link
Collaborator

tscrim commented Aug 23, 2017

Reviewer: Travis Scrimshaw

@tscrim
Copy link
Collaborator

tscrim commented Aug 23, 2017

comment:3

LGTM.

@tscrim
Copy link
Collaborator

tscrim commented Aug 23, 2017

Changed keywords from days88, IMA coding sprint to days88, IMA coding sprints

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 24, 2017

Changed commit from aa5fa1d to b3e7322

@sagetrac-git
Copy link
Mannequin

sagetrac-git mannequin commented Aug 24, 2017

Branch pushed to git repo; I updated commit sha1 and set ticket back to needs_review. New commits:

b3e7322Use "constant objective" for non-zero constant objectives

@jdemeyer
Copy link
Author

comment:6

I just realized that a constant objective function is essentially the same as a zero objective function. So it should also be special-cased. Please review the last commit.

@tscrim
Copy link
Collaborator

tscrim commented Aug 24, 2017

comment:7

Either way is fine with me.

@vbraun
Copy link
Member

vbraun commented Sep 10, 2017

Changed branch from u/jdemeyer/make__repr__of_milp_more_informative to b3e7322

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants