Skip to content

Conversation

@skamaroc
Copy link
Contributor

@skamaroc skamaroc commented Dec 4, 2025

This commit removes the scaling of the gravity-wave absorbing layer coefficient by the local mesh size. Previously, the absorbing layer coefficient was scaled by dx/dx_fine. For large values of dx/dx_fine instabilities have been ancountered associated with this scaling of the absorbing layer coefficient.

Units have been added to the configuration variable config_xnutr (1/s) in the Registry - it is used in calculating the absorbing layer coefficient. Units have also been added to config_visc4_2dsmag (m/s) in the Registry - it is used in the calculation of the hyperviscosity.

The title above should be a 1 line short summary of the pull request (i.e. what the project the PR represents is intended to do).

Enter a description of this PR. This should include why this PR was created, and what it does.

Testing and relations to other Pull Requests should be added as subsequent comments.

See the below examples for more information.
MPAS-Dev/MPAS#930
MPAS-Dev/MPAS#931

@mgduda mgduda self-requested a review December 28, 2025 21:41
@mgduda mgduda changed the base branch from master to develop December 28, 2025 21:42
…esh size

This commit removes the scaling of the gravity-wave absorbing layer coefficient
by the local mesh size.  Previously, the absorbing layer coefficient was scaled
by dx/dx_fine.  For large values of dx/dx_fine instabilities have been
encountered associated with this scaling of the absorbing layer coefficient.

Units have been added to the configuration variable config_xnutr (1/s) in the
Registry - it is used in calculating the absorbing layer coefficient.  Units
have also been added to config_visc4_2dsmag (m/s) in the Registry - it is used
in the calculation of the hyperviscosity.
@mgduda mgduda force-pushed the gw_absorbing_layer_fix branch from d2d214e to 6b6c272 Compare December 28, 2025 22:16
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