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.

Sign up for GitHub

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

Jump to bottom

also include csSAM extension in R 3.5.1 easyconfigs #7423

Merged
merged 1 commit into from
Jan 15, 2019
Merged

also include csSAM extension in R 3.5.1 easyconfigs #7423

merged 1 commit into from
Jan 15, 2019

Conversation

boegel
Copy link
Member

@boegel boegel commented Jan 15, 2019

No description provided.

@boegel boegel added this to the 3.8.1 milestone Jan 15, 2019
@boegel
Copy link
Member Author

boegel commented Jan 15, 2019

Test report by @boegel
SUCCESS
Build succeeded for 2 out of 2 (2 easyconfigs in this PR)
node2704.swalot.os - Linux centos linux 7.6.1810, Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz, Python 2.7.5
See https://gist.github.com/5d77987df5d98a9975903989e1b85492 for a full test report.

@boegel
Copy link
Member Author

boegel commented Jan 15, 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/0557547e088cf4ac3ecfc623ff78eb2d for a full test report.

@boegel
Copy link
Member Author

boegel commented Jan 15, 2019

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

@boegel boegel mentioned this pull request Jan 15, 2019
Merged
akesandgren
akesandgren approved these changes Jan 15, 2019
View reviewed changes
Copy link
Contributor

@akesandgren akesandgren left a comment

Choose a reason for hiding this comment

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

LGTM

@akesandgren
Copy link
Contributor

Why did that second test report fail? (I.e. why the mpi errors)

@boegel
Copy link
Member Author

boegel commented Jan 15, 2019

@akesandgren That's because I was running that test report in an interactive job in Slurm, which causes problems with MPI commands sometimes. That's a local issue we need to figure out.

@akesandgren
Copy link
Contributor

Going in, thanks @boegel!

@akesandgren akesandgren merged commit 51b7043 into easybuilders:develop Jan 15, 2019
@boegel boegel deleted the R351_csSAM branch January 15, 2019 21:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@akesandgren akesandgren akesandgren approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
3.8.1
Development

Successfully merging this pull request may close these issues.
2 participants
@boegel @akesandgren