update trilinos integration scripts #335
Conversation
Add script to support Pascal GPU testing
| TRILINOS_UPDATE_BRANCH=$2 | ||
| TRILINOS_PRISTINE_BRANCH=$3 | ||
| KOKKOS_UPDATE_BRANCH=$4 | ||
|
|
There was a problem hiding this comment.
Does this needs a separate file ? I thought this would an i/p parameter like the four parameters here. These are few line changes, right ?
There was a problem hiding this comment.
We could make it a single file with extra parameters, but this seemed the easier path to me.
| export JENKINS_DO_TESTS=ON | ||
| export JENKINS_DO_EXAMPLES=ON | ||
|
|
||
| export QUEUE=rhel7G |
There was a problem hiding this comment.
it used to be 7F and now it is 7G. I like it.
| fi | ||
|
|
||
| module load devpack/20180521/openmpi/2.1.2/gcc/7.2.0/cuda/9.2.88 | ||
| module swap openblas/0.2.20/gcc/7.2.0 netlib/3.8.0/gcc/7.2.0 |
There was a problem hiding this comment.
any reason not to use openblas ?
There was a problem hiding this comment.
There is a Trilinos issue where use of OpenBLAS resulted in lots of problems on White/Ride and the solution they decided upon was to use Netlib instead. I just followed the same course as the ATDM testing.
|
|
||
| export JENKINS_ARCH="Power8,Pascal60" | ||
| export JENKINS_ARCH_CXX_FLAG="-mcpu=power8 -arch=sm_60 --expt-extended-lambda --std=c++11" | ||
| export JENKINS_ARCH_C_FLAG="-mcpu=power8" |
There was a problem hiding this comment.
is this manual setup for cflags necessary ? it should be set from cmake script ?
There was a problem hiding this comment.
I'm not sure how/if the KOKKOS_ARCH info propogates to the cflags, this may be a remnant that can be removed but was in the scripts when Christian updated them after introducing KOKKOS_ARCH so I left it in.
There was a problem hiding this comment.
I guessed so. However, as this is trilinos integration test script, those environment variable should be caught from trilinos cmake script. If we provide correct environment, then we cannot test trilinos configure is right or not. While integrating trilinos, this problem is not important but it would be good to be addressed in my opinion.
Add script to support Pascal GPU testing