Opened 5 years ago

Closed 5 years ago

#23681 closed enhancement (fixed)

Make _repr_ of MILP more informative

Reported by: jdemeyer Owned by:
Priority: major Milestone: sage-8.1
Component: user interface Keywords: days88, IMA coding sprints
Cc: Merged in:
Authors: Jeroen Demeyer Reviewers: Travis Scrimshaw
Report Upstream: N/A Work issues:
Branch: b3e7322 (Commits, GitHub, GitLab) Commit: b3e73220fae152e78f31e4eca2022d910c463432
Dependencies: Stopgaps:

Status badges

Description (last modified by jdemeyer)

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 )

Change History (8)

comment:1 Changed 5 years ago by jdemeyer

  • Branch set to u/jdemeyer/make__repr__of_milp_more_informative

comment:2 Changed 5 years ago by jdemeyer

  • Commit set to aa5fa1daf5445a97ec6f1f1192cd97709c8149e0
  • Description modified (diff)
  • Status changed from new to needs_review

New commits:

​aa5fa1dMake _repr_ of MILP more informative

comment:3 Changed 5 years ago by tscrim

  • Keywords days88 IMA coding sprint added
  • Reviewers set to Travis Scrimshaw
  • Status changed from needs_review to positive_review

LGTM.

comment:4 Changed 5 years ago by tscrim

  • Keywords sprints added; sprint removed

comment:5 Changed 5 years ago by git

  • Commit changed from aa5fa1daf5445a97ec6f1f1192cd97709c8149e0 to b3e73220fae152e78f31e4eca2022d910c463432
  • Status changed from positive_review to needs_review

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

comment:6 Changed 5 years ago by jdemeyer

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.

comment:7 Changed 5 years ago by tscrim

  • Status changed from needs_review to positive_review

Either way is fine with me.

comment:8 Changed 5 years ago by vbraun

  • Branch changed from u/jdemeyer/make__repr__of_milp_more_informative to b3e73220fae152e78f31e4eca2022d910c463432
  • Resolution set to fixed
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.