Skip to content

branch with 5.3.05Y: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided #2844

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 59 commits into from
Jun 14, 2025
Merged

branch with 5.3.05Y: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided #2844

merged 59 commits into from
Jun 14, 2025

Conversation

ekluzek
Copy link
Collaborator

@ekluzek ekluzek commented Oct 23, 2024 •
edited
Loading

Description of changes

Changes in the FORTRAN code to properly abort when fire-emission is asked for it can't be provided.

To develop this a PF unit test for CNFireFactoryMod is added in.

Specific notes

Contributors other than yourself, if any:

CTSM Issues Fixed (include github issue #):
Fixes #2762
Fixes #3073
CTSM namelist checking for: NGEET/fates#1356
Some work on #2643

Are answers expected to change (and if so in what way)? No

Any User Interface Changes (namelist or namelist defaults changes)? No

Does this create a need to change or add documentation? Did you do so? No

Testing performed, if any: Currently just the PF unit tests will run regular testing when done.

@ekluzek ekluzek added enhancement new capability or improved behavior of existing capability code health improving internal code structure to make easier to maintain (sustainability) bfb bit-for-bit usability Improve or clarify user-facing options labels Oct 23, 2024
@ekluzek ekluzek added this to the cesm3_0_beta05 milestone Oct 23, 2024
@ekluzek ekluzek self-assigned this Oct 23, 2024
@ekluzek ekluzek marked this pull request as draft October 23, 2024 17:12
@ekluzek ekluzek linked an issue Nov 5, 2024 that may be closed by this pull request
Don't allow SP, fates, or nofire tests to turn fire_emis on (and make sure our tests don't do this either) #2762
Closed
4 tasks
ekluzek added 13 commits February 6, 2025 15:15
@ekluzek
Merge remote-tracking branch 'escomp/cesm3_0_beta04_changes' into fir…
69e1ccf 
…e_emis
@ekluzek
Change to full ESMF library
2c330c0
@ekluzek
Add tests for each of the Li fire types
6d0f1e5
@ekluzek
Add LAI stream to compile list
3d81a1b
@ekluzek
Add check_dim_size
9f7527e
@ekluzek
Add FireClean method
e022783
@ekluzek
Make pointers set to NULL, and add BaseFireClean method
29b5b61
@ekluzek
Add shr_expr and fire_emis to list
61826eb
@ekluzek
Add more setup and teardown, and also add several more tests, and inc…
de0c358 
…lude tests that test that fire emission on with nofire or SP fail, which right now they don't as expected
@ekluzek
Initialize pointer to NULL, add clean method, add some returns after …
d4707b6 
…endrun calls for unit testing
@ekluzek
Add a note about the call to the base BaseFireClean method failing at…
fad3d1b 
… compile
@ekluzek
Get one of the failing tests working by implementing an init method t…
2fcc081 
…hat checks that fire-emission is off for fire_method==nofire
@ekluzek
Get the last test working by aborting if fire-emissions are on in SP …
39661d6 
…mode
@wwieder wwieder moved this from In progress - master/b4b-dev to In progress - b4b-dev in CTSM: Upcoming tags Apr 17, 2025
@samsrabin samsrabin moved this from In progress - b4b-dev to In progress - master in CTSM: Upcoming tags Apr 17, 2025
@wwieder wwieder moved this from In progress - master to In progress - b4b-dev in CTSM: Upcoming tags Apr 17, 2025
@wwieder wwieder moved this from In progress - b4b-dev to In progress - master in CTSM: Upcoming tags Apr 17, 2025
@wwieder wwieder changed the title branch: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided branch with 5.3.0141: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided Apr 17, 2025
@wwieder wwieder changed the title branch with 5.3.0141: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided branch with 5.3.041: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided Apr 17, 2025
@ekluzek ekluzek mentioned this pull request Apr 18, 2025
Open
@ekluzek ekluzek changed the title branch with 5.3.041: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided branch with 5.3.042: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided Apr 21, 2025
@ekluzek
Copy link
Collaborator Author

ekluzek commented Apr 21, 2025

Error caught on Izumi for initialization with:

SMS_Ld10_D_Mmpi-serial.CLM_USRDAT.I1PtClm60Fates.izumi_nag.clm-FatesPRISM--clm-NEON-FATES-YELL.GC.ctsm5341acl_nag

This is partially due to what I point out in #3078

 (t_initf)       profile_papi_enable=      F
Runtime Error: /fs/cgd/data0/erik/ctsm_worktree/quickfix/src/biogeochem/CNVegetationFacade.F90, line 1019: ALLOCATABLE variable is not currently allocated
Program terminated by fatal error
/fs/cgd/data0/erik/ctsm_worktree/quickfix/src/biogeochem/CNVegetationFacade.F90, line 1019: Error occurred in CNVEGETATIONFACADE:ECOSYSTEMDYNAMICSPREDRAINAGE
/fs/cgd/data0/erik/ctsm_worktree/quickfix/src/main/clm_driver.F90, line 1020: Called by CLM_DRIVER:CLM_DRV
/fs/cgd/data0/erik/ctsm_worktree/quickfix/src/cpl/nuopc/lnd_comp_nuopc.F90, line 899: Called by LND_COMP_NUOPC:MODELADVANCE
/fs/cgd/data0/erik/ctsm_worktree/quickfix/components/cmeps/cime_config/../cesm/driver/esmApp.F90, line 141: Called by ESMAPP

@ekluzek
Copy link
Collaborator Author

ekluzek commented Apr 21, 2025

The izumi tests that have the problem above are:

ERI_D_Ld20.f10_f10_mg37.I2000Clm50Fates.izumi_nag.clm-FatesCold (RUN)
ERI_D_Ld20.f45_f45_mg37.I2000Clm50FatesRs.izumi_nag.clm-FatesColdTwoStream (RUN)
ERS_D_Ld5.f10_f10_mg37.I2000Clm50Fates.izumi_nag.clm-FatesCold (RUN)
ERS_D_Mmpi-serial_Ld5.1x1_brazil.I2000Clm50FatesRs.izumi_nag.clm-FatesCold (RUN)
SMS_D_Ld5.f10_f10_mg37.I2000Clm50FatesRs.izumi_nag.clm-FatesCold (RUN)
SMS_D_Ld5.f45_f45_mg37.I2000Clm60Fates.izumi_nag.clm-FatesCold (RUN)
SMS_D_Mmpi-serial_Ld5.5x5_amazon.I2000Clm60FatesCrujraRs.izumi_nag.clm-FatesCold (RUN)
SMS_D_Mmpi-serial_Ld5.5x5_amazon.I2000Clm60FatesRs.izumi_nag.clm-FatesCold (RUN)

There were some build fails, that I've been successful with just resending the build with:

ERP_D_Ld5_P48x1.f10_f10_mg37.I1850Clm50Bgc.izumi_nag.clm-ciso (SHAREDLIB_BUILD)
ERP_D_Ld5_P48x1.f10_f10_mg37.I1850Clm60Bgc.izumi_nag.clm-ciso (SHAREDLIB_BUILD)
ERP_D_Ld5_P48x1.f10_f10_mg37.I1850Clm60Bgc.izumi_nag.clm-ciso--clm-matrixcnOn (SHAREDLIB_BUILD)
ERP_D_Ld5_P48x1.f10_f10_mg37.I2000Clm50BgcCru.izumi_nag.clm-flexCN_FUN (SHAREDLIB_BUILD)
ERP_D_Ld5_P48x1.f10_f10_mg37.I2000Clm50BgcCru.izumi_nag.clm-flexCN_FUN--clm-matrixcnOn (SHAREDLIB_BUILD)
ERP_D_Ld5_P48x1.f10_f10_mg37.I2000Clm50BgcCru.izumi_nag.clm-flexCN_FUN_BNF (SHAREDLIB_BUILD)
ERP_D_Ld5_P48x1.f10_f10_mg37.I2000Clm50BgcCru.izumi_nag.clm-luna (SHAREDLIB_BUILD)
ERP_D_Ld5_P48x1.f10_f10_mg37.I2000Clm50BgcCru.izumi_nag.clm-noFUN_flexCN (SHAREDLIB_BUILD)
ERP_D_Ld5_P48x1.f10_f10_mg37.I2000Clm50BgcCru.izumi_nag.clm-noFUN_flexCN--clm-matrixcnOn (SHAREDLIB_BUILD)
ERP_D_Ld5_P48x1.f10_f10_mg37.I2000Clm50BgcCru.izumi_nag.clm-reduceOutput (SHAREDLIB_BUILD)
ERP_D_Ld5_P48x1.f10_f10_mg37.I2000Clm50Sp.izumi_nag.clm-o3lombardozzi2015 (SHAREDLIB_BUILD NLCOMP)
ERP_D_Ld9.f10_f10_mg37.I1850Clm60BgcCrop.izumi_nag.clm-clm60cam7LndTuningModeLDust (SHAREDLIB_BUILD)
ERP_D_P48x1.f10_f10_mg37.IHistClm60Bgc.izumi_nag.clm-decStart (SHAREDLIB_BUILD)
ERP_D_P48x1.f10_f10_mg37.IHistClm60Bgc.izumi_nag.clm-decStart--clm-matrixcnOn_ignore_warnings (SHAREDLIB_BUILD)

