Skip to content

Conversation

@vkarak
Copy link
Contributor

@vkarak vkarak commented Nov 23, 2019

Fixes #1059.

@rtmclay Can you try this out? It's the same as the one we tried on the fly, but just to be on the safe side.

@codecov-io
Copy link

codecov-io commented Nov 23, 2019

Codecov Report

Merging #1066 into master will decrease coverage by <.01%.
The diff coverage is 75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1066      +/-   ##
==========================================
- Coverage      92%   91.99%   -0.01%     
==========================================
  Files          81       81              
  Lines       11020    11024       +4     
==========================================
+ Hits        10139    10142       +3     
- Misses        881      882       +1
Impacted Files Coverage Δ
reframe/core/launchers/mpi.py 98.43% <75%> (-1.57%) ⬇️

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 d0ee9ee...14d687e. Read the comment docs.

@rtmclay
Copy link

rtmclay commented Nov 24, 2019

Yes, your feat/ibrun-launcher branch worked for me. Thanks!

@vkarak vkarak merged commit e491ae9 into reframe-hpc:master Nov 25, 2019
@vkarak vkarak deleted the feat/ibrun-launcher branch November 25, 2019 09:54
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.

Add support for TACC's ibrun parallel job launcher

4 participants