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.

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

intel/2017b toolchain definition (REVIEW) #4618

Merged
merged 7 commits into from Aug 31, 2017

Conversation

boegel
Copy link
Member

@boegel boegel commented May 19, 2017

(created using eb --new-pr)

requires easybuilders/easybuild-framework#2215 since SHA256 checksums are used

requires #4794 for GCC 6.4.0

…017.4.196-GCC-7.1.0-2.28.eb, ifort-2017.4.196-GCC-7.1.0-2.28.eb, impi-2017.3.196-iccifort-2017.4.196-GCC-7.1.0-2.28.eb, imkl-2017.3.196-iimpi-2017.05-GCC-7.1.0-2.28.eb, iccifort-2017.4.196-GCC-7.1.0-2.28.eb, iimpi-2017.05-GCC-7.1.0-2.28.eb
@boegel boegel added this to the 3.3.0 milestone May 19, 2017
@boegel
Copy link
Member Author

boegel commented May 19, 2017

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

@boegel
Copy link
Member Author

boegel commented May 19, 2017

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

@boegel
Copy link
Member Author

boegel commented Jun 21, 2017

Reply from Intel support on use of Intel compilers 2017 update 4 on top of GCC 7.1.0:

In the release note, we specify that we only have support for GCC 6.

https://software.intel.com/en-us/articles/intel-c-compiler-170-for-linux-release-notes-for-intel-parallel-studio-xe-2017

Even the 2018 update will only support GCC 6.3

You might end up having some issues of compatibility.

So I guess we'll stick to GCC 6.3.0...

Thoughts @wpoely86, @akesandgren, @damianam, @ocaisa?

@wpoely86
Copy link
Member

If that is what Intel says, we stick to it? No reason to go all adventurous, especially as GCC 7 has no killer features (or not as far as I know)

@akesandgren
Copy link
Contributor

akesandgren commented Jun 23, 2017 via email

@boegel boegel modified the milestones: 3.3.0, 3.3.1 Jun 25, 2017
@boegel boegel changed the title {toolchain} intel/2017.05 + HPL 2.2 testcase intel/2017b (WIP) Jun 26, 2017
@boegel boegel changed the title intel/2017b (WIP) intel/2017b toolchain definition (WIP) Jun 26, 2017
@boegel boegel closed this Jun 26, 2017
@boegel boegel deleted the 20170519145158_new_pr_HPL22 branch June 26, 2017 17:42
@boegel boegel restored the 20170519145158_new_pr_HPL22 branch June 26, 2017 17:42
@boegel
Copy link
Member Author

boegel commented Jun 26, 2017

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

@boegel boegel reopened this Jun 26, 2017
@easybuilders easybuilders deleted a comment from boegelbot Jun 26, 2017
@boegel boegel closed this Jun 26, 2017
@boegel boegel reopened this Jun 26, 2017
toolchain = {'name': 'dummy', 'version': 'dummy'}

compver = '2017.4.196'
gccver = '6.3.0'
Copy link
Member Author

Choose a reason for hiding this comment

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

@boegel
Copy link
Member Author

boegel commented Jun 27, 2017

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

@boegel boegel modified the milestones: 3.3.1, 3.4.0 Jul 9, 2017
toolchainopts = {'usempi': True}

sources = [SOURCELOWER_TAR_GZ]
source_urls = ['http://www.netlib.org/benchmark/%(namelower)s']
Copy link
Member

Choose a reason for hiding this comment

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

Add a checksum here.

@@ -0,0 +1,33 @@
# This is an easyconfig file for EasyBuild, see http://hpcugent.github.io/easybuild
Copy link
Member

Choose a reason for hiding this comment

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

Old URL?

@@ -0,0 +1,19 @@
# This is an easyconfig file for EasyBuild, see http://hpcugent.github.io/easybuild
Copy link
Member

Choose a reason for hiding this comment

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

Old URL again

@@ -0,0 +1,36 @@
# This is an easyconfig file for EasyBuild, see http://hpcugent.github.io/easybuild
Copy link
Member

