Add SCIP mixed-integer programming solver #17242

hedtke · 2023-04-26T06:35:02Z

Specify library name and version: scip/8.0.3

I am not the author of this lib, but we use it at my company. This PR resolves #14180

I've read the contributing guidelines.
I've used a recent Conan client version close to the currently deployed.
I've tried at least one configuration locally with the conan-center hook activated.

hedtke · 2023-05-09T14:07:07Z

HI @prince-chrismc, this is a follow-up to #16017. I need help here. I do not know why the pipeline fails. I cannot see the reason.

prince-chrismc · 2023-05-09T17:30:38Z

It had already passed, I am not sure why you triggered it again?

prince-chrismc · 2023-05-09T17:31:37Z

Known bug, race condition when it merges PRs sometimes build fail

hudson.remoting.ProxyException: Assertion failed: 

assert parentCommits[1] == masterCommit

	at org.codehaus.groovy.runtime.InvokerHelper.assertFailed(InvokerHelper.java:404)

prince-chrismc · 2023-05-09T17:32:48Z

This is a big PR so I suspect it will take time for someone to review it. you'll need some extra patience on this one :)

CLAassistant · 2023-05-18T13:38:16Z

All committers have signed the CLA.

Co-authored-by: Uilian Ries <uilianries@gmail.com>

… are aligned Co-authored-by: Uilian Ries <uilianries@gmail.com>

Signed-off-by: Uilian Ries <uilianries@gmail.com>

uilianries · 2023-06-06T10:18:33Z

@hedtke Please, consider https://github.com/hedtke/conan-center-index/pull/2. That PR has several fixes for this PR. Regards.

Prepare SCIP recipe for CCI PR

hedtke · 2023-06-06T12:37:10Z

Please, consider hedtke#2. That PR has several fixes for this PR. Regards.

@uilianries Thank you very much for your help!

hedtke · 2023-07-05T07:39:51Z

I would like to publish other software that depends on this. Please have a look into it. I think everything is fine. Failed pipelines are due to a technical issue on the server side, I think.

uilianries · 2023-07-05T07:45:47Z

@hedtke I just started a new CI build internal. Let's hope it does not fail again.

uilianries · 2023-07-05T08:15:41Z

recipes/scip/all/conanfile.py

+        tc.variables["SHARED"] = self.options.shared
+        tc.variables["READLINE"] = False  # required for interactive stuff
+        tc.variables["GMP"] = self.options.with_gmp
+        tc.variables["BOOST"] = self.options.with_boost


I didn't find BOOST variable in 8.0.3, how do you activate/deactivate it?

% grep BOOST -r ./INSTALL.md:| `BOOST_ROOT` | `<custom/path/to/.../boost>` | -- | hint for location of boost | ./src/cppad/core/testvector.hpp:$code CPPAD_BOOSTVECTOR$$ will be true. ./src/cppad/core/testvector.hpp:# if CPPAD_BOOSTVECTOR ./src/cppad/core/test_vector.hpp:of $code CPPAD_BOOSTVECTOR$$: ./src/cppad/core/test_vector.hpp:# if CPPAD_BOOSTVECTOR ./src/cppad/core/test_vector.hpp:# undef CPPAD_BOOSTVECTOR ./src/cppad/core/test_vector.hpp:# define CPPAD_BOOSTVECTOR 0 ./src/cppad/core/test_vector.hpp:$code CPPAD_BOOSTVECTOR$$ is true ./src/cppad/core/test_vector.hpp:# if CPPAD_BOOSTVECTOR ./src/cppad/core/undef.hpp:# undef CPPAD_BOOSTVECTOR ./src/cppad/configure.hpp:\def CPPAD_BOOSTVECTOR ./src/cppad/configure.hpp:# define CPPAD_BOOSTVECTOR 0

It's clear that Soplex needs boost, but it seems like libscip does not use it. Could you show where boost is used and activated?

Indeed, just an include path depends on this setting, not the package itself. I dropped the option.

conan-center-bot · 2023-07-05T11:25:56Z

Conan v1 pipeline ✔️

All green in build 22 (7a0b70961a495172044e3e96aee1f75d1e039254):

scip/8.0.3@:
All packages built successfully! (All logs)

Conan v2 pipeline ✔️

Note: Conan v2 builds may be required once they are on the v2 ready list

All green in build 17 (7a0b70961a495172044e3e96aee1f75d1e039254):

scip/8.0.3@:
All packages built successfully! (All logs)

* Add SCIP mixed-integer solver * Do not export bliss in shared mode * Add option with_sym * Bliss does not support libc++ * Use normpath on Windows * Replace backslash in Windows paths * Bliss is built in another directory under Windows * Add package_type * Add zlib as dependency Co-authored-by: Uilian Ries <uilianries@gmail.com> * Check that the options between this package and its dependency soplex are aligned Co-authored-by: Uilian Ries <uilianries@gmail.com> * Remove v1 test package * Always depend on boost and gmp, independently of the parameters * improve scip recipe Signed-off-by: Uilian Ries <uilianries@gmail.com> * use bliss 0.77 Signed-off-by: Uilian Ries <uilianries@gmail.com> * Drop option with_boost --------- Signed-off-by: Uilian Ries <uilianries@gmail.com> Co-authored-by: Uilian Ries <uilianries@gmail.com>

