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

*Correct indexing bug in ALE_sponge #1244

Merged

Conversation

MJHarrison-GFDL
Copy link
Contributor

  • Answer change for tc4 or any configurations
    which use the sponge option (tracer nudging to data)
    in ALE mode with USE_TEMPERATURE = True.

MJHarrison-GFDL and others added 2 commits November 9, 2020 08:29
 - Answer change for tc4 or any configurations
   which use the sponge option (tracer nudging to data)
   in ALE mode with USE_TEMPERATURE = True.
@codecov-io
Copy link

codecov-io commented Nov 11, 2020

Codecov Report

Merging #1244 (b3656fa) into dev/gfdl (42a9eaf) will decrease coverage by 0.04%.
The diff coverage is 32.03%.

Impacted file tree graph

@@             Coverage Diff              @@
##           dev/gfdl    #1244      +/-   ##
============================================
- Coverage     46.08%   46.04%   -0.05%     
============================================
  Files           214      224      +10     
  Lines         69399    71037    +1638     
============================================
+ Hits          31984    32707     +723     
- Misses        37415    38330     +915     
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 229 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 6b41926...b3656fa. Read the comment docs.

Copy link
Collaborator

@marshallward marshallward left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Correctly replaces index bound with the index.

@marshallward
Copy link
Collaborator

marshallward commented Nov 11, 2020

@marshallward
Copy link
Collaborator

This has passed Gaea regression, although it does not pass our .testing regression. This is most likely because tc4 touches code which the regressions do not.

As this is a bugfix, I am merging this one.

@marshallward marshallward merged commit ad53667 into mom-ocean:dev/gfdl Nov 11, 2020
@MJHarrison-GFDL MJHarrison-GFDL deleted the ale_sponge_tracer_index_patch branch December 3, 2020 15:20
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