Skip to content

part of cam6_4_183: Add rad_diag CARMA testmod to existing Derecho FCARMA2000climo test#1562

Merged
3 commits merged into
ESCOMP:cam_developmentfrom
jimmielin:hplin/carma_rad_diag_test
Jun 22, 2026
Merged

part of cam6_4_183: Add rad_diag CARMA testmod to existing Derecho FCARMA2000climo test#1562
3 commits merged into
ESCOMP:cam_developmentfrom
jimmielin:hplin/carma_rad_diag_test

Conversation

@jimmielin

Copy link
Copy Markdown
Collaborator

Closes #1560

@jimmielin jimmielin requested a review from fvitt May 22, 2026 14:28
@jimmielin jimmielin self-assigned this May 22, 2026

@fvitt fvitt left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I wonder if we should be include CARMA's ability to generate optics file in this regression test? Thanks for the PR.

'B:MXAER17:$DIN_LOC_ROOT/atm/cam/chem/carma/rrtmg/aerosol_cld_CRMIX17_rrtmg.nc',
'B:MXAER18:$DIN_LOC_ROOT/atm/cam/chem/carma/rrtmg/aerosol_cld_CRMIX18_rrtmg.nc',
'B:MXAER19:$DIN_LOC_ROOT/atm/cam/chem/carma/rrtmg/aerosol_cld_CRMIX19_rrtmg.nc',
'B:MXAER20:$DIN_LOC_ROOT/atm/cam/chem/carma/rrtmg/aerosol_cld_CRMIX20_rrtmg.nc'

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Would it be worthwhile to turn on carma's ability generate the optics files in this test -- carma_do_optics = .true.?

The rad_climate and rad_diag_* would point to aerosol optics files in the run director.
Something like:

rad_climate   = 'A:Q:H2O', 'N:O2:O2', 'A:CO2:CO2', 'A:O3:O3', 'A:N2O:N2O', 'A:CH4:CH4',
         'N:CFC11:CFC11', 'N:CFC12:CFC12',
         'B:MXAER01:trop_strat_soa5_MXAER01_rrtmg.nc', 
         'B:MXAER02:trop_strat_soa5_MXAER02_rrtmg.nc',
         'B:MXAER03:trop_strat_soa5_MXAER03_rrtmg.nc',
         'B:MXAER04:trop_strat_soa5_MXAER04_rrtmg.nc',
         'B:MXAER05:trop_strat_soa5_MXAER05_rrtmg.nc',
         'B:MXAER06:trop_strat_soa5_MXAER06_rrtmg.nc',
...

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Thanks for the suggestion. I added carma_do_optics but used the generated optics files in rad_diag_3 instead of rad_climate, to avoid potentially introducing answer changes in the baseline. In this way cprnc will just say diff_test: the two files DIFFER only in their field lists since the new diagnostic list added new fields compared to the existing test. Please let me know if you like me to replace rad_climate instead.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the explanation. I think what you did here is fine.

@jimmielin jimmielin requested a review from fvitt June 16, 2026 15:08
@cacraigucar cacraigucar changed the title Add rad_diag CARMA testmod to existing Derecho FCARMA2000climo test part of cam6_4_183: Add rad_diag CARMA testmod to existing Derecho FCARMA2000climo test Jun 22, 2026
@jimmielin jimmielin closed this pull request by merging all changes into ESCOMP:cam_development in a1dcf88 Jun 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Tag

Development

Successfully merging this pull request may close these issues.

3 participants