Skip to content

Conversation

@lucamar
Copy link
Contributor

@lucamar lucamar commented Feb 23, 2022

I propose to add the workaround reported in PROGENV-8 to the check OpenaccCudaCpp, namely reducing the MPI tasks per node to 11, until we get the fix by HPE Cray.

@codecov-commenter
Copy link

Codecov Report

Merging #2446 (b7ab2e3) into master (14ae053) will decrease coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2446      +/-   ##
==========================================
- Coverage   85.71%   85.71%   -0.01%     
==========================================
  Files          56       56              
  Lines       10525    10536      +11     
==========================================
+ Hits         9022     9031       +9     
- Misses       1503     1505       +2     
Impacted Files Coverage Δ
reframe/utility/osext.py 84.36% <0.00%> (ø)
reframe/core/schedulers/lsf.py 48.75% <0.00%> (+5.27%) ⬆️

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 14ae053...b7ab2e3. Read the comment docs.

@pep8speaks
Copy link

pep8speaks commented Feb 23, 2022

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 at 2022-02-23 20:26:37 UTC

@vkarak vkarak changed the title [test] Temporary workaround for OpenaccCudaCpp [test] Temporary workaround for OpenaccCudaCppCheck Feb 23, 2022
@vkarak vkarak merged commit 050ef43 into reframe-hpc:master Feb 23, 2022
@lucamar lucamar deleted the fix-openacc branch February 24, 2022 11:00
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.

4 participants