Ticket #7805 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

MixedIntegerLinearProgram.show should use the constraints' names

Reported by: ncohen Owned by: jkantor
Priority: major Milestone: sage-4.4
Component: numerical Keywords:
Cc: Work issues:
Report Upstream: N/A Reviewers: David Joyner
Authors: Nathann Cohen Merged in: sage-4.4.alpha0
Dependencies: Stopgaps:

Description

As the title says, this function currently doesn't use them ;-)

Nathann

Attachments

trac_7805.patch Download (1.9 KB) - added by ncohen 3 years ago.

Change History

comment:1 Changed 3 years ago by ncohen

  • Status changed from new to needs_review
  • Summary changed from MixedIntegerLinearProgram.show should use the variables' and constraints' names to MixedIntegerLinearProgram.show should use the constraints' names

Changed 3 years ago by ncohen

comment:2 Changed 3 years ago by wdj

This seems to pass all tests on sage 4.3.5 on a 10.6.2 mac (the only failure was that of the fresh copy of 4.3.5).

Like #8639, this seems like a very very minor change. As far as I can seen, positive review.

comment:3 Changed 3 years ago by wdj

  • Status changed from needs_review to positive_review

comment:4 Changed 3 years ago by jhpalmieri

  • Status changed from positive_review to closed
  • Reviewers set to David Joyner
  • Resolution set to fixed
  • Merged in set to sage-4.4.alpha0
  • Authors set to Nathann Cohen

Merged "trac_7805.patch" into 4.4.alpha0

Note: See TracTickets for help on using tickets.