search

NCAR / MOM6

NCAR/CESM fork of the Modular Ocean Model v.6 (MOM6)
Other
2 stars 19 forks source link

Add option to apply non-local momentum flux #256

Closed gustavo-marques closed 1 year ago

gustavo-marques commented 1 year ago

On behalf of Bill Large. This PR adds the option to apply non-local momentum flux throughout the surface boundary layer. This is done when the local shear is not aligned with the wind. The scheme adds non-local momentum flux in the direction of the wind. To enable this option set FPMIX=True. Additional details on this scheme can be found at http://journals.ametsoc.org/doi/abs/10.1175/JPO-D-18-0066.1

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage has no change and project coverage change: +9.12% :tada:

Comparison is base (d4aa108) 38.04% compared to head (3ea84ff) 47.16%.

:exclamation: Current head 3ea84ff differs from pull request most recent head 65ac638. Consider uploading reports for the commit 65ac638 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## dev/ncar #256 +/- ## ============================================ + Coverage 38.04% 47.16% +9.12% ============================================ Files 269 41 -228 Lines 76872 4575 -72297 Branches 14125 805 -13320 ============================================ - Hits 29243 2158 -27085 + Misses 42356 2236 -40120 + Partials 5273 181 -5092 ``` [see 254 files with indirect coverage changes](https://app.codecov.io/gh/NCAR/MOM6/pull/256/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=NCAR)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

alperaltuntas commented 1 year ago

@gustavo-marques Is this PR ready for review and testing?

gustavo-marques commented 1 year ago

@alperaltuntas; yes, it is. Thanks!