Skip to content

Conversation

@lucamar
Copy link
Contributor

@lucamar lucamar commented Feb 25, 2019

I have inserted the PrgEnv-gnu-nompi on Kesch in order to build DGEMM using OpenBLAS:

  • we don't need to link to the MPI library as we only use a single task to run
  • only this programming environment loads the module gmvolf (which loads the module openblas), while PrgEnv-gnu only loads gmvapich2/17.02_cuda_8.0_gdr (see config/cscs.py)

@lucamar lucamar self-assigned this Feb 25, 2019
@pep8speaks
Copy link

pep8speaks commented Feb 25, 2019

Hello @lucamar, Thank you for updating!

Cheers! There are no PEP8 issues in this Pull Request!Do see the ReFrame Coding Style Guide

Comment last updated on March 01, 2019 at 16:25 Hours UTC

@lucamar
Copy link
Contributor Author

lucamar commented Feb 25, 2019

Note that Dom is currently offline, therefore the CI will not manage to run.

@teojgo teojgo changed the title Extend DGEMM test to build with OpenBLAS [test] Extend DGEMM test to build with OpenBLAS Feb 26, 2019
Change if test as suggested

Co-Authored-By: lucamar <luca.marsella@cscs.ch>
@lucamar
Copy link
Contributor Author

lucamar commented Feb 28, 2019

Thanks for checking @vkarak! I have updated the if test as you suggested.

@vkarak vkarak added this to the Upcoming Sprint milestone Feb 28, 2019
@vkarak
Copy link
Contributor

vkarak commented Feb 28, 2019

@jenkins-cscs retry daint

@lucamar
Copy link
Contributor Author

lucamar commented Mar 1, 2019

It looks like the error was the following in reframeCI on Piz Daint:

E           sbatch: error: Batch job submission failed: Socket timed out on send/recv operation

@vkarak
Copy link
Contributor

vkarak commented Mar 1, 2019

@lucamar No problem, we will retry it. This is a Slurm problem.

@codecov-io
Copy link

Codecov Report

Merging #691 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #691   +/-   ##
=======================================
  Coverage   91.83%   91.83%           
=======================================
  Files          77       77           
  Lines        9494     9494           
=======================================
  Hits         8719     8719           
  Misses        775      775

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f5ff16c...0029413. Read the comment docs.

@codecov-io
Copy link

codecov-io commented Mar 1, 2019

Codecov Report

Merging #691 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #691   +/-   ##
=======================================
  Coverage   91.83%   91.83%           
=======================================
  Files          77       77           
  Lines        9494     9494           
=======================================
  Hits         8719     8719           
  Misses        775      775

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f5ff16c...0029413. Read the comment docs.

@vkarak vkarak merged commit 5583743 into reframe-hpc:master Mar 1, 2019
@lucamar lucamar deleted the dgemm-openblas branch March 1, 2019 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants