Skip to content

add sanity check command to PSI4 1.2.1 easyconfigs to ensure that 'import psi4' works#8393

Merged
smoors merged 1 commit intoeasybuilders:developfrom
boegel:PSI4121_import_psi4
May 30, 2019
Merged

add sanity check command to PSI4 1.2.1 easyconfigs to ensure that 'import psi4' works#8393
smoors merged 1 commit intoeasybuilders:developfrom
boegel:PSI4121_import_psi4

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented May 28, 2019

also adding the import psi4 check here for consistency (cfr. #8362)

@boegel boegel added this to the next release (3.9.2) milestone May 28, 2019
@boegel boegel requested a review from smoors May 28, 2019 18:03
@boegel
Copy link
Copy Markdown
Member Author

boegel commented May 28, 2019

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node3100.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/c7a1b80f8d185f0e3cf32fe7d767787d for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented May 28, 2019

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2498.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/6135b1050cec88f8f9ff24157e67e80f for a full test report.

@boegel
Copy link
Copy Markdown
Member Author

boegel commented May 29, 2019

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2016.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/10e4a9c7fa93c9757634a1a06abf2604 for a full test report.

@smoors
Copy link
Copy Markdown
Contributor

smoors commented May 29, 2019

Test report by @smoors
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
nic29 - Linux centos linux 7.5.1804, Intel(R) Xeon(R) Gold 6148 CPU @ 2.40GHz, Python 2.7.5
See https://gist.github.com/445a16742b96979d91800b35b36b9de7 for a full test report.

Copy link
Copy Markdown
Contributor

@smoors smoors left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@smoors
Copy link
Copy Markdown
Contributor

smoors commented May 30, 2019

Going in, thanks @boegel!

@smoors smoors merged commit 0c30640 into easybuilders:develop May 30, 2019
@boegel boegel deleted the PSI4121_import_psi4 branch June 3, 2019 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants