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

GLPK for Sage #6602

Closed
nathanncohen mannequin opened this issue Jul 23, 2009 · 16 comments
Closed

GLPK for Sage #6602

nathanncohen mannequin opened this issue Jul 23, 2009 · 16 comments

Comments

@nathanncohen
Copy link
Mannequin

nathanncohen mannequin commented Jul 23, 2009

GLPK ( http://www.gnu.org/software/glpk/ ) is a Linear Program solver from GNU. It can also solve Mixed Integer Programs, and is needed for :
#6502

More informations on :
http://groups.google.com/group/sage-devel/browse_thread/thread/9da47e06bcdfc49f

SPKG available there : http://www-sop.inria.fr/members/Nathann.Cohen/glpk-4.38.spkg

CC: @williamstein @sagetrac-mvngu

Component: packages: optional

Author: Nathann Cohen

Reviewer: David Joyner

Merged: Sage 4.1.1.rc1

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

@nathanncohen

This comment has been minimized.

@wdjoyner
Copy link

comment:2

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.

@wdjoyner wdjoyner changed the title [with SPKG, need review] GLPK for SAGE [with SPKG] GLPK for SAGE Jul 25, 2009
@nathanncohen
Copy link
Mannequin Author

nathanncohen mannequin commented Jul 26, 2009

comment:3

I think that most of the tests of this spkg will be done in #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 !! )

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Jul 26, 2009

comment:4

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.

@sagetrac-mvngu sagetrac-mvngu mannequin changed the title [with SPKG] GLPK for SAGE GLPK for SAGE Jul 26, 2009
@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Jul 26, 2009

comment:5

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

@sagetrac-mvngu sagetrac-mvngu mannequin changed the title GLPK for SAGE GLPK for Sage Jul 26, 2009
@sagetrac-mhampton
Copy link
Mannequin

sagetrac-mhampton mannequin commented Jul 31, 2009

comment:6

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

@wdjoyner
Copy link

comment:7

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

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Jul 31, 2009

comment:8

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

@nathanncohen
Copy link
Mannequin Author

nathanncohen mannequin commented Aug 1, 2009

comment:9

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 ;-)

@nathanncohen
Copy link
Mannequin Author

nathanncohen mannequin commented Aug 1, 2009

comment:10

The vote is going on there :
http://groups.google.com/group/sage-devel/browse_thread/thread/fed15c54478e8d5

@wdjoyner
Copy link

wdjoyner commented Aug 1, 2009

comment:11

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.

@nathanncohen
Copy link
Mannequin Author

nathanncohen mannequin commented Aug 1, 2009

comment:12

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 ;-)

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Aug 4, 2009

Reviewer: David Joyner

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Aug 4, 2009

Merged: Sage 4.1.1.rc1

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Aug 4, 2009

Author: Nathann Cohen

@sagetrac-mvngu
Copy link
Mannequin

sagetrac-mvngu mannequin commented Aug 4, 2009

comment:14

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

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

@sagetrac-mvngu sagetrac-mvngu mannequin removed the s: positive review label Aug 4, 2009
@sagetrac-mvngu sagetrac-mvngu mannequin closed this as completed Aug 4, 2009
@sagetrac-mvngu sagetrac-mvngu mannequin added this to the sage-4.1.1 milestone Aug 4, 2009
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

1 participant