Opened 13 years ago

Closed 13 years ago

#6602 closed enhancement (fixed)

[with spkg, positive review] GLPK for Sage

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

Status badges

Change History (14)

comment:1 Changed 13 years ago by Nathann Cohen

Description: modified (diff)

comment:2 Changed 13 years ago by David Joyner

Summary: [with SPKG, need review] GLPK for SAGE[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 13 years ago by Nathann Cohen

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 13 years ago by Minh Van Nguyen

Summary: [with SPKG, positive review] GLPK for SAGE[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 13 years ago by Minh Van Nguyen

Summary: [with spkg, positive review] GLPK for SAGE[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 13 years ago by mhampton

Cc: wstein Minh Van Nguyen 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 13 years ago by David Joyner

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

comment:8 Changed 13 years ago by Minh Van Nguyen

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

comment:9 Changed 13 years ago by Nathann Cohen

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 13 years ago by David Joyner

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 13 years ago by Nathann Cohen

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 13 years ago by Minh Van Nguyen

Component: numericaloptional packages
Owner: changed from jkantor to tbd

comment:14 Changed 13 years ago by Minh Van Nguyen

Authors: Nathann Cohen
Merged in: Sage 4.1.1.rc1
Milestone: sage-4.1.1
Resolution: fixed
Reviewers: David Joyner
Status: newclosed

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.