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

point to icepack consortium master #39

Closed
DeniseWorthen opened this issue Aug 27, 2021 · 0 comments · Fixed by ufs-community/ufs-weather-model#765
Closed

point to icepack consortium master #39

DeniseWorthen opened this issue Aug 27, 2021 · 0 comments · Fixed by ufs-community/ufs-weather-model#765

Comments

@DeniseWorthen
Copy link
Collaborator

DeniseWorthen commented Aug 27, 2021

NOAA-EMC CICE currently points to an EMC fork of Icepack because the non-benchmark IC for the current coupled regression tests would abort at this section of icepack/columnphysics/icepack_therm_itd.F90.

              write(warnstr,*) subname, &
                 'ITD Thermodynamics: hicen_init(n+1) <= hicen_init(n)'
              call icepack_warnings_add(warnstr)
              call icepack_warnings_setabort(.true.)

NOAA-EMC has maintained a version with icepack_warnings_setabort(.false.) to prevent the code aborting on the first timestep.

Testing of the new IC as part of ufs-weather-model PR #765 has shown this flag is no longer required to be set .false. to prevent failures on the initial timestep.

The flag should be set to true, because that will indicate that this specific condition is causing the model to abort.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant