-
Notifications
You must be signed in to change notification settings - Fork 244
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
Comments
When compiling with "-check all,noarg_temp_created,nopointer", the GFS forecast can run until the wall clock is reached. |
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. |
@bensonr <https://github.com/bensonr> We may still need to allocate the
dtdt_m without the if statement of "if ( idiag%id_mdt > 0 .and.
(.not.do_adiabatic_init) ) then" (line #384 in
FV3/atmos_cubed_sphere/model/fv_dynamics.F90), if there is no other
simpler way to fix the error.
…On Wed, Mar 10, 2021 at 10:05 AM Dom Heinzeller ***@***.***> wrote:
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
<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 <https://github.com/bensonr> last
week.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<#455 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AI7D6TMVIIFGTR2AASKVMGLTC54BVANCNFSM4YZ2XD6A>
.
|
The code changes (-check all,noarg_temp_created,nopointer) was committed to production branch, close this issue. |
…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>
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.
The text was updated successfully, but these errors were encountered: