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

package Osi #23680

Open
videlec opened this issue Aug 22, 2017 · 14 comments
Open

package Osi #23680

videlec opened this issue Aug 22, 2017 · 14 comments

Comments

@videlec
Copy link
Contributor

videlec commented Aug 22, 2017

Osi (Open Solver Interface) is a generic C++ interface to several (floating point) linear optimizers. It would be easier for Sage to use it. However, as cbc is, Osi is released under Eclise license. So before pushing for dual licensing, just make it an optional package.

tarball: https://www.coin-or.org/download/source/Osi/Osi-0.107.9.tgz

CC: @mkoeppe @jplab

Component: numerical

Keywords: days88, IMA coding sprints

Author: Vincent Delecroix

Branch/Commit: u/vdelecroix/23680 @ 886ee13

Reviewer: Thierry Monteil

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

@videlec videlec added this to the sage-8.1 milestone Aug 22, 2017
@videlec

This comment has been minimized.

@videlec
Copy link
Contributor Author

videlec commented Aug 22, 2017

Changed keywords from none to days88, IMA coding sprints

@videlec videlec changed the title package OSI package Osi Aug 22, 2017
@videlec
Copy link
Contributor Author

videlec commented Aug 22, 2017

Branch: u/vdelecroix/23679

@videlec
Copy link
Contributor Author

videlec commented Aug 22, 2017

New commits:

9820a0723679: additive_order / multiplicative_order

@videlec
Copy link
Contributor Author

videlec commented Aug 22, 2017

Author: Vincent Delecroix

@videlec
Copy link
Contributor Author

videlec commented Aug 22, 2017

Commit: 9820a07

@videlec

This comment has been minimized.

@videlec
Copy link
Contributor Author

videlec commented Aug 22, 2017

Changed commit from 9820a07 to 886ee13

@videlec
Copy link
Contributor Author

videlec commented Aug 22, 2017

Changed branch from u/vdelecroix/23679 to u/vdelecroix/23680

@videlec
Copy link
Contributor Author

videlec commented Aug 22, 2017

New commits:

886ee1323680: package Osi

@mkoeppe
Copy link
Contributor

mkoeppe commented Aug 22, 2017

comment:4

see also #18862

@videlec
Copy link
Contributor Author

videlec commented Aug 22, 2017

comment:5

Replying to @mkoeppe:

see also #18862

Indeed... at least I have a branch ;-)

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented Aug 24, 2017

Reviewer: Thierry Monteil

@sagetrac-tmonteil
Copy link
Mannequin

sagetrac-tmonteil mannequin commented Aug 24, 2017

comment:6

The ticket is not yet in needs_review, but here are two points anyway:

  • you should remove any reference to FLINT in spkg-install (probably copy/pasted) (i see two occurrences)
  • you should provide an spkg-check script since the Makefile has a test rule.

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

2 participants