@ekluzek ekluzek changed the title branch with 5.3.042: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided branch with 5.3.044: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided Apr 29, 2025
@ekluzek ekluzek moved this from In Progress to Done in LMWG: Sprint Planning Board Apr 29, 2025
@ekluzek ekluzek changed the title branch with 5.3.044: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided branch with 5.3.046: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided May 7, 2025
ekluzek added 3 commits May 9, 2025 15:25
@ekluzek
Fix a few more tests that need to use nofireemis
09c06a8
@ekluzek
Merge tag 'ctsm5.3.043' into fire_emis
a576300 
Merge b4bdev 20250509

Updates to the b4b-dev branch since its last merge to master (PRs ESCOMP#3091 ESCOMP#3092), as shown by git log:

- Merge pull request Update docs infrastructure ESCOMP#2809 from samsrabin/update-docs-builder-2
  Update docs infrastructure
- Merge pull request Update externals to cesm3_0_alpha06c ESCOMP#3106 from ekluzek/update_to_alpha06c
  Update externals to cesm3_0_alpha06c
- Merge pull request User control over snow thermal conductivity scheme over glaciers ESCOMP#3072 from wwieder/JordanGlacier
  User control over snow thermal conductivity scheme over glaciers

 Conflicts:
	src/biogeochem/CNFireNoFireMod.F90
@ekluzek
Allocate cnfire_method for fates as nofire, because it's passed down …
38798b0 
…to subroutines that FATES needs
@ekluzek ekluzek changed the title branch with 5.3.046: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided branch with 5.3.04Y: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided May 12, 2025
@ekluzek ekluzek changed the title branch with 5.3.04Y: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided branch with 5.3.05Y: Make sure FORTRAN code properly aborts when fire-emission is asked for and it can't be provided May 22, 2025
ekluzek added 4 commits June 2, 2025 01:29
@ekluzek
Merge tag 'ctsm5.3.053' into fire_emis
607b96b 
Fix and improve anomaly forcings for ISSP cases

 Conflicts:
	bld/unit_testers/build-namelist_test.pl
	src/biogeochem/SatellitePhenologyMod.F90
@ekluzek
Fix more tests with sp that need nofireemis
11f313d
@ekluzek
Another nofireemis change
f8ad8a1
@ekluzek
Merge branch 'cesm3_0_beta04_changes' into fire_emis
99aed55
@ekluzek
Copy link
Collaborator Author

ekluzek commented Jun 14, 2025

Tests are now all passing as expected.

So I'll merge this to the branch, and that merge will take care of this.

@ekluzek ekluzek merged commit 61955c5 into ESCOMP:cesm3_0_beta04_changes Jun 14, 2025
4 checks passed
@github-project-automation github-project-automation bot moved this from In progress - master to Done (non release/external) in CTSM: Upcoming tags Jun 14, 2025
@ekluzek ekluzek deleted the fire_emis branch June 14, 2025 23:32
@ekluzek ekluzek mentioned this pull request Jun 14, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wwieder wwieder wwieder approved these changes

@slevis-lmwg slevis-lmwg slevis-lmwg left review comments

@glemieux glemieux Awaiting requested review from glemieux

@adrifoster adrifoster Awaiting requested review from adrifoster

Assignees

@ekluzek ekluzek

Labels
bfb bit-for-bit code health improving internal code structure to make easier to maintain (sustainability) enhancement new capability or improved behavior of existing capability usability Improve or clarify user-facing options
Projects
CTSM: Upcoming tags
Status: Done (non release/external)
LMWG: Sprint Planning Board
Status: Done
Milestone
CESM3 Answer changing freeze
Development

Successfully merging this pull request may close these issues.

Don't allow SP, fates, or nofire tests to turn fire_emis on (and make sure our tests don't do this either)
4 participants
@ekluzek @wwieder @slevis-lmwg @samsrabin