Skip to content

MMR (mean mixing ratio) limiter update - #754

Open
ta440 wants to merge 2 commits into
mainfrom
mmr_update
Open

MMR (mean mixing ratio) limiter update#754
ta440 wants to merge 2 commits into
mainfrom
mmr_update

Conversation

@ta440

@ta440 ta440 commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Updates the formulation and implementation of the mean mixing ratio (MMR) limiter. The previous limiter worked in the Terminator Toy test, but uses in other transport tests highlighted an isssue with conservation. The change is to now compute mean fields after transport, rather than transporting the mean fields throughout a single timestep.

This pull request includes the following modifications:

  1. Computes the mean fields after each transport step, rather than transporting the mean fields. This avoids issues with dispersion errors in the lowest order space breaking conservation when blending.
  2. The MMR augmentation does not require a modified residual any more, as the mean fields are not transported, but just computed via projection. This greatly shortens the code!
  3. The limiting kernel uses WR instead of INC for lamda (the blending weights) to correct an error in the previous version.
  4. The test now is for a transport case, rather than the Terminator Toy, as this is found to be a more thorough test of the limiter.

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.

1 participant