Skip to content

Conversation

@ekluzek
Copy link
Collaborator

@ekluzek ekluzek commented Oct 15, 2025

Description of changes

Only turn C13/C14 for clm6_0 and BGC, and also require the DATM forcing to be CRUJRA2024 or CAM7 and for the historical period. So for example SSP compsets won't have C13/C14 on, single point compsets won't, or for using older forcing (like CAM6, or GSWP3v1, Qian, or CRUv7). Also whenever use_c13 or use_c14 is on turn on the corresponding time series file.

Also change default for irrigate to on for historical SP/BGC cases.

Specific notes

Contributors other than yourself, if any: @wwieder @slevis-lmwg

CTSM Issues Fixed (include github issue #):
Fixes #3527
Fixes #3551
Some work on #3346

Are answers expected to change (and if so in what way)? Yes
by default C13/C14 won't be on as much
Irrigation will be on for historical SP/BGC cases

Any User Interface Changes (namelist or namelist defaults changes)?
Namelist defaults change for use_c13/use_c14 so only on for clm6_0 Bgc, for CRUJRA2024 or CAM7 forcing historical period.
Name default changes for clm60 for HistClm60Sp/HistClm60Bgc compsets so irrigation is on

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

Testing performed, if any: So far running build-namelist tests. Will run aux_clm also

@ekluzek ekluzek requested a review from slevis-lmwg October 15, 2025 04:11
@ekluzek ekluzek self-assigned this Oct 15, 2025
@ekluzek ekluzek added enhancement new capability or improved behavior of existing capability usability Improve or clarify user-facing options non-bfb Changes answers (incl. adding tests) labels Oct 15, 2025
@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 15, 2025

Whoops this ended up including changes on b4b-dev again. So I might need to redo this and create a new PR...

@ekluzek ekluzek force-pushed the ciso_namelist_default_changes branch from 0de4339 to d8c3f53 Compare October 15, 2025 04:16
@github-project-automation github-project-automation bot moved this to Ready to start (or start again) in CTSM: Upcoming tags Oct 15, 2025
@ekluzek ekluzek moved this from Ready to start (or start again) to In progress - master in CTSM: Upcoming tags Oct 15, 2025
@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 15, 2025

One unexpected fail:

ERP_D_P64x2_Ld3.f10_f10_mg37.I1850Clm50BgcCrop.derecho_intel.clm-default--clm-matrixcnOn_ignore_warnings

in the sparse matrix. We've seen this before.

dec0471.hsn.de.hpc.ucar.edu 8:  ERROR in SparseMatrixMultiplyMod.F90 at line 973
dec0471.hsn.de.hpc.ucar.edu 47:  Negative conc. in ch4tran. c,j,deficit (mol):           7           2
dec0471.hsn.de.hpc.ucar.edu 47:   1.046852049270273E-003
dec0471.hsn.de.hpc.ucar.edu 8: Image              PC                Routine            Line        Source
dec0471.hsn.de.hpc.ucar.edu 8: cesm.exe           00000000048F5819  shr_abort_mod_mp_         110  shr_abort_mod.F90
dec0471.hsn.de.hpc.ucar.edu 8: cesm.exe           00000000048F57AB  shr_abort_mod_mp_          65  shr_abort_mod.F90
dec0471.hsn.de.hpc.ucar.edu 8: cesm.exe           00000000048F672E  shr_assert_mod_mp          95  shr_assert_mod.F90.in
dec0471.hsn.de.hpc.ucar.edu 8: cesm.exe           00000000048F6B32  shr_assert_mod_mp         112  shr_assert_mod.F90.in
dec0471.hsn.de.hpc.ucar.edu 8: cesm.exe           00000000035D8C47  sparsematrixmulti         973  SparseMatrixMultiplyMod.F90
dec0471.hsn.de.hpc.ucar.edu 8: cesm.exe           0000000001647AC4  cnsoilmatrixmod_m         660  CNSoilMatrixMod.F90
dec0471.hsn.de.hpc.ucar.edu 8: cesm.exe           0000000003FC39B1  cndrivermod_mp_cn        1113  CNDriverMod.F90
dec0471.hsn.de.hpc.ucar.edu 8: cesm.exe           0000000001A94D14  cnvegetationfacad        1127  CNVegetationFacade.F90
dec0471.hsn.de.hpc.ucar.edu 8: cesm.exe           0000000000AB5AB7  clm_driver_mp_clm        1147  clm_driver.F90

@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 15, 2025

65 tests are different:

13 tests that show differences because at least irrigate was changed:

ERP_D.f10_f10_mg37.IHistClm60Bgc.derecho_gnu.clm-decStart--clm-matrixcnOn_ignore_warnings.GC.ctsm538acl_gnu/TestStatus.log:  COMP: irrigate = .true.
ERP_D.f10_f10_mg37.IHistClm60Bgc.derecho_gnu.clm-decStart.GC.ctsm538acl_gnu/TestStatus.log:  COMP: irrigate = .true.
ERP_D.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-decStart.GC.ctsm538acl_int/TestStatus.log:  COMP: irrigate = .true.
ERP_D_Ld5.f10_f10_mg37.IHistClm60Sp.derecho_intel.clm-default--clm-nofireemis.GC.ctsm538acl_int/TestStatus.log:  COMP: irrigate = .true.
ERP_D_Ld5.ne30pg3_t232.IHistClm60Sp.derecho_intel.clm-default--clm-nofireemis.GC.ctsm538acl_int/TestStatus.log:  COMP: irrigate = .true.
ERP_P64x2_Ld396.f10_f10_mg37.IHistClm60Bgc.derecho_gnu.clm-monthly.GC.ctsm538acl_gnu/TestStatus.log:  COMP: irrigate = .true.
ERP_P64x2_Ld396.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-monthly--clm-matrixcnOn_ignore_warnings.GC.ctsm538acl_int/TestStatus.log:  COMP: irrigate = .true.
ERP_P64x2_Ld396.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-monthly.GC.ctsm538acl_int/TestStatus.log:  COMP: irrigate = .true.
REP_P64x2_Ld13.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-monthly--clm-matrixcnOn_ignore_warnings.GC.ctsm538acl_int/TestStatus.log:  COMP: irrigate = .true.
REP_P64x2_Ld396.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-monthly--clm-matrixcnOn_ignore_warnings.GC.ctsm538acl_int/TestStatus.log:  COMP: irrigate = .true.
SMS_Ld5_Mmpi-serial.1x1_brazil.IHistClm60Bgc.derecho_gnu.clm-mimics.GC.ctsm538acl_gnu/TestStatus.log:  COMP: irrigate = .true.
SMS_Lm1.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-leafcn_t_evolving.GC.ctsm538acl_int/TestStatus.log:  COMP: irrigate = .true.
SMS_Ly1_Mmpi-serial.1x1_brazil.IHistClm60BgcQianRs.derecho_intel.clm-output_bgc_highfreq.GC.ctsm538acl_int/TestStatus.log:  COMP: irrigate = .true.

15 tests where c13_timeseries was turned on:
ERP_D_Ld5.f10_f10_mg37.I2000Clm50BgcCru.derecho_gnu.clm-ciso_flexCN_FUN--clm-matrixcnOn.GC.ctsm538acl_gnu/TestStatus.log: COMP: use_c13_timeseries = .true.
ERP_D_Ld5.f10_f10_mg37.I2000Clm50BgcCru.derecho_gnu.clm-ciso_flexCN_FUN.GC.ctsm538acl_gnu/TestStatus.log: COMP: use_c13_timeseries = .true.
ERP_P64x2_D_Ld5.f10_f10_mg37.I1850Clm45BgcCru.derecho_intel.clm-ciso.GC.ctsm538acl_int/TestStatus.log: COMP: use_c13_timeseries = .true.
ERP_P64x2_D_Ld5.f10_f10_mg37.I1850Clm50Bgc.derecho_intel.clm-ciso--clm-matrixcnOn_ignore_warnings.GC.ctsm538acl_int/TestStatus.log: COMP: use_c13_timeseries = .true.
ERP_P64x2_D_Ld5.f10_f10_mg37.I1850Clm50Bgc.derecho_intel.clm-ciso.GC.ctsm538acl_int/TestStatus.log: COMP: use_c13_timeseries = .true.
ERS_D.f19_g17.I1850Clm50BgcCrop.derecho_intel.clm-ciso_monthly_matrixcn_spinup.GC.ctsm538acl_int/TestStatus.log: COMP: use_c13_timeseries = .true.
ERS_Ld1640_Mmpi-serial.1x1_numaIA.I2000Clm50BgcCrop.derecho_intel.clm-ciso_monthly_matrixcn_spinup.GC.ctsm538acl_int/TestStatus.log: COMP: use_c13_timeseries = .true.
ERS_Ly5_Mmpi-serial.1x1_smallvilleIA.I1850Clm50BgcCrop.derecho_gnu.clm-ciso_monthly--clm-matrixcnOn.GC.ctsm538acl_gnu/TestStatus.log: COMP: use_c13_timeseries = .true.
ERS_Ly5_Mmpi-serial.1x1_smallvilleIA.I1850Clm50BgcCrop.derecho_gnu.clm-ciso_monthly.GC.ctsm538acl_gnu/TestStatus.log: COMP: use_c13_timeseries = .true.
LCISO_Ld396.f10_f10_mg37.I1850Clm50BgcCrop.derecho_intel.clm-ciso_monthly_matrixcn_spinup.GC.ctsm538acl_int/TestStatus.log: COMP: use_c13_timeseries = .true.
LGRAIN2_Ly1_P128x1.f10_f10_mg37.I1850Clm50BgcCrop.derecho_gnu.clm-ciso--clm-cropMonthOutput.GC.ctsm538acl_gnu/TestStatus.log: COMP: use_c13_timeseries = .true.
LGRAIN2_Ly2_P128x1.f10_f10_mg37.I1850Clm45BgcCrop.derecho_gnu.clm-ciso--clm-cropMonthOutput.GC.ctsm538acl_gnu/TestStatus.log: COMP: use_c13_timeseries = .true.
LREPRSTRUCT_Ly1_P128x1.f10_f10_mg37.I1850Clm50BgcCrop.derecho_gnu.clm-ciso--clm-cropMonthOutput.GC.ctsm538acl_gnu/TestStatus.log: COMP: use_c13_timeseries = .true.
LREPRSTRUCT_Ly2_P128x1.f10_f10_mg37.I1850Clm45BgcCrop.derecho_gnu.clm-ciso--clm-cropMonthOutput.GC.ctsm538acl_gnu/TestStatus.log: COMP: use_c13_timeseries = .true.
SSP_D_Ld4.f10_f10_mg37.I1850Clm50BgcCrop.derecho_intel.clm-ciso_rtmColdSSP.GC.ctsm538acl_int/TestStatus.log: COMP: use_c13_timeseries = .true.

@ekluzek ekluzek changed the title Some namelist default changes for clm6_0 physics ctsm5.3.071: Some namelist default changes for clm6_0 physics Oct 16, 2025
@ekluzek ekluzek changed the title ctsm5.3.071: Some namelist default changes for clm6_0 physics ctsm5.3.081: Some namelist default changes for clm6_0 physics Oct 16, 2025
Copy link
Contributor

@slevis-lmwg slevis-lmwg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @ekluzek. I had one comment motivated by reduced code clutter.

Before approving, I also wanted to confirm resolution of the unexpected fail that you posted here #3549 (comment)

Copy link
Collaborator Author

@ekluzek ekluzek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment from our discussion from Monday

@ekluzek ekluzek requested a review from slevis-lmwg October 22, 2025 08:27
@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 22, 2025

Before approving, I also wanted to confirm resolution of the unexpected fail that you posted here #3549 (comment)

This is the someone screwy matrix test with threading on that has known problems. It can sometimes fail, but will pass on resubmit. So this isn't something we should worry about. But, I did rerun to confirm that and it passed on resubmission.

I'm now resending aux_clm.

Copy link
Contributor

@slevis-lmwg slevis-lmwg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ekluzek and I worked together on updating some comments. Now I can approve.

@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 22, 2025

OK, testing is as expected on Derecho and Izumi. Some tests only differ in the field list (because C13/C14 was switched). So only the following tests show difference answers as expected because of the irrigate change:

ERP_D.f10_f10_mg37.IHistClm60Bgc.derecho_gnu.clm-decStart--clm-matrixcnOn_ignore_warnings.GC.ctsm5381acl_gnu/TestStatus:FAIL ERP_D.f10_f10_mg37.IHistClm60Bgc.derecho_gnu.clm-decStart--clm-matrixcnOn_ignore_warnings BASELINE ctsm5.3.080: DIFF
ERP_D.f10_f10_mg37.IHistClm60Bgc.derecho_gnu.clm-decStart.GC.ctsm5381acl_gnu/TestStatus:FAIL ERP_D.f10_f10_mg37.IHistClm60Bgc.derecho_gnu.clm-decStart BASELINE ctsm5.3.080: DIFF
ERP_D.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-decStart.GC.ctsm5381acl_int/TestStatus:FAIL ERP_D.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-decStart BASELINE ctsm5.3.080: DIFF
ERP_D_Ld5.f10_f10_mg37.IHistClm60Sp.derecho_intel.clm-default--clm-nofireemis.GC.ctsm5381acl_int/TestStatus:FAIL ERP_D_Ld5.f10_f10_mg37.IHistClm60Sp.derecho_intel.clm-default--clm-nofireemis BASELINE ctsm5.3.080: DIFF
ERP_D_Ld5.ne30pg3_t232.IHistClm60Sp.derecho_intel.clm-default--clm-nofireemis.GC.ctsm5381acl_int/TestStatus:FAIL ERP_D_Ld5.ne30pg3_t232.IHistClm60Sp.derecho_intel.clm-default--clm-nofireemis BASELINE ctsm5.3.080: DIFF
ERP_P64x2_Ld396.f10_f10_mg37.IHistClm60Bgc.derecho_gnu.clm-monthly.GC.ctsm5381acl_gnu/TestStatus:FAIL ERP_P64x2_Ld396.f10_f10_mg37.IHistClm60Bgc.derecho_gnu.clm-monthly BASELINE ctsm5.3.080: DIFF
ERP_P64x2_Ld396.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-monthly--clm-matrixcnOn_ignore_warnings.GC.ctsm5381acl_int/TestStatus:FAIL ERP_P64x2_Ld396.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-monthly--clm-matrixcnOn_ignore_warnings BASELINE ctsm5.3.080: DIFF
ERP_P64x2_Ld396.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-monthly.GC.ctsm5381acl_int/TestStatus:FAIL ERP_P64x2_Ld396.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-monthly BASELINE ctsm5.3.080: DIFF
REP_P64x2_Ld13.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-monthly--clm-matrixcnOn_ignore_warnings.GC.ctsm5381acl_int/TestStatus:FAIL REP_P64x2_Ld13.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-monthly--clm-matrixcnOn_ignore_warnings BASELINE ctsm5.3.080: DIFF
REP_P64x2_Ld396.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-monthly--clm-matrixcnOn_ignore_warnings.GC.ctsm5381acl_int/TestStatus:FAIL REP_P64x2_Ld396.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-monthly--clm-matrixcnOn_ignore_warnings BASELINE ctsm5.3.080: DIFF
SMS_Lm1.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-leafcn_t_evolving.GC.ctsm5381acl_int/TestStatus:FAIL SMS_Lm1.f10_f10_mg37.IHistClm60Bgc.derecho_intel.clm-leafcn_t_evolving BASELINE ctsm5.3.080: DIFF
ERP_D_Ld5.f10_f10_mg37.IHistClm60Bgc.izumi_nag.clm-decStart--clm-nvmovement.GC.ctsm5381acl_nag/TestStatus:FAIL ERP_D_Ld5.f10_f10_mg37.IHistClm60Bgc.izumi_nag.clm-decStart--clm-nvmovement BASELINE ctsm5.3.080: DIFF
ERP_D_P48x1.f10_f10_mg37.IHistClm60Bgc.izumi_nag.clm-decStart--clm-matrixcnOn_ignore_warnings.GC.ctsm5381acl_nag/TestStatus:FAIL ERP_D_P48x1.f10_f10_mg37.IHistClm60Bgc.izumi_nag.clm-decStart--clm-matrixcnOn_ignore_warnings BASELINE ctsm5.3.080: DIFF
ERP_D_P48x1.f10_f10_mg37.IHistClm60Bgc.izumi_nag.clm-decStart.GC.ctsm5381acl_nag/TestStatus:FAIL ERP_D_P48x1.f10_f10_mg37.IHistClm60Bgc.izumi_nag.clm-decStart BASELINE ctsm5.3.080: DIFF
SMS_Ld5_D_P48x1.f10_f10_mg37.IHistClm60Bgc.izumi_nag.clm-decStart.GC.ctsm5381acl_nag/TestStatus:FAIL SMS_Ld5_D_P48x1.f10_f10_mg37.IHistClm60Bgc.izumi_nag.clm-decStart BASELINE ctsm5.3.080: DIFF

Tests that have different answers because C13/C14 timeseries files were turned on (for just C13/C14 fields):
These are tests that use the ciso testmod.

ERP_D_Ld5.f10_f10_mg37.I2000Clm50BgcCru.derecho_gnu.clm-ciso_flexCN_FUN--clm-matrixcnOn.GC.ctsm5381acl_gnu/TestStatus:FAIL ERP_D_Ld5.f10_f10_mg37.I2000Clm50BgcCru.derecho_gnu.clm-ciso_flexCN_FUN--clm-matrixcnOn BASELINE ctsm5.3.080: DIFF
ERP_D_Ld5.f10_f10_mg37.I2000Clm50BgcCru.derecho_gnu.clm-ciso_flexCN_FUN.GC.ctsm5381acl_gnu/TestStatus:FAIL ERP_D_Ld5.f10_f10_mg37.I2000Clm50BgcCru.derecho_gnu.clm-ciso_flexCN_FUN BASELINE ctsm5.3.080: DIFF
ERP_P64x2_D_Ld5.f10_f10_mg37.I1850Clm45BgcCru.derecho_intel.clm-ciso.GC.ctsm5381acl_int/TestStatus:FAIL ERP_P64x2_D_Ld5.f10_f10_mg37.I1850Clm45BgcCru.derecho_intel.clm-ciso BASELINE ctsm5.3.080: DIFF
ERP_P64x2_D_Ld5.f10_f10_mg37.I1850Clm50Bgc.derecho_intel.clm-ciso--clm-matrixcnOn_ignore_warnings.GC.ctsm5381acl_int/TestStatus:FAIL ERP_P64x2_D_Ld5.f10_f10_mg37.I1850Clm50Bgc.derecho_intel.clm-ciso--clm-matrixcnOn_ignore_warnings BASELINE ctsm5.3.080: DIFF
ERP_P64x2_D_Ld5.f10_f10_mg37.I1850Clm50Bgc.derecho_intel.clm-ciso.GC.ctsm5381acl_int/TestStatus:FAIL ERP_P64x2_D_Ld5.f10_f10_mg37.I1850Clm50Bgc.derecho_intel.clm-ciso BASELINE ctsm5.3.080: DIFF
ERS_Ly5_Mmpi-serial.1x1_smallvilleIA.I1850Clm50BgcCrop.derecho_gnu.clm-ciso_monthly--clm-matrixcnOn.GC.ctsm5381acl_gnu/TestStatus:FAIL ERS_Ly5_Mmpi-serial.1x1_smallvilleIA.I1850Clm50BgcCrop.derecho_gnu.clm-ciso_monthly--clm-matrixcnOn BASELINE ctsm5.3.080: DIFF
ERS_Ly5_Mmpi-serial.1x1_smallvilleIA.I1850Clm50BgcCrop.derecho_gnu.clm-ciso_monthly.GC.ctsm5381acl_gnu/TestStatus:FAIL ERS_Ly5_Mmpi-serial.1x1_smallvilleIA.I1850Clm50BgcCrop.derecho_gnu.clm-ciso_monthly BASELINE ctsm5.3.080: DIFF
LGRAIN2_Ly1_P128x1.f10_f10_mg37.I1850Clm50BgcCrop.derecho_gnu.clm-ciso--clm-cropMonthOutput.GC.ctsm5381acl_gnu/TestStatus:FAIL LGRAIN2_Ly1_P128x1.f10_f10_mg37.I1850Clm50BgcCrop.derecho_gnu.clm-ciso--clm-cropMonthOutput BASELINE ctsm5.3.080: DIFF
LGRAIN2_Ly2_P128x1.f10_f10_mg37.I1850Clm45BgcCrop.derecho_gnu.clm-ciso--clm-cropMonthOutput.GC.ctsm5381acl_gnu/TestStatus:FAIL LGRAIN2_Ly2_P128x1.f10_f10_mg37.I1850Clm45BgcCrop.derecho_gnu.clm-ciso--clm-cropMonthOutput BASELINE ctsm5.3.080: DIFF
LREPRSTRUCT_Ly1_P128x1.f10_f10_mg37.I1850Clm50BgcCrop.derecho_gnu.clm-ciso--clm-cropMonthOutput.GC.ctsm5381acl_gnu/TestStatus:FAIL LREPRSTRUCT_Ly1_P128x1.f10_f10_mg37.I1850Clm50BgcCrop.derecho_gnu.clm-ciso--clm-cropMonthOutput BASELINE ctsm5.3.080: DIFF
LREPRSTRUCT_Ly2_P128x1.f10_f10_mg37.I1850Clm45BgcCrop.derecho_gnu.clm-ciso--clm-cropMonthOutput.GC.ctsm5381acl_gnu/TestStatus:FAIL LREPRSTRUCT_Ly2_P128x1.f10_f10_mg37.I1850Clm45BgcCrop.derecho_gnu.clm-ciso--clm-cropMonthOutput BASELINE ctsm5.3.080: DIFF
SSP_D_Ld4.f10_f10_mg37.I1850Clm50BgcCrop.derecho_intel.clm-ciso_rtmColdSSP.GC.ctsm5381acl_int/TestStatus:FAIL SSP_D_Ld4.f10_f10_mg37.I1850Clm50BgcCrop.derecho_intel.clm-ciso_rtmColdSSP BASELINE ctsm5.3.080: DIFF
ERP_D_Ld5_P48x1.f10_f10_mg37.I1850Clm50Bgc.izumi_nag.clm-ciso.GC.ctsm5381acl_nag/TestStatus:FAIL ERP_D_Ld5_P48x1.f10_f10_mg37.I1850Clm50Bgc.izumi_nag.clm-ciso BASELINE ctsm5.3.080: DIFF

So all tests are as expected.

@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 22, 2025

I didn't change anything in python code for this to fail. But, I'm seeing 11 of 117 the python system tests for subsetdata failing here, when the ctsm5.3.080 works fine.

I'll look at this more to see if it could be something I changed. But, if I don't see it, I may merge it in anyway...

ERROR: test_subset_data_reg_amazon_noregname (test.test_sys_subset_data.TestSubsetDataSys.test_subset_data_reg_amazon_noregname)
Test subset_data for Amazon region
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/glade/work/erik/ctsm_worktrees/answer_changes/python/ctsm/test/test_sys_subset_data.py", line 209, in test_subset_data_reg_amazon_noregname
    self._do_test_subset_data_reg_amazon(include_regname=False)
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^
  File "/glade/work/erik/ctsm_worktrees/answer_changes/python/ctsm/test/test_sys_subset_data.py", line 191, in _do_test_subset_data_reg_amazon
    self.assertTrue(self._check_result_file_matches_expected(expected_output_files, 2))
                    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/glade/work/erik/ctsm_worktrees/answer_changes/python/ctsm/test/test_sys_subset_data.py", line 124, in _check_result_file_matches_expected
    ds_expected = xr.open_dataset(expected_file)
  File "/glade/work/erik/conda-envs/ctsm_pylib/lib/python3.13/site-packages/xarray/backends/api.py", line 667, in open_dataset
    engine = plugins.guess_engine(filename_or_obj)
  File "/glade/work/erik/conda-envs/ctsm_pylib/lib/python3.13/site-packages/xarray/backends/plugins.py", line 194, in guess_engine
    raise ValueError(error_msg)
ValueError: did not find a match in any of xarray's currently installed IO backends ['netcdf4', 'scipy']. Consider explicitly selecting one of the installed engines via the ``engine`` parameter, or installing additional IO dependencies, see:
https://docs.xarray.dev/en/stable/getting-started-guide/installing.html
https://docs.xarray.dev/en/stable/user-guide/io.html

@ekluzek
Copy link
Collaborator Author

ekluzek commented Oct 22, 2025

#3205 gave the answer to the python system testing problem above. A couple of the files weren't pulled from LFS correctly. And I just needed to pull them in again.

@ekluzek ekluzek merged commit e1efc3c into ESCOMP:master Oct 22, 2025
4 checks passed
@ekluzek ekluzek deleted the ciso_namelist_default_changes branch October 22, 2025 23:32
@github-project-automation github-project-automation bot moved this from In progress - master to Done (non release/external) in CTSM: Upcoming tags Oct 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement new capability or improved behavior of existing capability non-bfb Changes answers (incl. adding tests) usability Improve or clarify user-facing options

Projects

Status: Done (non release/external)
Status: Done

3 participants