Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#7804 closed defect (fixed)

Move mip_coin and mip_glpk to Sage

Reported by: ncohen Owned by: AlexGhitza
Priority: major Milestone: sage-4.3.1
Component: numerical Keywords:
Cc: mvngu Merged in: sage-4.3.1.alpha2
Authors: Nathann Cohen Reviewers: Robert Miller
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description (last modified by ncohen)

Hello !!

This patches moves the files mip_coin and mip_glpk to Sage. They are currently included in the packages CBC and GLPK and are a lot harder to work on because of this.

This patch copies them in sage/numerical/ and adds several lines to modules_list so that they will only be compiled if the corresponding packages are installed.

For the moment, the copies of these files included in the packages will not be removed, in order to preserve backward-compatibility : the users of earlier versions of Sage will then be able to keep using the same packages.

*Only the changes to file modules_list need to be reviewed -- mip_coin and mip_glpk are copies of what is included in the spkg and have already been checked ! This should be a short review :-) *

Attachments (1)

trac_7804.patch (24.3 KB) - added by ncohen 10 years ago.

Download all attachments as: .zip

Change History (13)

comment:1 Changed 10 years ago by ncohen

  • Status changed from new to needs_review

comment:2 Changed 10 years ago by ncohen

  • Component changed from algebra to numerical
  • Milestone set to sage-4.3.1

comment:3 Changed 10 years ago by ncohen

  • Description modified (diff)

comment:4 Changed 10 years ago by ncohen

  • Summary changed from Move mipCoin and mipGlpk to Sage to Move mip_coin and mip_glpk to Sage

comment:5 Changed 10 years ago by was

I was curious and skimmed this patch for ~ 3 minutes and it "looks good" (not a positive review -- I didn't test it yet).

Changed 10 years ago by ncohen

comment:6 Changed 10 years ago by ncohen

I just modified it so that it is now independent from the huge changes going on in graph.py. Would it be possible to have this merged to the next release ? It would let me write another speed-up patch now that solve_glpk and solve_cbc are available ;-)

Nathann

comment:7 Changed 10 years ago by rlm

  • Authors set to Nathann Cohen
  • Merged in set to 4.3.1.alpha2
  • Resolution set to fixed
  • Reviewers set to Robert Miller
  • Status changed from needs_review to closed

positive review

comment:8 Changed 10 years ago by ncohen

Yessssssssssss !! :-) Thanks !!!!

Nathann

comment:9 Changed 10 years ago by mvngu

  • Merged in changed from 4.3.1.alpha2 to sage-4.3.1.alpha2

comment:10 Changed 10 years ago by jhpalmieri

How did this get a positive review when the new pyx files have no doctests?

comment:11 Changed 10 years ago by rlm

That's a good point-- I suppose my review was a bit rushed.

Nathann,

Do you want to make a separate ticket for making some doctests here, or would you prefer it if I just pull the patch back out?

comment:12 Changed 10 years ago by ncohen

see #7925 :-)

But they will be way harder to write with the spkg GLPK and cbc broken :-/

Nathann

Note: See TracTickets for help on using tickets.