Skip to content

{dev} [dummy] intel/2017.00 (REVIEW)#3543

Merged
boegel merged 6 commits intoeasybuilders:developfrom
wpoely86:intel2017
Oct 13, 2016
Merged

{dev} [dummy] intel/2017.00 (REVIEW)#3543
boegel merged 6 commits intoeasybuilders:developfrom
wpoely86:intel2017

Conversation

@wpoely86
Copy link
Copy Markdown
Member

@wpoely86 wpoely86 commented Sep 7, 2016

Installer currently doesn't work (invalid components)

edit: requires easybuilders/easybuild-easyblocks#1012

@boegel
Copy link
Copy Markdown
Member

boegel commented Sep 7, 2016

@wpoely86 please mark this (WIP)

@wpoely86 wpoely86 changed the title {dev} [dummy] intel/2017 stuff {dev} [dummy] intel/2017 stuff (WIP) Sep 7, 2016
@bartoldeman
Copy link
Copy Markdown
Contributor

The quotes needs to be removed in silent.cfg. I'll submit an easyblock PR.

@wpoely86
Copy link
Copy Markdown
Member Author

@bartoldeman already on it. Intel support told us it's changed. But there is more, sanity check fails with missing symlink and some other minor details.

@bartoldeman
Copy link
Copy Markdown
Contributor

Thanks. icc installation was ok here (with the quote removal in intelbase.py) but will see further.

@jesgp
Copy link
Copy Markdown

jesgp commented Sep 26, 2016

Hi All,
Any updates on this issue? I'm facing exactly what you're mentioning. Any timeline where you guys believe intel 2017 will be fully working?
Thanks!

@bartoldeman
Copy link
Copy Markdown
Contributor

Actually after the quote removal the whole thing appeared to have installed fine for me. I did have to change versions at a few places though:
imkl: 2017.0.098
impi: 2017.0.098
instead of 2017.0.98.

And the following small patches:
for impi:

@@ -1,7 +1,7 @@
 # This is an easyconfig file for EasyBuild, see http://hpcugent.github.io/easybuild

 name = 'impi'
-version = '2017.0.98'
+version = '2017.0.098'

 homepage = 'http://software.intel.com/en-us/intel-mpi-library/'
 description = """The Intel(R) MPI Library for Linux* OS is a multi-fabric message
@@ -10,7 +10,7 @@

 toolchain = {'name': 'iccifort', 'version': '2017.0.35-GCC-5.4.0-2.26'}

-sources = ['l_mpi_p_%(version)s.tgz']
+sources = ['l_mpi_%(version)s.tgz']

 checksums = ['fc123875773816b7084a91e419d54d20']

for imkl:

@@ -1,7 +1,7 @@
 # This is an easyconfig file for EasyBuild, see http://hpcugent.github.io/easybuild

 name = 'imkl'
-version = '2017.0.98'
+version = '2017.0.098'

 homepage = 'http://software.intel.com/en-us/intel-mkl/'
 description = """Intel Math Kernel Library is a library of highly optimized,
