Package CyLP

[mkoeppe]

https://github.com/coin-or/CyLP

Needed for running cvxpy (#31962) with a CBC backend.

CC: @sheerluck @tkralphs @seblabbe @dcoudert @dimpase

Component: packages: optional

Author: Matthias Koeppe

Branch/Commit: 8b27e69

Reviewer: Travis Scrimshaw

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

[mkoeppe]

Branch: u/mkoeppe/package_cylp

[mkoeppe]

comment:2

Is building with CYLP_USE_CYTHON=1 still supported / ever tested?
Fails for me without diagnostics

[cylp-0.91.4]   running build_ext
[cylp-0.91.4]   cythoning cylp/cy/CyClpPrimalColumnPivotBase.pyx to cylp/cy/CyClpPrimalColumnPivotBase.cpp
[cylp-0.91.4]   error: /Users/mkoeppe/s/sage/sage-rebasing/worktree-gcc11/local/var/lib/sage/venv-python3.10/var/tmp/sage/build/cylp-0.91.4/src/cylp/cy/CyClpPrimalColumnPivotBase.pyx
[cylp-0.91.4]   error: subprocess-exited-with-error

---

New commits:

3edafec

build/pkgs/cylp: New

[mkoeppe]

Commit: 3edafec

[mkoeppe]

comment:3

Without CYLP_USE_CYTHON, running into coin-or/CyLP#148 with Python 3.10.

[sagetrac-git]

Changed commit from 3edafec to 8b27e69

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

8b27e69

build/pkgs/cylp: Add megapatch 'Re-generating with Cython 0.29.27 for Python 3.10 compatibility', do not use CYLP_USE_CYTHON

[mkoeppe]

Author: Matthias Koeppe

[mkoeppe]

comment:5

This works for me on macOS with Python 3.10 and CBC 2.10.7 installed from homebrew. https://github.com/coin-or/CyLP#modeling-example works as expected.

[mkoeppe]

comment:8

Also builds OK with tox -e docker-ubuntu-trusty-maximal -- cylp (this uses our cbc spkg, currently still at 2.9.4)

[mkoeppe]

comment:9

Also builds OK with tox -e docker-ubuntu-bionic-maximal -- cylp (using system coinor-cbc (2.9.9+repack1-1))

[mkoeppe]

comment:10

I'll not run the full portability suite now. I think this is good enough for a new optional package.

[tscrim]

comment:11

That is a pretty massive patch. Although I guess the time for them for the next release is probably a bit away, so I guess it is necessary.

This builds and works fine for me.

[tscrim]

Reviewer: Travis Scrimshaw

[mkoeppe]

comment:13

Thanks!

[vbraun]

Changed branch from u/mkoeppe/package_cylp to 8b27e69