Choose a reason for hiding this comment

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

URL

@@ -0,0 +1,20 @@
# This is an easyconfig file for EasyBuild, see http://hpcugent.github.io/easybuild
Copy link
Member

Choose a reason for hiding this comment

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

URL

@@ -0,0 +1,36 @@
# This is an easyconfig file for EasyBuild, see http://hpcugent.github.io/easybuild
Copy link
Member

Choose a reason for hiding this comment

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

URL again

@@ -0,0 +1,33 @@
# This is an easyconfig file for EasyBuild, see http://hpcugent.github.io/easybuild
Copy link
Member

Choose a reason for hiding this comment

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

URL fix required


homepage = 'http://software.intel.com/en-us/intel-cluster-toolkit-compiler/'
description = """Intel Cluster Toolkit Compiler Edition provides Intel C/C++ and Fortran compilers, Intel MPI &
Intel MKL."""
Copy link
Member

Choose a reason for hiding this comment

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

Wouldn't it be better to have a proper toolchain description here?

Copy link
Member Author

Choose a reason for hiding this comment

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

define proper?

Copy link
Member

Choose a reason for hiding this comment

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

Well, that just reads like some Intel promotional material.

Compare it with the foss text.

Copy link
Member Author

Choose a reason for hiding this comment

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

Something like this then?

Compiler toolchain including Intel compilers, Intel MPI and Intel Math Kernel Library (MKL).

Copy link
Member

Choose a reason for hiding this comment

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

Yes, that's much better.

JackPerdue added a commit to JackPerdue/easybuild-easyconfigs that referenced this pull request Aug 19, 2017
JackPerdue added a commit to JackPerdue/easybuild-easyconfigs that referenced this pull request Aug 19, 2017
@boegel
Copy link
Member Author

boegel commented Aug 28, 2017

@akesandgren You need to make sure a license is available for these...

@akesandgren
Copy link
Contributor

Yes i know, i was just a bit too tired this morning. New build is already running.
And stop reading comments i have already deleted :-) :-)

@akesandgren
Copy link
Contributor

Test report by @akesandgren
FAILED
Build succeeded for 19 out of 22 (8 easyconfigs in this PR)
janson.hpc2n.umu.se - Linux ubuntu 16.04, Intel(R) Core(TM) i7-5600U CPU @ 2.60GHz, Python 2.7.12
See https://gist.github.com/60aea5ef110bc56ed38a00ada12f6e8e for a full test report.

@boegel
Copy link
Member Author

boegel commented Aug 28, 2017

@akesandgren Hmm, this one looks trickier, some missing system dependency for Intel MKL? :-/

/usr/include/features.h(367): catastrophic error: cannot open source file "sys/cdefs.h"
  #  include <sys/cdefs.h>

@akesandgren
Copy link
Contributor

No, it's a feature of Debian/Ubuntu that Intel doesn't handle. I have a patch to the icc easyconfig that fixes this. I'll prep a PR for it.

@boegel boegel changed the title intel/2017b toolchain definition (WIP) intel/2017b toolchain definition (REVIEW) Aug 29, 2017
@easybuilders easybuilders deleted a comment from boegelbot Aug 30, 2017
@verdurin
Copy link
Member

Ignore that failed test...

@easybuilders easybuilders deleted a comment from verdurin Aug 31, 2017
@vanzod
Copy link
Member

vanzod commented Aug 31, 2017

Test report by @vanzod
SUCCESS
Build succeeded for 8 out of 8 (8 easyconfigs in this PR)
cermis - Linux debian 8.8, Intel(R) Xeon(R) CPU E5620 @ 2.40GHz, Python 2.7.9
See https://gist.github.com/9cc4210b65e4103268202e3b2d68c86c for a full test report.

@vanzod
Copy link
Member

vanzod commented Aug 31, 2017

Going in, thanks @boegel!

@vanzod vanzod merged commit d6e3a53 into easybuilders:develop Aug 31, 2017
@boegel boegel deleted the 20170519145158_new_pr_HPL22 branch September 1, 2017 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants