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

Time Stepping design doc #94

Conversation

mark-petersen
Copy link

@mark-petersen mark-petersen commented Jun 18, 2024

The time stepping design document for Omega-0. During this review, the documentation is available here for convenient viewing.

Checklist

  • Documentation:
    • Documentation has been built locally and changes look as expected

@mark-petersen mark-petersen added the documentation Improvements or additions to documentation label Jun 18, 2024
@mark-petersen mark-petersen self-assigned this Jun 18, 2024
Copy link

@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.

@mark-petersen, I think this looks good. I just have one comment about adding a temporal convergence study to the testing section.

@mark-petersen
Copy link
Author

Please review when you get a chance: @sbrus89 @mwarusz @hyungyukang

I would like to make any revisions and merge this week.

@mark-petersen mark-petersen added Design and removed documentation Improvements or additions to documentation labels Jun 21, 2024
@mwarusz
Copy link
Member

mwarusz commented Jun 24, 2024

Looks good. I have a comment regarding rhs functions that depend explicitly on time. I remember that in MPAS-O the RK4 scheme had some issues with that. Can we add treating this properly as a required or desired capability ? The description of the forward/backward scheme also doesn't treat this case, but it is needed for the manufactured solution test.

Copy link

@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.

Looks good, @mark-petersen. Thanks!

Copy link

@hyungyukang hyungyukang left a comment

Choose a reason for hiding this comment

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

@mark-petersen , Overall, all look great. I left a comment on adding other time stepping methods. We can add more later together after merging.

Co-authored-by: Hyun-Gyu Kang <47987430+hyungyukang@users.noreply.github.com>
@mark-petersen mark-petersen merged commit cf5fd40 into E3SM-Project:develop Jul 1, 2024
1 check passed
@mark-petersen mark-petersen deleted the mark-petersen/omega/design-omega0-time-stepping branch July 1, 2024 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants