Opened 10 years ago

Closed 3 years ago

#7574 closed enhancement (wontfix)

clean up of MIP interface

Reported by: malb Owned by: jkantor
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: numerical Keywords: lp
Cc: Merged in:
Authors: Reviewers: Frédéric Chapoton, Matthias Koeppe
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by malb)

There are a few issues with the MIP code:

  • max and min are built-in core functions in Python and shouldn't be used as variable names
  • id shouldn't be used as a variable name
  • I don't think we should have try: foo except: bar blocks without a specific except ThisandThatError?.

Change History (8)

comment:1 Changed 10 years ago by malb

  • Description modified (diff)

comment:2 Changed 6 years ago by jdemeyer

  • Milestone changed from sage-5.11 to sage-5.12

comment:3 Changed 6 years ago by vbraun_spam

  • Milestone changed from sage-6.1 to sage-6.2

comment:4 Changed 5 years ago by vbraun_spam

  • Milestone changed from sage-6.2 to sage-6.3

comment:5 Changed 5 years ago by vbraun_spam

  • Milestone changed from sage-6.3 to sage-6.4

comment:6 Changed 3 years ago by mkoeppe

  • Cc ncohen removed
  • Keywords lp added
  • Milestone changed from sage-6.4 to sage-duplicate/invalid/wontfix
  • Status changed from new to needs_review

This is outdated. The first issue is duplicated in #20664. Can be closed.

comment:7 Changed 3 years ago by chapoton

  • Reviewers set to Frédéric Chapoton, Matthias Koeppe
  • Status changed from needs_review to positive_review

ok

comment:8 Changed 3 years ago by embray

  • Resolution set to wontfix
  • Status changed from positive_review to closed

Determined to be invalid/duplicate/wontfix (closing as "wontfix" as a catch-all resolution).

Note: See TracTickets for help on using tickets.