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

production/GFSv16: check all compile option failed for GFSv16 atmospere model #455

Closed
junwang-noaa opened this issue Mar 8, 2021 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@junwang-noaa
Copy link
Collaborator

junwang-noaa commented Mar 8, 2021

Description

NCO requires to test "-check all" option with GFSv16 optimization mode (FFLAGS_OPT). GDAS ENKF, GADS forecast, GFS forecast Model failed with the executable built with the "-check all" option.

To Reproduce:

Checked out the GFS.v16.0.15 tag and add "-check all,noarg_temp_created" in FFLAGS_OPT compile options, then run GFS parallel test on wcoss, the run will fail with error message:
"forrtl: severe (408): fort: (8): Attempt to fetch from allocatable variable DTDT_M when it is not allocated
ERR=152"

bugzilla ticket is #1204.

Fixes will be made to production/GFSv16 branch for future GFS minor upgrades.

@junwang-noaa junwang-noaa added the bug Something isn't working label Mar 8, 2021
@junwang-noaa junwang-noaa added this to the GFSv16 bug fixes milestone Mar 8, 2021
@junwang-noaa
Copy link
Collaborator Author

When compiling with "-check all,noarg_temp_created,nopointer", the GFS forecast can run until the wall clock is reached.

@junwang-noaa junwang-noaa self-assigned this Mar 9, 2021
@climbfuji
Copy link
Collaborator

This dtdt_m issue was raised more than a year ago in the NOAA-EMC GFDL_atmos_cubed_sphere fork: NOAA-EMC/GFDL_atmos_cubed_sphere#29

I closed the issue after removing the IPD code, because the CCPP code doesn't have the problem, and since issues generally related to dev/emc on that fork. But GFSv16 is using IPD, which still has the issue.

I also discussed this with @bensonr last week.

@junwang-noaa
Copy link
Collaborator Author

junwang-noaa commented Mar 10, 2021 via email

@junwang-noaa
Copy link
Collaborator Author

junwang-noaa commented Apr 23, 2021

The code changes (-check all,noarg_temp_created,nopointer) was committed to production branch, close this issue.

pjpegion pushed a commit to NOAA-PSL/ufs-weather-model that referenced this issue Apr 4, 2023
…files (ufs-community#455)

* Fix uninitialized variable zmtnblck in ccpp/data/GFS_typedefs.F90
* Fix typo in CCPP standard name for ncnvwind in ccpp/data/GFS_typedefs.meta
* Add ten 2d variables required for MYNNSFC restart reproducibility to ccpp/driver/GFS_restart.F90
* create initial p8 suites, P8 initial SDFs as copies of FV3_GFS_v16_coupled_nsstNoahmpUGWPv1
and FV3_GFS_v16_nsstNoahmpUGWPv1

Co-authored-by: Denise Worthen <denise.worthen@noaa.gov>
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

No branches or pull requests

2 participants