Opened 3 years ago

#20664 new enhancement

MixedIntegerLinearProgram/MIPVariable API cleanup

Reported by: mkoeppe Owned by:
Priority: major Milestone: sage-7.3
Component: numerical Keywords: lp
Cc: dimpase, vdelecroix, jdemeyer Merged in:
Authors: Reviewers:
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: #20462 Stopgaps:

Description

Follow-up on #20462:

  • Why does a MIPVariable only have set_max, but no get_max?
  • Why does MixedIntegerLinearProgram.set_max delegate to a MIPVariable method of the same name, when other MixedIntegerLinearProgram methods such as set_integer etc. just do everything directly, and no corresponding MIPVariable methods exist?
  • Actually these things should properly be referred to as lower_bound and upper_bound rather than min and max. Some deprecation fun to be had.
  • Why is there no way to query the values of the components of the _default_mipvariable?

Change History (0)

Note: See TracTickets for help on using tickets.