Remove tasks per node from OMP performance groups#597
Remove tasks per node from OMP performance groups#597Benjamin Went (MetBenjaminWent) wants to merge 1 commit into
Conversation
|
Here is a snippet example after running the ex1a_omp_developer group, with Genoa Note that this is a snippet from the 4t run, which is in the tasks list here (which this PR modifes, as well as the 2T): The The cores/tasks per node ( |
Joseph Abram (J-J-Abram)
left a comment
There was a problem hiding this comment.
Happy with changes. Did a quick run to test and all passed.
Wolfgang Hayek (tinyendian)
left a comment
There was a problem hiding this comment.
We don't currently run this test, but looks good to me.



PR Summary
Remove ranks/tasks per node from the OMP developer groups, so that they can be effectively run on the Genoa nodes.
Sci/Tech Reviewer: Joseph Abram (@J-J-Abram)
Code Reviewer: mo-marqh
Code Quality Checklist
Testing
trac.log
Test Suite Results - lfric_apps - remove-rpn-omp-tasks/run1
Suite Information
Task Information
✅ succeeded tasks - 51
Security Considerations
Performance Impact
AI Assistance and Attribution
Documentation
PSyclone Approval
Sci/Tech Review
(Please alert the code reviewer via a tag when you have approved the SR)
Code Review