Skip to content

Conversation

@kraushm
Copy link
Contributor

@kraushm kraushm commented Sep 3, 2018

Settings file changed to incorporate the c2sm modules provided by MCH.

Fixes #430.

@kraushm kraushm self-assigned this Sep 3, 2018
@vkarak vkarak changed the title WIP: [kesch] Mch settings c2sm [WIP] [ci] MCH settings for c2sm environments Sep 3, 2018
@vkarak vkarak changed the title [WIP] [ci] MCH settings for c2sm environments [WIP] [ci] MCH settings for C2SM environments Sep 3, 2018
@vkarak vkarak added this to the ReFrame sprint 2018w35 milestone Sep 3, 2018
},
'PrgEnv-c2sm-gnu-gpu': {
'type': 'ProgEnvironment',
'modules': ['PrgEnv-gnu', '/apps/common/UES/sandbox/kraushm/c2sm-rcm-env/env', 'c2sm/gnu-env/gpu'],
Copy link
Contributor

@ajocksch ajocksch Sep 3, 2018

Choose a reason for hiding this comment

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

For

'modules': ['PrgEnv-gnu', '/scratch/ajocksch/c2sm-rcm-env.git/modules/kesch/c2sm/gnu-env/gpu', 'mvapich2gdr_gnu/2.2_cuda_8.0'],

it is working

btw. why is 'PrgEnv-gnu' needed?

Copy link
Contributor

Choose a reason for hiding this comment

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

btw. why is 'PrgEnv-gnu' needed?

I think we should focus only on the Cray and PGI version for the moment.

'modules': ['/apps/common/UES/sandbox/kraushm/c2sm-rcm-env/env', 'c2sm/cray-env/base'],
'cc': 'cc',
'cxx': 'CC',
'ftn': 'ftn -D__CRAY_FORTRAN_',
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the compiler variables should be using the corresponding environment variables that the guys set, e.g. $CXX etc. That's because they have a complex environment setup that is mixing up the standard programming environment. I don't think what we have here is enough.

@vkarak
Copy link
Contributor

vkarak commented Oct 12, 2018

Closing this. Replaced by #512.

@vkarak vkarak closed this Oct 12, 2018
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.

3 participants