Skip to content

{bio}[gompi/2019a] HyPhy v2.5.1#9405

Merged
akesandgren merged 2 commits intoeasybuilders:developfrom
omula:20191129103322_new_pr_HyPhy251
Dec 3, 2019
Merged

{bio}[gompi/2019a] HyPhy v2.5.1#9405
akesandgren merged 2 commits intoeasybuilders:developfrom
omula:20191129103322_new_pr_HyPhy251

Conversation

@omula
Copy link
Copy Markdown
Contributor

@omula omula commented Nov 29, 2019

(created using eb --new-pr)

using techniques in phylogenetics, molecular evolution, and machine learning"""

toolchain = {'name': 'gompi', 'version': '2019a'}
toolchainopts = {'usempi': True}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Since this code can do OpenMP you should add 'openmp': True to toolchainsopts.


buildopts = [
'hyphy',
'HYPHYMPI',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

buildopts should preferably (for style) be below dependencies, before sanity_check_paths.

@akesandgren akesandgren added the new label Dec 3, 2019
@omula
Copy link
Copy Markdown
Contributor Author

omula commented Dec 3, 2019

@akesandgren thanks for your suggestions. I already made the changes.

Copy link
Copy Markdown
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
Copy Markdown
Contributor

Test report by @akesandgren
SUCCESS
Build succeeded for 1 out of 1 (1 easyconfigs in this PR)
b-an03.hpc2n.umu.se - Linux ubuntu 16.04, Intel(R) Xeon(R) CPU E5-2690 v4 @ 2.60GHz, Python 2.7.12
See https://gist.github.com/a4db51cf463e8a26da4e5a47baa16dea for a full test report.

@akesandgren
Copy link
Copy Markdown
Contributor

Going in, thanks @omula!

@akesandgren akesandgren merged commit 8be7e38 into easybuilders:develop Dec 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants