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

Gurobi Support ! #11949

Closed
nathanncohen mannequin opened this issue Oct 24, 2011 · 13 comments
Closed

Gurobi Support ! #11949

nathanncohen mannequin opened this issue Oct 24, 2011 · 13 comments

Comments

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Oct 24, 2011

Here it is ! It cost me several nights along with my first evening in Belgium, but it works ! ;-)

This patch implements a new interface between Sage and Gurobi (http://www.gurobi.com)

YEAH !

(and now I'm going out ! :-D)

Nathann

APPLY:

CC: @malb @sagetrac-mvngu @dcoudert @sagetrac-lsampaio @ypfmde @novoselt @jasongrout

Component: linear programming

Author: Nathann Cohen

Reviewer: Peter Müller

Merged: sage-4.8.alpha2

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

@nathanncohen nathanncohen mannequin added this to the sage-4.8 milestone Oct 24, 2011
@nathanncohen nathanncohen mannequin self-assigned this Oct 24, 2011
@nathanncohen

This comment has been minimized.

@nathanncohen nathanncohen mannequin added the s: needs review label Oct 24, 2011
@jdemeyer
Copy link
Contributor

jdemeyer commented Nov 3, 2011

Milestone sage-4.7.3 deleted

@jdemeyer jdemeyer removed this from the sage-4.8 milestone Nov 3, 2011
@ypfmde
Copy link

ypfmde commented Nov 9, 2011

comment:5

I tried this patch in 4.8.alpha0 for many examples, for which I previously used the Coin-cbc solver or the standalone versions of scip and gurobi via the mps-export of the linear program. It seems to work fine with Gurobi versions 4.5 and 4.6. The docs are updated, and the thematic tutorial on linear programming explains well how to make gurobi visible to sage.

It's great that this powerful solver (for which one can get a free academic license) can now be used from inside Sage.

Peter Müller

@ypfmde ypfmde added this to the sage-4.8 milestone Nov 9, 2011
@nathanncohen
Copy link
Mannequin Author

nathanncohen mannequin commented Nov 9, 2011

comment:6

My GOD O_O

Gurobi in Sage. Thank you soooooooooooooo much for reviewing it :-)

Nathann

@jdemeyer
Copy link
Contributor

jdemeyer commented Nov 9, 2011

comment:7

You should change "optional - required GUROBI" to "optional - GUROBI"

@nathanncohen
Copy link
Mannequin Author

nathanncohen mannequin commented Nov 9, 2011

comment:8

Oopps.... Fixed ! :-)

Nathann

@jdemeyer
Copy link
Contributor

comment:9

There are some issues with the documentation:

dochtml.log:/mnt/usb1/scratch/jdemeyer/merger/sage-4.8.alpha2/devel/sage/doc/en/thematic_tutorials/linear_programming.rst:521: WARNING: Bullet list ends without a blank line; unexpected unindent.
dochtml.log:/mnt/usb1/scratch/jdemeyer/merger/sage-4.8.alpha2/devel/sage/doc/en/thematic_tutorials/linear_programming.rst:532: WARNING: Inline strong start-string without end-string.

@nathanncohen
Copy link
Mannequin Author

nathanncohen mannequin commented Nov 12, 2011

comment:10

Ahem... Updated ^^;

Nathann

@nathanncohen
Copy link
Mannequin Author

nathanncohen mannequin commented Nov 12, 2011

Attachment: trac_11949.patch.gz

@jdemeyer
Copy link
Contributor

comment:11

pmueller: please add your real name as Reviewer on this ticket and also add yourself to http://trac.sagemath.org/sage_trac/#AccountNamesMappedtoRealNames.

@ypfmde
Copy link

ypfmde commented Nov 14, 2011

Reviewer: Peter Mueller

@jdemeyer
Copy link
Contributor

Merged: sage-4.8.alpha2

@jdemeyer
Copy link
Contributor

Changed reviewer from Peter Mueller to Peter Müller

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

2 participants