@@ -22,11 +22,13 @@

 postinstallcmds = [
     # extract the examples
-    'tar xvzf %(installdir)s/mkl/examples/examples_cluster.tgz -C %(installdir)s/mkl/examples/',
+    'tar xvzf %(installdir)s/mkl/examples/examples_cluster_c.tgz -C %(installdir)s/mkl/examples/',
+    'tar xvzf %(installdir)s/mkl/examples/examples_cluster_f.tgz -C %(installdir)s/mkl/examples/',
     'tar xvzf %(installdir)s/mkl/examples/examples_core_c.tgz -C %(installdir)s/mkl/examples/',
     'tar xvzf %(installdir)s/mkl/examples/examples_core_f.tgz -C %(installdir)s/mkl/examples/',
     'tar xvzf %(installdir)s/mkl/examples/examples_f95.tgz -C %(installdir)s/mkl/examples/',
-    'tar xvzf %(installdir)s/mkl/examples/examples_mic.tgz -C %(installdir)s/mkl/examples/'
+    'tar xvzf %(installdir)s/mkl/examples/examples_mic_c.tgz -C %(installdir)s/mkl/examples/',
+    'tar xvzf %(installdir)s/mkl/examples/examples_mic_f.tgz -C %(installdir)s/mkl/examples/'
 ]

 modextravars = {

@akesandgren
Copy link
Copy Markdown
Contributor

@wpoely86 Did you create an easyblock PR for this? I can't find any. If not i have the required patches including sanity check fix for mkl ready.

@boegel boegel added this to the 3.0 milestone Oct 12, 2016
@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 12, 2016

@damianam you mentioned running into some trouble when trying to install these, because of missing required changes to the easyblock?

@akesandgren I believe you mentioned you already succeeded in installing these, can you share what you had to tweak to make it work?

@akesandgren
Copy link
Copy Markdown
Contributor

Yes i have easyblock changes available for 2017, i was just waiting for @wpoely86 to push his PR for it to see if they matched.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 12, 2016

@akesandgren it may be quicker to open your PR, and ping @wpoely86 to review it ;)

@akesandgren
Copy link
Copy Markdown
Contributor

akesandgren commented Oct 12, 2016

Added easyblock PR easybuilders/easybuild-easyblocks#1012

# This is an easyconfig file for EasyBuild, see http://hpcugent.github.io/easybuild

name = 'impi'
version = '2017.0.98'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@wpoely86 upstream version number for latest impi is 2017.0.098?

cc @akesandgren

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.

Yes intel changed all version numbers to match the compiler.

boegel and others added 3 commits October 13, 2016 13:08
fix icc/ifort/impi/imkl 2017.* versions, add intel/2017.00 definition + HPL 2.2
@wpoely86
Copy link
Copy Markdown
Member Author

Test report by @wpoely86
SUCCESS
Build succeeded for 8 out of 8 (8 easyconfigs in this PR)
build2901.metapod.os - Linux centos linux 7.2.1511, Intel(R) Xeon(R) CPU X5675 @ 3.07GHz, Python 2.7.5
See https://gist.github.com/bd3e50af4ce8b77b4a88892740e1249f for a full test report.

'tar xvzf %(installdir)s/mkl/examples/examples_core_f.tgz -C %(installdir)s/mkl/examples/',
'tar xvzf %(installdir)s/mkl/examples/examples_f95.tgz -C %(installdir)s/mkl/examples/',
'tar xvzf %(installdir)s/mkl/examples/examples_mic_f.tgz -C %(installdir)s/mkl/examples/',
'tar xvzf %(installdir)s/mkl/examples/examples_mic_c.tgz -C %(installdir)s/mkl/examples/',
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@wpoely86 shouldn't we let the imkl easyblock deal with this instead?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think you already know my opinion about moving things to the easyblock? 😉

I don't mind but I also don't really care in this case.

@jhein32
Copy link
Copy Markdown
Collaborator

jhein32 commented Oct 13, 2016

Is there any energy left regarding other components of the Intel cluster edition? Things I am interested in are e.g. vtune, advisor inspector and also they make a bit of noise about their python (yet another python install).

I am asking to avoid duplicated effort.

@bartoldeman
Copy link
Copy Markdown
Contributor

I'll create a PR for VTune. I have the easyconfig sitting here.

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 13, 2016

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 13, 2016

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

@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 13, 2016

@wpoely86 please update PR title?

lgtm, we can handle the imkl examples in the easyblock later, if we care enough ;)

@wpoely86 wpoely86 changed the title {dev} [dummy] intel/2017 stuff (WIP) {dev} [dummy] intel/2017.00 (REVIEW) Oct 13, 2016
@boegel
Copy link
Copy Markdown
Member

boegel commented Oct 13, 2016

Going in, thanks @wpoely86!

@boegel boegel merged commit c319724 into easybuilders:develop Oct 13, 2016
@wpoely86 wpoely86 deleted the intel2017 branch October 13, 2016 14:48
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.

6 participants