Skip to content

Complete CCPPization of CAM7 hb_free_atm: add regression test - #513

Open
jimmielin wants to merge 1 commit into
ESCOMP:developmentfrom
jimmielin:hplin/hb_free_atm
Open

Complete CCPPization of CAM7 hb_free_atm: add regression test#513
jimmielin wants to merge 1 commit into
ESCOMP:developmentfrom
jimmielin:hplin/hb_free_atm

Conversation

@jimmielin

@jimmielin jimmielin commented Jun 22, 2026

Copy link
Copy Markdown
Collaborator

Tag name (required for release branches):
Originator(s): @jimmielin
AI tools used (if applicable; please also add the "AI-generated code" label to the PR):
What:
How:

Companion PR: ESCOMP/atmospheric_physics#414

Description (include the issue title, and the keyword ['closes', 'fixes', 'resolves'] followed by the issue number):

  • Add regression test for hb_free_atm for CAM7 above CLUBB

Describe any changes made to build system:

Describe any changes made to the namelist:

List any changes to the defaults for the input datasets (e.g. boundary datasets):

List all files eliminated and why:

List all files added and what they do:

A       cime_config/testdefs/testmods_dirs/cam/outfrq_hb_freeatm_vdiff_derecho/shell_commands
A       cime_config/testdefs/testmods_dirs/cam/outfrq_hb_freeatm_vdiff_derecho/user_nl_cam
  - new testmod

List all existing files that have been modified, and describe the changes:
(Helpful git command: git diff --name-status development...<your_branch_name>)

M       cime_config/testdefs/testlist_cam.xml
  - new testmod

If there are new failures (compared to the test/existing-test-failures.txt file),
have them OK'd by the gatekeeper, note them here, and add them to the file.
If there are baseline differences, include the test and the reason for the
diff. What is the nature of the change? Roundoff?

derecho/intel/aux_sima:

derecho/gnu/aux_sima:

derecho/nvhpc/aux_sima (test is run via Github workflow. Only run the test manually if we need to save new baselines):

If this changes climate describe any run(s) done to evaluate the new
climate in enough detail that it(they) could be reproduced:

CAM-SIMA date used for the baseline comparison tests if different than latest:

@jimmielin jimmielin self-assigned this Jun 22, 2026
@jimmielin
jimmielin marked this pull request as ready for review June 22, 2026 15:15
@jimmielin
jimmielin had a problem deploying to CI-tests-on-CIRRUS June 22, 2026 15:15 — with GitHub Actions Failure
@nusbaume
nusbaume requested a review from cacraigucar July 15, 2026 16:18
Comment on lines +2 to +3
ncdata = '/glade/campaign/cesm/community/amwg/sima_baselines/cam_sima_test_snapshots/cam_ne3pg3_fhistc_ltso_vertical_diffusion_hb_free_atm_snapshot_derecho_gnu_before_c20260604.nc'
ncdata_check = '/glade/campaign/cesm/community/amwg/sima_baselines/cam_sima_test_snapshots/cam_ne3pg3_fhistc_ltso_vertical_diffusion_hb_free_atm_snapshot_derecho_gnu_after_c20260604.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.

Should this use the new variable that Jesse is introducing for the ROOT? I guess it depends on what comes in first, but we need to make sure this one doesn't slip through the cracks if it comes in first

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.

Will do! Not sure which will come in first, but I have plenty of test mods I will merge in, so I'll keep this in mind. Thanks for the reminder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants