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

+(*)Report heat stocks relative to liquid at 0 degC #163

Merged
merged 1 commit into from
Jan 11, 2022

Conversation

Hallberg-NOAA
Copy link
Member

Revised ice_stock_pe() to report the heat stock relative to liquid water at 0
degrees C, rather than just the heat required to melt the ice, as this gives a
much more robust estimates of the enthalpy budget when sea ice salinities are
allowed to vary. To help implement this, the new function energy_0degC was
added to SIS2_ice_thm.F90. Several module use statements that were referring to
unused routines from within SIS2_ice_thm were also redacted. All solutions are
bitwise identical, but there are changes in the reported stock values, and there
is a new publicly visible function.

  Revised ice_stock_pe() to report the heat stock relative to liquid water at 0
degrees C, rather than just the heat required to melt the ice, as this gives a
much more robust estimates of the enthalpy budget when sea ice salinities are
allowed to vary.  To help implement this, the new function energy_0degC was
added to SIS2_ice_thm.F90.  Several module use statements that were referring to
unused routines from within SIS2_ice_thm were also redacted.  All solutions are
bitwise identical, but there are changes in the reported stock values, and there
is a new publicly visible function.
Copy link
Contributor

@MJHarrison-GFDL MJHarrison-GFDL left a comment

Choose a reason for hiding this comment

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

In addition to some code cleanup and documentation, this change will allow for a more accurate accounting of enthalpy exchange between the sea-ice and ocean. Answers are not impacted, but as noted the stocks report (which is used within the FMS coupling framework) will change but these results are not being tracked in current MOM6 regression tests

@Hallberg-NOAA Hallberg-NOAA merged commit 94ac8bd into NOAA-GFDL:dev/gfdl Jan 11, 2022
@Hallberg-NOAA Hallberg-NOAA deleted the stock_0degC branch August 26, 2022 20:01
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.

2 participants