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

Fix iceberg melt temperature #123

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

cbegeman
Copy link
Collaborator

@cbegeman cbegeman commented Mar 4, 2025

Currently:

  • MPAS-Seaice iceberg heat flux term includes both latent heat associated with melting and the heat needed to raise icebergs from -4 degC to 0 degC
  • MPAS-Ocean assumes that iceberg mass flux entered at the local freezing point, which is inconsistent with the temperature adjustment above

With this PR:

  • MPAS-Seaice iceberg heat flux term includes only the latent heat associated with melting
  • MPAS-Ocean assumes that iceberg mass flux entered at the local freezing point, and includes the heat flux needed to raise icebergs from a namelist-derived initial temperature (default -4 degC) to the local freezing point

Local freezing point is the pressure and salinity-dependent freezing point consistent with mushy thermodynamics.

@cbegeman
Copy link
Collaborator Author

cbegeman commented Mar 4, 2025

Testing

The e3sm_cryo_developer suite has been run on chrys with expected diffs.
The e3sm_prod suite has been run on chrys with diffs only in the SMS_D_Ld1.ne30pg2_r05_IcoswISC30E3r5.CRYO1850.chrysalis_intel case for which config_remove_ais_ice_runoff = .true.

Standalone testing planned for verification but on hold given issues in compass and polaris environments

@cbegeman cbegeman added the bug Something isn't working label Mar 4, 2025
@cbegeman cbegeman self-assigned this Mar 4, 2025
@cbegeman cbegeman requested review from xylar and darincomeau March 4, 2025 23:46
@cbegeman
Copy link
Collaborator Author

cbegeman commented Mar 4, 2025

@darincomeau and @xylar You can take a look now or just wait until I post the standalone verification test. Thanks for your input on this!

@darincomeau
Copy link
Collaborator

Thanks @cbegeman for putting this together. Agree that prescribed iceberg temperature should be a namelist option.

I'll make a note here that snowcapping iceRunoff is assumed by MPAS-Ocean to be 0 degC coming in (solid), and enters the ocean at 0 degC (liquid). Once we hook up MALI calving fluxes to this field, then we'll have an inconsistency in how icebergs are treated with data icebergs and solid runoff icebergs. Not anything to fix now (changing snowcapping solid ice runoff temperature would be presumably quite disruptive to the larger project), but just something to keep in our minds (especially if we consider down the road having MALI send temperature of calving fluxes).

@cbegeman
Copy link
Collaborator Author

cbegeman commented Mar 6, 2025

A preliminary look into temperatures at the base of the snowpack suggested that the mean temperature of ice runoff could be significantly lower than -4 degC. I used a single_column test to evaluate the potential impacts of reducing the iceberg temperature from -4 degC to -14 degC and it could result in significant mixed layer deepening at the limit of frazil formation and no frazil ice porosity (along with more frazil formation). So we should keep an eye out for this with future B-case tests.

That result didn't feel right so I found a bug and now here are the differences in the frazil limit:
this branch with iceberg T=-4 degC vs. master: -0.1% frazil formation rate
this branch with iceberg T=-14 degC vs. master: +0.5% frazil formation rate
this branch with iceberg T=-14 degC vs. this branch with iceberg T=-4 degC: +0.7% frazil formation rate

@cbegeman cbegeman force-pushed the fix-iceberg-melt-temperature branch from 8ae86a6 to b8158fb Compare March 6, 2025 18:19
@cbegeman cbegeman marked this pull request as ready for review March 6, 2025 18:20
Copy link
Collaborator

@xylar xylar left a comment

Choose a reason for hiding this comment

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

This looks ready for E3SM as far as I'm concerned. Anything else you want to do before we move over?

@darincomeau
Copy link
Collaborator

We probably want a bug report issue for this since it's a bug fix PR?

@cbegeman
Copy link
Collaborator Author

This PR addresses E3SM-Project#7110

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants