-
Notifications
You must be signed in to change notification settings - Fork 65
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 #35
Ice dynamics #35
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.
Codecov Report
@@ Coverage Diff @@
## dev/gfdl #35 +/- ##
============================================
- Coverage 28.97% 28.96% -0.02%
============================================
Files 240 240
Lines 71185 71212 +27
============================================
Hits 20623 20623
- Misses 50562 50589 +27
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.
Thank you for these updates. Overall they look like significant improvements. I would be happy to approve these, but first there are a handful of specific comments (already added to the conversation that I would like you to consider. Each of these changes should be easy enough to make, and they should not adversely impact any working test cases, but they will avoid possible problems later and facilitate more extensive testing.
…elocity derivatives rotation-invariant. Changes in `update_ice_shelf()` utilize G%IareaT(:,:) instead of 1/G%areaT(:,:).
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.
Thanks for making the updates. These code changes now look good to me, and they should be merged in once the pipeline and TC tests have passed.
This PR has passed TC testing and passed pipeline testing at https://gitlab.gfdl.noaa.gov/ogrp/MOM6/-/pipelines/14370. |
In this PR major changes are made in MOM_ice_shelf_dynamics.F90 to
calc_shelf_visc()
andcalc_shelf_driving_stress()
to make them work properly on an irregular quadrilateral grid. In MOM_ice_shelf_initialize.F90 changes are made to initialize_ice_thickness_from_file() to correct masks at initialization.