-
Notifications
You must be signed in to change notification settings - Fork 66
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
Passing shelf_sfc_mass_flux to ice shelf #818
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are a few issues that I would to see addressed before we could merge in this useful new field.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev/gfdl #818 +/- ##
============================================
+ Coverage 38.15% 38.17% +0.01%
============================================
Files 296 296
Lines 87254 87471 +217
Branches 16288 16389 +101
============================================
+ Hits 33296 33388 +92
- Misses 47971 48076 +105
- Partials 5987 6007 +20 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am convinced that this updated PR is correctly implemented.
* Removed second registration of sfc_mass_flux with register_diag_field. * Registered sfc_mass_flux to runs with static and dynamic ice sheets.
…gister_diag_field()'
…added back and 'register_restart_field()' of 'fluxes_in%shelf_sfc_mass_flux' is moved before 'restore_state()'
9ad5539
to
2f06732
Compare
This PR has passed pipeline testing at https://gitlab.gfdl.noaa.gov/ogrp/mom6ci/MOM6/-/pipelines/26435. |
This is the first PR from gfdl-cryopshere group. It includes several changes related to the surface mass flux from the land to the ice sheet,
shelf_sfc_mass_flux
. It is now defined in MOM_surface_forcing_gfdl.F90 and its values are passed tofluxes%shelf_sfc_mass_flux
. This field can be used in configurations with a static ice sheet. To accommodate that it is now registered as a diagnostic field withCS%active_shelf_dynamics=.false.
.