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

Evaluate time-dependent tendencies at correct time for each RK4 stage #6685

Closed
wants to merge 10 commits into from

Conversation

sbrus89
Copy link
Contributor

@sbrus89 sbrus89 commented Oct 14, 2024

This PR corrects a longstanding issue with RK4 and tendency terms which have an explicit time dependence (see #5364 and MPAS-Dev/MPAS-Model#375). Big thanks to @gcapodag and @jeremy-lilly for providing the fix in their LTS development branch linked in #5364.

Here, I have migrated these changes from their MPAS-Dev branch to E3SM and added the modifications to correct the convergence issues for the manufactured solution test case: E3SM-Project/polaris#72

@sbrus89 sbrus89 added the MPAS-Ocean standalone Issues and features for standalone MPAS-Ocean code that dont impact E3SM. label Oct 14, 2024
@sbrus89
Copy link
Contributor Author

sbrus89 commented Oct 14, 2024

Testing:

With this PR, the manufactured solution test case in Polaris converges at 2nd order when refining in space/time:
image

When refining in time only, 4th order convergence is achieved:
image

However, I have not been able to reproduce the 4th order convergence for the hurricane test case from Lilly et al. 2023 (Figure 4). The best I have been able to achieve is first order:
convergence
For this result, I ran from 2012-10-13_00:00:00 to 2012-10-13_01:00:00 starting from a 2012-10-13_00:00:00 restart file. I used dt=0.1 as a reference solution.

@sbrus89
Copy link
Contributor Author

sbrus89 commented Oct 17, 2024

Closing to fix branch name.

@sbrus89 sbrus89 closed this Oct 17, 2024
@gcapodag
Copy link
Contributor

Testing:

With this PR, the manufactured solution test case in Polaris converges at 2nd order when refining in space/time: image

When refining in time only, 4th order convergence is achieved: image

However, I have not been able to reproduce the 4th order convergence for the hurricane test case from Lilly et al. 2023 (Figure 4). The best I have been able to achieve is first order: convergence For this result, I ran from 2012-10-13_00:00:00 to 2012-10-13_01:00:00 starting from a 2012-10-13_00:00:00 restart file. I used dt=0.1 as a reference solution.

@jeremy-lilly 👀

@sbrus89
Copy link
Contributor Author

sbrus89 commented Oct 17, 2024

Moved to #6695

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
MPAS-Ocean standalone Issues and features for standalone MPAS-Ocean code that dont impact E3SM.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants