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

Allow variable number of levels for upper-level horizontal Laplacian filter #1040

Merged

Conversation

skamaroc
Copy link
Contributor

@skamaroc skamaroc commented Apr 7, 2023

This PR adds an option to specify the number of levels for the upper-level horizontal Laplacian filter.

The horizontal Laplacian that is used in the upper-level damping layer was hardwired to be applied over the topmost 3 layers. This update allows for the namelist specification of the number of topmost layers over which it is applied along with a linear ramping of the viscosity used in the Laplacian.

Change Dynamics 1.2 for the Version 8 release.

@skamaroc skamaroc requested a review from mgduda April 7, 2023 19:26
@mgduda mgduda changed the base branch from master to develop April 7, 2023 20:06
@mgduda mgduda changed the title Option to specify number of levels for upper-level horz. Laplacian fi… Allow variable number of levels for upper-level horizontal Laplacian filter Jun 1, 2023
…lter

The horizontal Laplacian that is used in the upper-level damping layer was
hardwired to be applied over the topmost 3 layers.  This update allows for
the namelist specification of the number of topmost layers over which it is
applied along with a linear ramping of the viscosity used in the Laplacian.

Tested and coefficients checked.

Change Dynamics 1.2 for the Version 8 release.
@mgduda mgduda force-pushed the v8_add_1.2_wave_absorber_generalization branch from 2bb1b88 to 9c2b1d7 Compare June 1, 2023 22:54
@mgduda mgduda merged commit 0ed63a6 into MPAS-Dev:develop Jun 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants