Skip to content

{tools}[intel/2017.02] HPL v2.2#4248

Merged
boegel merged 2 commits intoeasybuilders:developfrom
boegel:20170303095415_new_pr_HPL22
Mar 3, 2017
Merged

{tools}[intel/2017.02] HPL v2.2#4248
boegel merged 2 commits intoeasybuilders:developfrom
boegel:20170303095415_new_pr_HPL22

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Mar 3, 2017 •
edited
Loading

(created using eb --new-pr)

requires easybuilders/easybuild-easyblocks#1117 to install included imkl and impi versions...

@boegel boegel added this to the 3.1.1 milestone Mar 3, 2017
ocaisa
ocaisa approved these changes Mar 3, 2017
View reviewed changes
Copy link
Copy Markdown
Member

@ocaisa ocaisa left a comment

Choose a reason for hiding this comment

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

Update to reflect license changes

Comment thread easybuild/easyconfigs/i/imkl/imkl-2017.2.174-iimpi-2017.02-GCC-6.3.0-2.27.eb Outdated

dontcreateinstalldir = 'True'

license_file = HOME + '/licenses/intel/license.lic'
Copy link
Copy Markdown
Member

@ocaisa ocaisa Mar 3, 2017 •
edited
Loading

Choose a reason for hiding this comment

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

This could now be removed (it is ignored anyway)

Copy link
Copy Markdown
Member Author

@boegel boegel Mar 3, 2017

Choose a reason for hiding this comment

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

ah, good point, thanks

Comment thread easybuild/easyconfigs/i/impi/impi-2017.2.174-iccifort-2017.2.174-GCC-6.3.0-2.27.eb Outdated

components = ['intel-mpi', 'intel-psxe', 'intel-imb']

license_file = HOME + '/licenses/intel/license.lic'
Copy link
Copy Markdown
Member

@ocaisa ocaisa Mar 3, 2017

Choose a reason for hiding this comment

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

Can also be removed

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 3, 2017

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

@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 3, 2017

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

@boegel boegel mentioned this pull request Mar 3, 2017
Merged
@boegel
Copy link
Copy Markdown
Member Author

boegel commented Mar 3, 2017

tested on top of easybuilders/easybuild-easyblocks#1117 which is merged now, so good to go, thanks for the review @ocaisa!

@boegel boegel merged commit f90d450 into easybuilders:develop Mar 3, 2017
@boegel boegel deleted the 20170303095415_new_pr_HPL22 branch March 3, 2017 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ocaisa ocaisa ocaisa approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

3.1.1

Development

Successfully merging this pull request may close these issues.

2 participants

@boegel @ocaisa