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 new limiter function for the implicit unstructured #692

Closed
wants to merge 1 commit into from

Conversation

aronroland
Copy link
Collaborator

@aronroland aronroland commented Apr 27, 2022

Pull Request Summary

Hersbach & Janssen with the motivation to reduce time step dependency for the implicit scheme when big time steps is chosen.

Description

We maximize the limiter of Komen et al. and Hersbach and Jannnsen. In the absence of wind and small time steps, the max operator between both limiter values makes sure that the limiter remains convergent (Tolman https://doi.org/10.1080/10236730290003392). When USTAR goes to zero and deltat is small, the Komen et al. limiter will lead the solution and for big time steps and significant friction velocities we retain as much as possible the max growth rate. Such implementation redues significantly the time step dependency due to the Komen et al. limiter.

new feature

Issue(s) addressed

fixed #704

Commit Message

Add a new limiter function for the implicit unstructured part.

Check list

Testing

  • How were these changes tested?
  • Are the changes covered by regression tests? (If not, why? Do new tests need to be added?)
  • Have the matrix regression tests been run (if yes, please note HPC and compiler)?
  • Please provide the summary output of matrix.comp (matrix.Diff.txt, matrixCompFull.txt and matrixCompSummary.txt):
  • Please indicate the expected changes in the regression test output (Note the known list of non-identical tests).
  • Please provide the summary output of matrix.comp (matrix.Diff.txt, matrixCompFull.txt and matrixCompSummary.txt):

@aliabdolali aliabdolali changed the title scal_limiter: add new limiter Add new limiter function for the implicit unstructured May 3, 2022
@aliabdolali aliabdolali marked this pull request as draft July 7, 2022 13:17
@JessicaMeixner-NOAA
Copy link
Collaborator

JessicaMeixner-NOAA commented Sep 14, 2022

@aronroland any update on this draft PR? Should it be closed at this point or re-visited?

@aronroland
Copy link
Collaborator Author

Hi Jessica,
we can abort this and delete this PR. I worked further and we need to make a new pull request. Many thanks looking at this!
Cheers
Aron

@JessicaMeixner-NOAA
Copy link
Collaborator

@aronroland we'll look forward to the new PR when you are ready!

@thesser1 thesser1 deleted the scal_limiter branch October 26, 2023 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Limiter implementation to reduce time step dependency for the implicit scheme
3 participants