Skip to content
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

Add single missing variable to OMP private #6037

Merged
merged 1 commit into from
Nov 14, 2023
Merged

Conversation

jonbob
Copy link
Contributor

@jonbob jonbob commented Oct 31, 2023

Adds one new variable to an OMP private directive in a recently modified routine. It doesn't appear to cause PET failures but still should be added.

[non-BFB] potentially for threaded runs with active ocean

@jonbob jonbob requested a review from mark-petersen October 31, 2023 19:21
@jonbob jonbob self-assigned this Oct 31, 2023
Copy link

PR Preview Action v1.4.4
🚀 Deployed preview to https://E3SM-Project.github.io/E3SM/pr-preview/pr-6037/
on branch gh-pages at 2023-10-31 19:22 UTC

Copy link
Contributor

@mark-petersen mark-petersen left a comment

Choose a reason for hiding this comment

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

Approved by visual inspection. Looks good, thanks.

@rljacob rljacob added the BFB PR leaves answers BFB label Oct 31, 2023
@mark-petersen
Copy link
Contributor

In MPAS standalone tests with gnu debug on chrysalis, master currently fails

01:13 FAIL ocean_global_ocean_QU240_WOA23_threads_test
01:10 FAIL ocean_global_ocean_QU240_WOA23_RK4_threads_test

and this PR fixes it so they both pass. Thanks @jonbob!

For some master passes with gnu optimized, so the tests do not always catch this problem.

jonbob added a commit that referenced this pull request Nov 13, 2023
Add single missing variable to OMP private

Adds one new variable to an OMP private directive in a recently modified
routine. It doesn't appear to cause PET failures but still should be
added.

[non-BFB] potentially for threaded runs with active ocean
@jonbob
Copy link
Contributor Author

jonbob commented Nov 13, 2023

passes:

  • ERS_Ld5.T62_oQU120.CMPASO-NYF.chrysalis_intel

merged to next

@jonbob jonbob merged commit c336779 into master Nov 14, 2023
@jonbob jonbob deleted the jonbob/mpaso/fix-small-omp branch November 14, 2023 17:10
@jonbob
Copy link
Contributor Author

jonbob commented Nov 14, 2023

merged to master

xylar added a commit to xylar/compass that referenced this pull request Dec 3, 2023
This merge updates the E3SM-Project submodule from [894b5b2](https://github.com/E3SM-Project/E3SM/tree/894b5b2) to [5d5f15c](https://github.com/E3SM-Project/E3SM/tree/5d5f15c).

This update includes the following MPAS-Ocean and MPAS-Frameworks PRs (check mark indicates bit-for-bit with previous PR in the list):
- [ ]  (ocn) E3SM-Project/E3SM#5945
- [ ]  (ocn) E3SM-Project/E3SM#5946
- [ ]  (ocn) E3SM-Project/E3SM#5947
- [ ]  (ocn) E3SM-Project/E3SM#5999
- [ ]  (ocn) E3SM-Project/E3SM#6037
- [ ]  (ocn) E3SM-Project/E3SM#5989
- [ ]  (ocn) E3SM-Project/E3SM#6035
- [ ]  (ocn) E3SM-Project/E3SM#6077
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BFB PR leaves answers BFB mpas-ocean
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants