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

clean up of MIP interface #7574

Closed
malb opened this issue Dec 1, 2009 · 5 comments
Closed

clean up of MIP interface #7574

malb opened this issue Dec 1, 2009 · 5 comments

Comments

@malb
Copy link
Member

malb commented Dec 1, 2009

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.

Component: numerical

Reviewer: Frédéric Chapoton, Matthias Koeppe

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

@malb malb added this to the sage-5.11 milestone Dec 1, 2009
@malb

This comment has been minimized.

@jdemeyer jdemeyer modified the milestones: sage-5.11, sage-5.12 Aug 13, 2013
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.1, sage-6.2 Jan 30, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.2, sage-6.3 May 6, 2014
@sagetrac-vbraun-spam sagetrac-vbraun-spam mannequin modified the milestones: sage-6.3, sage-6.4 Aug 10, 2014
@mkoeppe
Copy link
Contributor

mkoeppe commented Jun 25, 2016

comment:6

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

@mkoeppe mkoeppe removed this from the sage-6.4 milestone Jun 25, 2016
@fchapoton
Copy link
Contributor

comment:7

ok

@fchapoton
Copy link
Contributor

Reviewer: Frédéric Chapoton, Matthias Koeppe

@embray
Copy link
Contributor

embray commented Aug 30, 2016

comment:8

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

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

5 participants