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

Ice shelf and barotropic rescale variable init #1021

Merged
merged 3 commits into from
Nov 8, 2019

Conversation

marshallward
Copy link
Collaborator

Some index bounds in the ice shelf code were not initialized to the grid
values. This patch sets those values.

The vel_rescale parameter used in barotropic_init was also absent from
one block of code, and has been added. (This is probably a temporary
variable that will be phased out as dimensional scaling progresses.)

Presumably none of this code was being run, because it would have
normally caused severe problems.

Some index bounds in the ice shelf code were not initialized to the grid
values.  This patch sets those values.

The vel_rescale parameter used in barotropic_init was also absent from
one block of code, and has been added.  (This is probably a temporary
variable that will be phased out as dimensional scaling progresses.)

Presumably none of this code was being run, because it would have
normally caused severe problems.
@codecov-io
Copy link

codecov-io commented Oct 16, 2019

Codecov Report

Merging #1021 into dev/gfdl will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##           dev/gfdl    #1021      +/-   ##
============================================
- Coverage     43.61%   43.61%   -0.01%     
============================================
  Files           212      212              
  Lines         62173    62176       +3     
============================================
  Hits          27116    27116              
- Misses        35057    35060       +3
Impacted Files Coverage Δ
src/ice_shelf/MOM_ice_shelf_dynamics.F90 0% <0%> (ø) ⬆️
src/core/MOM_barotropic.F90 71.88% <0%> (-0.04%) ⬇️

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 9a44252...afb932a. Read the comment docs.

Copy link
Collaborator

@Hallberg-NOAA Hallberg-NOAA left a comment

Choose a reason for hiding this comment

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

These modest changes are clearly correct.

Copy link
Collaborator

@adcroft adcroft left a comment

Choose a reason for hiding this comment

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

@adcroft adcroft merged commit c43ffaa into mom-ocean:dev/gfdl Nov 8, 2019
@marshallward marshallward deleted the variable_init_fixes branch February 13, 2020 16:39
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.

4 participants