Skip to content

{bio}[foss/2016b] ExaBayes v1.5#7801

Merged
akesandgren merged 2 commits intoeasybuilders:developfrom
pescobar:20190304122915_new_pr_ExaBayes15
Mar 11, 2019
Merged

{bio}[foss/2016b] ExaBayes v1.5#7801
akesandgren merged 2 commits intoeasybuilders:developfrom
pescobar:20190304122915_new_pr_ExaBayes15

Conversation

@pescobar
Copy link
Copy Markdown
Member

@pescobar pescobar commented Mar 4, 2019

(created using eb --new-pr)

@pescobar pescobar added the new label Mar 4, 2019
sources = [SOURCELOWER_TAR_GZ]
checksums = ['e401f1b4645e67e8879d296807131d0ab79bba81a1cd5afea14d7c3838b095a2']

configopts = '--enable-mpi '
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.

If you have enable-mpi, shouldn't you use toolchainopts 'usempi': True too?

Copy link
Copy Markdown
Contributor

@akesandgren akesandgren Mar 8, 2019

Choose a reason for hiding this comment

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

As a matter of fact, yes you do need 'usempi': True, so it will find the correct MPICXX command.

To clarify a bit, it works with foss since for foss mpicxx is the right command to use, but for intel it is mpiicpc and not mpicxx.

sources = [SOURCELOWER_TAR_GZ]
checksums = ['e401f1b4645e67e8879d296807131d0ab79bba81a1cd5afea14d7c3838b095a2']

configopts = '--enable-mpi '
Copy link
Copy Markdown
Contributor

@akesandgren akesandgren Mar 8, 2019

Choose a reason for hiding this comment

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

As a matter of fact, yes you do need 'usempi': True, so it will find the correct MPICXX command.

To clarify a bit, it works with foss since for foss mpicxx is the right command to use, but for intel it is mpiicpc and not mpicxx.

@boegel boegel added this to the 3.x milestone Mar 9, 2019
@pescobar
Copy link
Copy Markdown
Member Author

I have added toolchainopts = {'usempi': True}

I was not aware that this was mandatory for intel toolchains. Thanks for the clarification

@akesandgren
Copy link
Copy Markdown
Contributor

Without using usempi: True, exabayes finds mpicxx, which for IntelMPI is a wrapper around g++ and uses g++ compiled mpi libs.
That's quite different from using mpiicpc.

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 21 out of 21 (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/fc9117f2a9560b06aad33305e962a69b for a full test report.

@akesandgren
Copy link
Copy Markdown
Contributor

Going in, thanks @pescobar!

@akesandgren akesandgren merged commit 193994d into easybuilders:develop Mar 11, 2019
@boegel boegel modified the milestones: 3.x, next release (3.8.2) Mar 11, 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.

3 participants