conan-center-bot added the Failed label Apr 26, 2023

This comment has been minimized.

Sign in to view

Add SCIP mixed-integer solver

ba7f066

This comment has been minimized.

Sign in to view

Do not export bliss in shared mode

13d5c69

This comment has been minimized.

Sign in to view

Add option with_sym

6147ab7

This comment has been minimized.

Sign in to view

Bliss does not support libc++

49eff3c

This comment has been minimized.

Sign in to view

Use normpath on Windows

b698ed9

This comment has been minimized.

Sign in to view

hedtke added 3 commits April 29, 2023 22:43

Replace backslash in Windows paths

9ce88ab

Bliss is built in another directory under Windows

ab8889e

Add package_type

cd00653

conan-center-bot removed the Failed label Apr 29, 2023

This comment has been minimized.

Sign in to view

prince-chrismc mentioned this pull request Apr 29, 2023

Pull Requests Ready to be Reviewed prince-chrismc/conan-center-index-pending-review#1

Open

hedtke closed this May 9, 2023

hedtke reopened this May 9, 2023

prince-chrismc closed this May 9, 2023

prince-chrismc reopened this May 9, 2023

This comment has been minimized.

Sign in to view

conan-center-bot requested a review from franramirez688 May 22, 2023 12:00

hedtke and others added 3 commits June 6, 2023 10:17

Add zlib as dependency

b99626d

Co-authored-by: Uilian Ries <uilianries@gmail.com>

Check that the options between this package and its dependency soplex…

149cfb3

… are aligned Co-authored-by: Uilian Ries <uilianries@gmail.com>

Remove v1 test package

70ac9c6

This comment has been minimized.

Sign in to view

hedtke and others added 3 commits June 6, 2023 11:21

Always depend on boost and gmp, independently of the parameters

d83a876

improve scip recipe

739fbb0

Signed-off-by: Uilian Ries <uilianries@gmail.com>

use bliss 0.77

cfb9fed

Signed-off-by: Uilian Ries <uilianries@gmail.com>

This comment has been minimized.

Sign in to view

Merge pull request #2 from uilianries/hedtke-scip-mip

7398a6f

Prepare SCIP recipe for CCI PR

This comment has been minimized.

Sign in to view

conan-center-bot added the Unexpected Error label Jun 22, 2023

This comment has been minimized.

Sign in to view

hedtke closed this Jun 26, 2023

hedtke reopened this Jun 26, 2023

prince-chrismc mentioned this pull request Jun 26, 2023

Change approvals to include reviewer tier prince-chrismc/conan-center-index-pending-review#130

Merged

uilianries self-assigned this Jul 5, 2023

uilianries reviewed Jul 5, 2023

View reviewed changes

conan-center-bot removed the Unexpected Error label Jul 5, 2023

This comment has been minimized.

Sign in to view

Drop option with_boost

7a0b709

uilianries approved these changes Jul 5, 2023

View reviewed changes

SSE4 approved these changes Jul 8, 2023

View reviewed changes

conan-center-bot merged commit b70a905 into conan-io:master Jul 14, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add SCIP mixed-integer programming solver #17242

Add SCIP mixed-integer programming solver #17242

hedtke commented Apr 26, 2023

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

hedtke commented May 9, 2023

prince-chrismc commented May 9, 2023

prince-chrismc commented May 9, 2023

prince-chrismc commented May 9, 2023 •

edited

Loading

This comment has been minimized.

CLAassistant commented May 18, 2023 •

edited

Loading

This comment has been minimized.

uilianries commented Jun 6, 2023

This comment has been minimized.

This comment has been minimized.

hedtke commented Jun 6, 2023

This comment has been minimized.

This comment has been minimized.

hedtke commented Jul 5, 2023

uilianries commented Jul 5, 2023

uilianries Jul 5, 2023

hedtke Jul 5, 2023

This comment has been minimized.

conan-center-bot commented Jul 5, 2023

Add SCIP mixed-integer programming solver #17242

Add SCIP mixed-integer programming solver #17242

Conversation

hedtke commented Apr 26, 2023

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

hedtke commented May 9, 2023

prince-chrismc commented May 9, 2023

prince-chrismc commented May 9, 2023

prince-chrismc commented May 9, 2023 • edited Loading

This comment has been minimized.

CLAassistant commented May 18, 2023 • edited Loading

This comment has been minimized.

uilianries commented Jun 6, 2023

This comment has been minimized.

This comment has been minimized.

hedtke commented Jun 6, 2023

This comment has been minimized.

This comment has been minimized.

hedtke commented Jul 5, 2023

uilianries commented Jul 5, 2023

uilianries Jul 5, 2023

Choose a reason for hiding this comment

hedtke Jul 5, 2023

Choose a reason for hiding this comment

This comment has been minimized.

conan-center-bot commented Jul 5, 2023

Conan v1 pipeline ✔️

Conan v2 pipeline ✔️

prince-chrismc commented May 9, 2023 •

edited

Loading

CLAassistant commented May 18, 2023 •

edited

Loading