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

Bugfixes for icepack cleanup in Soca2Cice #1118

Merged
merged 2 commits into from
Jan 22, 2025
Merged

Conversation

shlyaeva
Copy link
Contributor

Description

Bugfixes for icepack cleanup (#1117) for tracers (ice/snow temperature, ice and snow enthalpies, ice salinity):

  • fixes problems with tracer index data passed to cleanup_itd (no need to compare to the previous code here, it's all wrong; the new code follows roughly what's done in icepack here (you'll also see why I don't want to deal with melt ponds right now :))
  • also fixes the cancelling bug: tracers updated after cleanup_itd are now copied back to analysis.

This time, besides running one high res forecast (it didn't fail), I also looked at how ice/snow temperature, ice and snow enthalpies and ice salinity changed with cleanup (spoiler alert: the code no longer generate 90C ice temperatures just to ignore them later). Cleanup redistributes ice between the categories if needed (and updates tracers accordingly), and zaps ice with small areas and snow when snow enthalpies are inconsistent.

All of the below plots are showing update to different ice categories from just cleanup (after the analysis increment was added); so the ice concentration difference is just redistribution of ice between the categories. Note that it's the first cycle and background is quite far from analysis, so the increment was quite large in a lot of areas, so there's a lot to redistribute.

cice_inc_aice
cice_inc_qsnow
cice_inc_tmp

Issue(s) addressed

Fixes #1117

Testing

hera/intel with soca tests (I needed an update to tolerance similar to #1116 for the test to pass on develop) and one analysis/forecast for high-res gdas experiments.

Copy link
Contributor

@guillaumevernieres guillaumevernieres left a comment

Choose a reason for hiding this comment

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

👍

@travissluka travissluka merged commit fc5ad34 into develop Jan 22, 2025
2 checks passed
@travissluka travissluka deleted the bugfix/icepack_cleanup branch January 22, 2025 20:14
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.

bugs in soca2cice call of icepack cleanup
3 participants