-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Option --ppn
in MPIEXEC LM
#3035
Conversation
Codecov Report
@@ Coverage Diff @@
## devel #3035 +/- ##
==========================================
+ Coverage 43.98% 43.99% +0.01%
==========================================
Files 96 96
Lines 10568 10570 +2
==========================================
+ Hits 4648 4650 +2
Misses 5920 5920
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@GKNB does this PR implements the behavior you expected? |
ping? |
I think this branch is also consistent with the main devel branch now, and in the main devel branch I can see a --ppn flag is already added for each task and it runs successfully for the mini-app, so I think it works fine. |
confirmed with the test run on Polaris |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - thanks!
No description provided.