Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#11949 closed enhancement (fixed)

Gurobi Support !

Reported by: ncohen Owned by: ncohen
Priority: major Milestone: sage-4.8
Component: linear programming Keywords:
Cc: malb, mvngu, dcoudert, lsampaio, pmueller, novoselt, jason Merged in: sage-4.8.alpha2
Authors: Nathann Cohen Reviewers: Peter Müller
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by ncohen)

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:

Attachments (1)

trac_11949.patch (58.7 KB) - added by ncohen 8 years ago.

Download all attachments as: .zip

Change History (16)

comment:1 Changed 8 years ago by ncohen

  • Description modified (diff)
  • Status changed from new to needs_review

comment:2 Changed 8 years ago by novoselt

  • Cc novoselt added

comment:3 Changed 8 years ago by jason

  • Cc jason added

comment:4 Changed 8 years ago by jdemeyer

  • Milestone sage-4.7.3 deleted

Milestone sage-4.7.3 deleted

comment:5 Changed 8 years ago by pmueller

  • Milestone set to sage-4.8
  • Status changed from needs_review to positive_review

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

comment:6 Changed 8 years ago by ncohen

My GOD O_O

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

Nathann

comment:7 Changed 8 years ago by jdemeyer

  • Status changed from positive_review to needs_work

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

comment:8 Changed 8 years ago by ncohen

  • Status changed from needs_work to positive_review

Oopps.... Fixed ! :-)

Nathann

comment:9 Changed 8 years ago by jdemeyer

  • Status changed from positive_review to needs_work

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.

comment:10 Changed 8 years ago by ncohen

  • Status changed from needs_work to positive_review

Ahem... Updated ^^;

Nathann

Changed 8 years ago by ncohen

comment:11 Changed 8 years ago by jdemeyer

  • Status changed from positive_review to needs_info

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

comment:12 Changed 8 years ago by pmueller

  • Reviewers set to Peter Mueller
  • Status changed from needs_info to needs_review

comment:13 Changed 8 years ago by pmueller

  • Status changed from needs_review to positive_review

comment:14 Changed 8 years ago by jdemeyer

  • Merged in set to sage-4.8.alpha2
  • Resolution set to fixed
  • Status changed from positive_review to closed

comment:15 Changed 8 years ago by jdemeyer

  • Reviewers changed from Peter Mueller to Peter Müller
Note: See TracTickets for help on using tickets.