Opened 10 years ago

Closed 10 years ago

#6602 closed enhancement (fixed)

[with spkg, positive review] GLPK for Sage

Reported by: ncohen Owned by: tbd
Priority: major Milestone: sage-4.1.1
Component: packages: optional Keywords:
Cc: wstein, mvngu Merged in: Sage 4.1.1.rc1
Authors: Nathann Cohen Reviewers: David Joyner
Report Upstream: Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Change History (14)

comment:1 Changed 10 years ago by ncohen

  • Description modified (diff)

comment:2 Changed 10 years ago by wdj

  • Summary changed from [with SPKG, need review] GLPK for SAGE to [with SPKG, positive review] GLPK for SAGE

I'm not sure what needs to be checked here. It installs fine on an amd64 ubuntu 9.04 machine and passes sage -testall.

Are there any tests I should run? I saw nothing on the url given in the ticket but I am not an OR person.

Positive review from me as far as I can tell, as an optional package.

comment:3 Changed 10 years ago by ncohen

I think that most of the tests of this spkg will be done in http://trac.sagemath.org/sage_trac/ticket/6502

I hope it will be possible to quickly include all of this into SAGE !! ( Oh, and this spkg is meant to be standard, not just optional !! )

comment:4 Changed 10 years ago by mvngu

  • Summary changed from [with SPKG, positive review] GLPK for SAGE to [with spkg, positive review] GLPK for SAGE

It's rather difficult to include this SPKG when there are (as yet) no functions in the Sage library to test its functionalities. Once #6502 gets positive review, this SPKG could then be merged in the Sage standard packages repository.

comment:5 Changed 10 years ago by mvngu

  • Summary changed from [with spkg, positive review] GLPK for SAGE to [with spkg, positive review] GLPK for Sage

We no longer use "SAGE". The days of that capitalization are over. Now use "Sage" instead.

comment:6 Changed 10 years ago by mhampton

  • Cc wstein mvngu added

I object to including this as standard just because of consistency concerns - I think this needs a vote on sage-devel.

comment:7 Changed 10 years ago by wdj

Note: I said "Positive review from me as far as I can tell, as an optional package. "

comment:8 Changed 10 years ago by mvngu

The proposal here is to merge the SPKG in the optional packages repository.

comment:9 Changed 10 years ago by ncohen

I forgot all about the voting process I immediately send a message on Sage-devel about it.

This package has to be included in the --standard-- package repository if we want Sage to have any native LP feature ( see #6502 ). Coin-or and Cplex are both GPL-uncompatible ;-)

comment:11 Changed 10 years ago by wdj

I could be wrong but I think trac is for optional packages. IIRC, once it is optional (ie, posted to http://www.sagemath.org/packages/optional/) then a public vote is carried out on sage-devel.

comment:12 Changed 10 years ago by ncohen

I did not know that !!!

I'm pretty new aboard, and the only thing I wrote for Sage was an interface for Cliquer, which has seemingly found a shortcut through all these steps ;-)

I may be in a hurry, but it is just because :

  • I am impatient to see this patch accepted
  • I have already written several graph functions waiting to be included in the Graph class that I will not post until MIP is included into Sage

Sorry again ! I'll try to be a bit more patient ;-)

comment:13 Changed 10 years ago by mvngu

  • Component changed from numerical to optional packages
  • Owner changed from jkantor to tbd

comment:14 Changed 10 years ago by mvngu

  • Authors set to Nathann Cohen
  • Merged in set to Sage 4.1.1.rc1
  • Milestone set to sage-4.1.1
  • Resolution set to fixed
  • Reviewers set to David Joyner
  • Status changed from new to closed

Merged in optional packages repository. The new optional package can be found here:

http://www.sagemath.org/packages/optional/glpk-4.38.spkg

Note: See TracTickets for help on using tickets.