Skip to content

Update tf_thaw_set in global_cycle - #1155

Merged
BrianCurtis-NOAA merged 2 commits into
ufs-community:developfrom
TravisElless-NOAA:mait/add_is_ice
May 5, 2026
Merged

Update tf_thaw_set in global_cycle#1155
BrianCurtis-NOAA merged 2 commits into
ufs-community:developfrom
TravisElless-NOAA:mait/add_is_ice

Conversation

@TravisElless-NOAA

@TravisElless-NOAA TravisElless-NOAA commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

DESCRIPTION OF CHANGES:

Compiling with debug mode and running tests for the GCAFS NCO stability tests, highlighted a failure for an undefined is_ice variable in the tf_thaw_set subroutine of sorc/global_cycle.fd/cycle.F90. This PR adds a default value of is_ice=.false. to the tf_thaw_set subroutine. This line is already included in the ADJUST_NSST subroutine which uses a similar algorithm.

TESTING

WHICH CONSISTENCY TESTS TO RUN:

  • global_cycle

TESTS CONDUCTED:

  • Compile branch using Intel:
    • Orion
    • Ursa
    • Hercules
    • WCOSS2
  • Compile branch using GNU:
    • Ursa
  • Compile branch in 'Debug' mode:
    • WCOSS2
  • Compile with Doxygen with no errors.
  • Run unit tests on any Tier 1 machine.
  • Run relevant consistency tests:
    • Orion
    • Ursa
    • Hercules
    • WCOSS2

DOCUMENTATION:

  • Doxygen does not need updates.

LINKED ISSUES:

Resolves #1151

@TravisElless-NOAA TravisElless-NOAA changed the title add is_ice Update tf_thaw_set in global_cycle Apr 24, 2026
@BrianCurtis-NOAA

Copy link
Copy Markdown
Collaborator

Good afternoon Travis, could you bump to the latest version of develop branch?

@BrianCurtis-NOAA
BrianCurtis-NOAA merged commit 1ca6b15 into ufs-community:develop May 5, 2026
3 of 4 checks passed
DavidHuber-NOAA pushed a commit to DavidHuber-NOAA/UFS_UTILS that referenced this pull request Jun 24, 2026
BrianCurtis-NOAA pushed a commit that referenced this pull request Jun 30, 2026
Co-authored-by: Travis Elless <113720457+TravisElless-NOAA@users.noreply.github.com>
DavidHuber-NOAA pushed a commit to NOAA-EMC/global-workflow that referenced this pull request Jul 6, 2026
#5088)

This PR updates the UFS_UTILS hash to include
ufs-community/UFS_UTILS#1155 in the dev/gfs.v17
branch. This resolves a global_cycle failure in debug mode found during
GCAFS review.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

GCAFS stability test failures in global_cycle/cycle.f90

2 participants