-
Notifications
You must be signed in to change notification settings - Fork 0
Modify land ice melt equations #136
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
base: master
Are you sure you want to change the base?
Modify land ice melt equations #136
Conversation
|
@xylar Do you want to take a quick look at the code before I proceed with testing and let me know if this looks like what we agreed upon? |
components/mpas-ocean/src/shared/mpas_ocn_surface_land_ice_fluxes.F
Outdated
Show resolved
Hide resolved
xylar
left a comment
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.
Let's chat about this when we meet today. I think the units as written don't quite work out.
components/mpas-ocean/src/shared/mpas_ocn_surface_land_ice_fluxes.F
Outdated
Show resolved
Hide resolved
components/mpas-ocean/src/shared/mpas_ocn_surface_land_ice_fluxes.F
Outdated
Show resolved
Hide resolved
6efe1c7 to
496cdd1
Compare
|
@xylar I think I addressed your comments with the latest commits. Resolve if you're happy. Still not yet tested. |
xylar
left a comment
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 think this looks good. Just one issue I'd like to resolve with 2 confusing local variable names.
a56885e to
4367695
Compare
…xToLandIce is now the total flux between ocn and glc
…ription and other coupler variables
b0c0d54 to
49d3b44
Compare
|
@xylar Just noting that I changed the sign of the advective heat flux for all melt routines. E.g., for the ISOMIP+ test case, |
TestingKey points:
Conservation statistics are daily; reported for 0001-02-01 B-case, this branch, conductive flux B-case, this branch, no conductive flux B-case, master |
This PR includes two changes to the 3-equation parameterization for ice-shelf basal melting in MPAS-Ocean:
iceTemperatureto the interface temperature (mistakenly assigned to the freezing point at local pressure and zero salinity).cterm of the quadratic equation. (If it depended on the interface temperature it would need to be added to thebterm.)