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

Pass wavebands from coupler to wave_parameters_CS #255

Merged

Conversation

gustavo-marques
Copy link
Collaborator

On behalf of Bill Large.

This commit passes the waveband information received from the coupler to wave_parameters_CS. This information is set to public so that it can be used elsewhere. To exercise this code the following must be set:

SURFBAND = COUPLER
WAVE_METHOD = SURFACE_BANDS

No answer changes.

Replaces LU_pred to L_diag, since now this logical only controls
if diagnostics should be posted.
This commit adds the latest updates to the vertFPmix subroutine
after Bill Large did some cleaning. We have highlight places in
the code where work must be done.
omega_w2x is the counter-clockwise angle of the wind stress with
respect to the horizontal abscissa (x-coordinate) at tracer
points [rad]. This variable is needed in the vertPFmix subroutine.
This line of code was lost during the last merge.
This commit passes the waveband information recieved
from the coupler to wave_parameters_CS. This information
is set to public so that it can be used elsewhere. To
exercise this code the following must be set:

SURFBAND = COUPLER
WAVE_METHOD = SURFACE_BANDS

No answer changes.
@gustavo-marques
Copy link
Collaborator Author

Please evaluate #256 before this PR.

@codecov-commenter
Copy link

codecov-commenter commented Sep 7, 2023

Codecov Report

Patch coverage: 3.94% and project coverage change: -0.14% ⚠️

Comparison is base (d4aa108) 38.04% compared to head (48913da) 37.90%.

❗ Current head 48913da differs from pull request most recent head 4cd7e8f. Consider uploading reports for the commit 4cd7e8f to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##           dev/ncar     #255      +/-   ##
============================================
- Coverage     38.04%   37.90%   -0.14%     
============================================
  Files           269      269              
  Lines         76872    77176     +304     
  Branches      14125    14170      +45     
============================================
+ Hits          29243    29256      +13     
- Misses        42356    42641     +285     
- Partials       5273     5279       +6     
Files Changed Coverage Δ
...c/parameterizations/vertical/MOM_set_viscosity.F90 60.98% <ø> (ø)
src/user/MOM_wave_interface.F90 1.22% <0.00%> (-0.03%) ⬇️
...c/parameterizations/vertical/MOM_vert_friction.F90 48.64% <2.86%> (-9.95%) ⬇️
src/core/MOM_forcing_type.F90 41.99% <9.09%> (-0.26%) ⬇️
src/core/MOM_dynamics_split_RK2.F90 62.01% <10.81%> (-2.80%) ⬇️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.

📢 Have feedback on the report? Share it here.

@alperaltuntas alperaltuntas merged commit 5bc0c5e into NCAR:dev/ncar Sep 8, 2023
10 checks passed
alperaltuntas pushed a commit to alperaltuntas/MOM6 that referenced this pull request Sep 15, 2023
.testing: Fix concurrency errors in tc4 rules
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.

3 participants