Skip to content

Initialize use_ice_shelf in ALE_regridding_and_remapping#872

Merged
Hallberg-NOAA merged 4 commits intoNOAA-GFDL:dev/gfdlfrom
JosephMouallem:dev/gfdl
Apr 21, 2025
Merged

Initialize use_ice_shelf in ALE_regridding_and_remapping#872
Hallberg-NOAA merged 4 commits intoNOAA-GFDL:dev/gfdlfrom
JosephMouallem:dev/gfdl

Conversation

@JosephMouallem
Copy link
Copy Markdown

use_ice_shelf is used in ALE_regridding_and_remapping without being defined and causes a model crash in debug mode.
This PR initialize use_ice_shelf = .false.

Copy link
Copy Markdown
Member

@adcroft adcroft left a comment

Choose a reason for hiding this comment

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

Thank you - good catch. It looks like we lost the pathway to change use_ice_shelf at all, which might explain some recent problems we've been having with the ice-shelf!

@theresa-cordero
Copy link
Copy Markdown

The code to set use_ice_shelf was if (associated(CS%frac_shelf_h)) use_ice_shelf = .true. that could be added where you have initialized it to false and that should fix the issue Alistair pointed out.

adcroft added 2 commits April 21, 2025 14:00
Recovers the conditional code that sets use_ice_shelf in ALE_regridding_and_remapping()
@adcroft
Copy link
Copy Markdown
Member

adcroft commented Apr 21, 2025

@Hallberg-NOAA
Copy link
Copy Markdown
Member

This PR has passed pipeline testing at https://gitlab.gfdl.noaa.gov/ogrp/mom6ci/MOM6/-/pipelines/27194.

@Hallberg-NOAA Hallberg-NOAA merged commit 24e20dc into NOAA-GFDL:dev/gfdl Apr 21, 2025
51 checks passed
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.

5 participants