-
Notifications
You must be signed in to change notification settings - Fork 354
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
Fix parabolic bowl registry for MPAS-Ocean init mode #5930
Conversation
components/mpas-ocean/src/mode_init/Registry_parabolic_bowl.xml
Outdated
Show resolved
Hide resolved
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.
6594c25
to
f755e37
Compare
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.
@xylar This fix works for me. Thanks!
@sbrus89, waiting for your review on this one, when you have time. |
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.
While we're at it, can you include this change as well?
components/mpas-ocean/src/mode_init/Registry_parabolic_bowl.xml
Outdated
Show resolved
Hide resolved
Co-authored-by: Steven Brus <[email protected]>
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.
Thanks @xylar for finding this bug. I apologize for letting it get in there.
This now works for me as well.
… (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]
passes:
merged to next |
merged to master |
@jonbob, thank you very much! |
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.