Skip to content

Higher Order Flux Corrected Horizontal Transport. #447

Draft
overfelt wants to merge 27 commits into
E3SM-Project:developfrom
overfelt:overfelt/FCTHigherOrderTendency
Draft

Higher Order Flux Corrected Horizontal Transport. #447
overfelt wants to merge 27 commits into
E3SM-Project:developfrom
overfelt:overfelt/FCTHigherOrderTendency

Conversation

@overfelt

@overfelt overfelt commented Jun 26, 2026

Copy link
Copy Markdown

Checklist

  • Documentation:
  • Linting
  • Building
    • CMake build does not produce any new warnings from changes in this PR
  • Testing
    • Add a comment to the PR titled Testing with the following:
      • Which machines CTest unit tests
        have been run on and indicate that are all passing.
      • The Polaris omega_pr test suite
        has passed, using the Polaris e3sm_submodules/Omega baseline
      • Document machine(s), compiler(s), and the build path(s) used for -p for both the baseline (Polaris e3sm_submodules/Omega) and the PR build
      • Indicate "All tests passed" or document failing tests
      • Document testing used to verify the changes including any tests that are added/modified/impacted.
      • Performance related PRs: Please include a relevant PACE experiment link documenting performance before and after.
    • New tests:
      • CTest unit tests for new features have been added per the approved design.
      • Polaris tests for new features have been added per the approved design (and included in a test suite)
  • Stealth Features
    • If any stealth features are included in the PR, please confirm that they have been documented.

@overfelt overfelt marked this pull request as draft June 26, 2026 18:24
@cbegeman

cbegeman commented Jul 2, 2026

Copy link
Copy Markdown

@overfelt I think in the meeting you mentioned success comparing the sphere_transport tests with MPAS-O? When this is ready for review, it would be great to see the convergence plots. Let me know if there are any hang-ups with the tests, as I imagine minimally namelist updates will be needed with this PR. I'm also thinking we would add versions of the tests that use FCT in addition to the current tests that do not, which would be a follow-on polaris PR with a submodule update to this PR after it is merged into Omega.

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.

2 participants