Skip to content

{math}[foss/2019a] OR-Tools v7.1 w/ Python 3.7.2#8364

Merged
boegel merged 7 commits intoeasybuilders:developfrom
wpoely86:20190521195334_new_pr_OR-Tools71
Jul 1, 2019
Merged

{math}[foss/2019a] OR-Tools v7.1 w/ Python 3.7.2#8364
boegel merged 7 commits intoeasybuilders:developfrom
wpoely86:20190521195334_new_pr_OR-Tools71

Conversation

@wpoely86
Copy link
Copy Markdown
Member

@wpoely86 wpoely86 commented May 21, 2019

(created using eb --new-pr)

@wpoely86 wpoely86 added the new label May 21, 2019
@wpoely86 wpoely86 requested a review from boegel May 21, 2019 17:55
@wpoely86
Copy link
Copy Markdown
Member Author

wpoely86 commented May 21, 2019

Test report by @wpoely86
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
login2.cerberus.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) Gold 6126 CPU @ 2.60GHz, Python 2.7.5
See https://gist.github.com/67e67dcfe43ddca89609ee7c68440966 for a full test report.

@boegel boegel added this to the 3.x milestone May 21, 2019
boegel
boegel requested changes May 21, 2019
View reviewed changes
Comment thread easybuild/easyconfigs/o/OR-Tools/OR-Tools-7.1-fix-makefiles.patch Outdated
Comment thread easybuild/easyconfigs/o/OR-Tools/OR-Tools-7.1-fix-makefiles.patch Outdated
Comment thread easybuild/easyconfigs/o/OR-Tools/OR-Tools-7.1-fix-makefiles.patch Outdated
Comment thread easybuild/easyconfigs/o/OR-Tools/OR-Tools-7.1-fix-makefiles.patch
Comment thread easybuild/easyconfigs/o/OR-Tools/OR-Tools-7.1-foss-2019a-Python-3.7.2.eb Outdated
Comment thread easybuild/easyconfigs/o/OR-Tools/OR-Tools-7.1-foss-2019a-Python-3.7.2.eb
Comment thread easybuild/easyconfigs/o/OR-Tools/OR-Tools-7.1-foss-2019a-Python-3.7.2.eb Outdated
@wpoely86
Copy link
Copy Markdown
Member Author

wpoely86 commented May 21, 2019

Test report by @wpoely86
SUCCESS
Build succeeded for 2 out of 2 (1 easyconfigs in this PR)
login2.cerberus.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) Gold 6126 CPU @ 2.60GHz, Python 2.7.5
See https://gist.github.com/a281fd726955e8b1a6fae9e0c4180c77 for a full test report.

@wpoely86
Copy link
Copy Markdown
Member Author

wpoely86 commented May 22, 2019

Test report by @wpoely86
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
login2.cerberus.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) Gold 6126 CPU @ 2.60GHz, Python 2.7.5
See https://gist.github.com/dbd2ce51d4c83ac7f6d39280ee008f4d for a full test report.

@wpoely86
Copy link
Copy Markdown
Member Author

wpoely86 commented May 22, 2019

@boegel remarks fixed

boegel
boegel requested changes May 22, 2019
View reviewed changes
Comment thread easybuild/easyconfigs/o/OR-Tools/OR-Tools-7.1-fix-makefiles.patch Outdated
@wpoely86 wpoely86 requested a review from boegel May 23, 2019 13:00
@boegel
Copy link
Copy Markdown
Member

boegel commented May 24, 2019

Test report by @boegel
FAILED
Build succeeded for 1 out of 2 (1 easyconfigs in this PR)
node3115.skitty.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 2.7.5
See https://gist.github.com/cab992a065c80f2885f45998c4e289da for a full test report.

@wpoely86
Copy link
Copy Markdown
Member Author

wpoely86 commented May 24, 2019

@boegel it fails because you don't have a license for Gurobi:

RuntimeError: 10009, Request denied: license not valid for Gurobi version 8

@boegel
Copy link
Copy Markdown
Member

boegel commented May 25, 2019

@wpoely86 Yes, I'm working on getting that fixed.

Maybe someone else who has access to Gurobi licenses can test this in the mean time?

@boegel boegel modified the milestones: 3.x, next release (3.9.3) Jul 1, 2019
@boegel
Copy link
Copy Markdown
Member

boegel commented Jul 1, 2019

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node3109.skitty.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) Gold 6140 CPU @ 2.30GHz, Python 2.7.5
See https://gist.github.com/122f9e9f993cbb2a53c70b94ddf1db3b for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jul 1, 2019

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2081.delcatty.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) CPU E5-2670 0 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/1eeb07d8e6b3fe68ee605df710fba23d for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jul 1, 2019

Test report by @boegel
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
node2419.golett.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) CPU E5-2680 v3 @ 2.50GHz, Python 2.7.5
See https://gist.github.com/8d02d0291bd0135407083b2ddef1aa6b for a full test report.

@boegel
Copy link
Copy Markdown
Member

boegel commented Jul 1, 2019

Going in, thanks @wpoely86!

@boegel boegel merged commit 668316b into easybuilders:develop Jul 1, 2019
@wpoely86 wpoely86 deleted the 20190521195334_new_pr_OR-Tools71 branch July 1, 2019 13:11
@boegel
Copy link
Copy Markdown
Member

boegel commented Jul 1, 2019

@wpoely86 Merging this broke the tests because we now have both Java/11 (for R-3.6.0-foss-2019a.eb and Unicycler-0.4.7-foss-2019a-Python-3.7.2.eb) and Java/8 for OR-Tools...

Can we bump the Java dep for OR-Tools to Java/11 to fix that?

@boegel boegel mentioned this pull request Jul 1, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@boegel boegel boegel approved these changes

Assignees

No one assigned

Labels

new

Projects

None yet

Milestone

3.9.3

Development

Successfully merging this pull request may close these issues.

2 participants

@wpoely86 @boegel