-
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
Ice dynamics #80
Ice dynamics #80
Conversation
…nd calc_shelf_driving_stress() to account for an irregular quadrilateral grid. In MOM_ice_shelf_initialize.F90 changes are made to initialize_ice_thickness_from_file() to correct masks at initialization.
…elocity derivatives rotation-invariant. Changes in `update_ice_shelf()` utilize G%IareaT(:,:) instead of 1/G%areaT(:,:).
Ice shelf dynamic sfc accum
Conditional allocation of ice shelf fluxes and data override call
call data override init for ice shelves
…kness in response to the atmospheric mass flux
- Introduces a new flag: DATA_OVERRIDE_SHELF_FLUXES, which if set to True, and ACTIVE_SHELF_DYNAMICS is also True, will enable data_override capability for the surface mass deposition (only avaialble for MOSAIC grid types)
Conditional calls to data_override
…o avoid negative values at the southern boundary of the domain. This is done for grids extending over Antarctica.
Codecov Report
@@ Coverage Diff @@
## dev/gfdl #80 +/- ##
============================================
- Coverage 28.98% 28.96% -0.03%
============================================
Files 245 245
Lines 72193 72240 +47
============================================
- Hits 20927 20925 -2
- Misses 51266 51315 +49
Continue to review full report at Codecov.
|
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.
Overall, this PR looks good, but there are 4 specific outstanding conversations about this PR that will need to be addressed, almost certainly with minor specific code changes, before this PR should be accepted.
…e_shelf_initialize.F90 and modified the four halo updates in MOM_ice_shelf.F90.
This PR should be ready to go once Matt's PR back to OlgaSergienko:ice_dynamics at OlgaSergienko#10 is accepted. |
Ice dynamics patch matt
Matt's PR has been merged and all requested changes have been addressed. |
@OlgaSergienko @MJHarrison-GFDL Is a squash-merge ok here? It will wipe out your commit histories. Otherwise, one of you will need to clean up the rebase conflicts. |
Gaea regression: https://gitlab.gfdl.noaa.gov/ogrp/MOM6/-/pipelines/15042 ✔️ |
This PR includes major changes that enable a new flux with ocean_only runs - the surface mass flux over Antarctica (
shelf_sfc_mass_flux
). Changes have been made to MOM_forcing_type.F90 and MOM_ice_shelf.F90.