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

Diagnostics for depth-averaged acceleration from diapycnal mixing #1217

Merged
merged 6 commits into from
Oct 15, 2020

Conversation

hmkhatri
Copy link
Contributor

@hmkhatri hmkhatri commented Sep 29, 2020

Adding to 2D fractional-thickness weighted momentum budget diagnostics, diagnostics for zonal and meridional acceleration terms from diapycnal mixing have been added.

Plus a small correction in read_data call for Idamp in sponge initialization.

@codecov-commenter
Copy link

Codecov Report

Merging #1217 into dev/gfdl will decrease coverage by 0.00%.
The diff coverage is 35.19%.

Impacted file tree graph

@@             Coverage Diff              @@
##           dev/gfdl    #1217      +/-   ##
============================================
- Coverage     46.08%   46.08%   -0.01%     
============================================
  Files           214      224      +10     
  Lines         69399    70645    +1246     
============================================
+ Hits          31984    32556     +572     
- Misses        37415    38089     +674     
Impacted Files Coverage Δ
...g_src/external/GFDL_ocean_BGC/FMS_coupler_util.F90 0.00% <0.00%> (ø)
...fig_src/external/GFDL_ocean_BGC/generic_tracer.F90 0.00% <0.00%> (ø)
...c/external/GFDL_ocean_BGC/generic_tracer_utils.F90 0.00% <0.00%> (ø)
config_src/external/ODA_hooks/kdtree.f90 0.00% <0.00%> (ø)
config_src/external/ODA_hooks/ocean_da_core.F90 0.00% <0.00%> (ø)
config_src/external/ODA_hooks/ocean_da_types.F90 0.00% <0.00%> (ø)
config_src/external/ODA_hooks/write_ocean_obs.F90 0.00% <0.00%> (ø)
config_src/solo_driver/MESO_surface_forcing.F90 0.00% <0.00%> (ø)
config_src/solo_driver/user_surface_forcing.F90 0.00% <0.00%> (ø)
src/ALE/MOM_regridding.F90 31.47% <0.00%> (-0.17%) ⬇️
... and 214 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7cb606e...9dcb866. Read the comment docs.

Changed the default for SPONGE_IDAMP_VAR from "IDAMP" to "Idamp" so that when this variable is used, as it would be with the proposed change, it does not break any existing experiments.  This could change some MOM_parameter_doc files, but will not change any solutions because the variable whose default is being set was not in use.
@codecov-io
Copy link

codecov-io commented Oct 15, 2020

Codecov Report

Merging #1217 into dev/gfdl will increase coverage by 0.05%.
The diff coverage is 33.65%.

Impacted file tree graph

@@             Coverage Diff              @@
##           dev/gfdl    #1217      +/-   ##
============================================
+ Coverage     46.08%   46.13%   +0.05%     
============================================
  Files           214      224      +10     
  Lines         69399    70997    +1598     
============================================
+ Hits          31984    32757     +773     
- Misses        37415    38240     +825     
Impacted Files Coverage Δ
...g_src/external/GFDL_ocean_BGC/FMS_coupler_util.F90 0.00% <0.00%> (ø)
...fig_src/external/GFDL_ocean_BGC/generic_tracer.F90 0.00% <0.00%> (ø)
...c/external/GFDL_ocean_BGC/generic_tracer_utils.F90 0.00% <0.00%> (ø)
config_src/external/ODA_hooks/kdtree.f90 0.00% <0.00%> (ø)
config_src/external/ODA_hooks/ocean_da_core.F90 0.00% <0.00%> (ø)
config_src/external/ODA_hooks/ocean_da_types.F90 0.00% <0.00%> (ø)
config_src/external/ODA_hooks/write_ocean_obs.F90 0.00% <0.00%> (ø)
config_src/solo_driver/MESO_surface_forcing.F90 0.00% <0.00%> (ø)
config_src/solo_driver/user_surface_forcing.F90 0.00% <0.00%> (ø)
src/ALE/MOM_regridding.F90 31.47% <0.00%> (-0.17%) ⬇️
... and 217 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f3196d3...1b5d79b. Read the comment docs.

@Hallberg-NOAA
Copy link
Collaborator

The pipeline testing for this PR can be found at https://gitlab.gfdl.noaa.gov/ogrp/MOM6/-/pipelines/11387.

@marshallward
Copy link
Collaborator

Passes up to a parameter update.

@marshallward marshallward merged commit 58a662b into mom-ocean:dev/gfdl Oct 15, 2020
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.

5 participants