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

Fix parabolic bowl registry for MPAS-Ocean init mode #5930

Merged

Conversation

xylar
Copy link
Contributor

@xylar xylar commented Sep 18, 2023

There were a few syntax issues with the MPAS-Ocean registry for the parabolic bowl test case that prevent it from being parsed correctly in Polaris. They are fixed here.

@xylar xylar added mpas-ocean BFB PR leaves answers BFB labels Sep 18, 2023
There were a few syntax issues with the MPAS-Ocean registry for
the parabolic bowl test case that prevent it from being parsed
correctly in Polaris.  They are fixed here.
@xylar xylar force-pushed the mpas-ocean/fix-parabolic-bowl-registry branch from 6594c25 to f755e37 Compare September 18, 2023 20:10
@xylar
Copy link
Contributor Author

xylar commented Sep 18, 2023

@cbegeman and @sbrus89, this seems to have fixed the Polaris issue. Can you test and make sure it works for you, too?

@xylar xylar added the MPAS-Ocean standalone Issues and features for standalone MPAS-Ocean code that dont impact E3SM. label Sep 19, 2023
@xylar xylar changed the title [MPAS-Ocean standalone] Fix parabolic bowl registry for MPAS-Ocean init mode Fix parabolic bowl registry for MPAS-Ocean init mode Sep 19, 2023
Copy link
Contributor

@cbegeman cbegeman left a comment

Choose a reason for hiding this comment

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

@xylar This fix works for me. Thanks!

@xylar
Copy link
Contributor Author

xylar commented Sep 22, 2023

@sbrus89, waiting for your review on this one, when you have time.

Copy link
Contributor

@sbrus89 sbrus89 left a comment

Choose a reason for hiding this comment

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

While we're at it, can you include this change as well?

Co-authored-by: Steven Brus <[email protected]>
Copy link
Contributor

@sbrus89 sbrus89 left a comment

Choose a reason for hiding this comment

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

Thanks @xylar for finding this bug. I apologize for letting it get in there.

This now works for me as well.

jonbob added a commit that referenced this pull request Sep 26, 2023
… (PR #5930)

Fix parabolic bowl registry for MPAS-Ocean init mode

There were a few syntax issues with the MPAS-Ocean registry for the
parabolic bowl test case that prevent it from being parsed correctly in
Polaris. They are fixed here. Not used in E3SM.

[BFB]
@jonbob
Copy link
Contributor

jonbob commented Sep 26, 2023

passes:

  • ERS_Ld5.T62_oQU120.CMPASO-NYF.chrysalis_intel
  • SMS_D_Ld3.T62_oQU120.CMPASO-IAF.chrysalis_intel

merged to next

@jonbob jonbob merged commit 894b5b2 into E3SM-Project:master Sep 27, 2023
3 checks passed
@jonbob
Copy link
Contributor

jonbob commented Sep 27, 2023

merged to master

@xylar
Copy link
Contributor Author

xylar commented Sep 27, 2023

@jonbob, thank you very much!

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 standalone Issues and features for standalone MPAS-Ocean code that dont impact E3SM. mpas-ocean
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants