Skip to content
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

cam6_4_034: Add missing total energy in physics state from dycore in snapshots and cleanup total water #1142

Merged

Conversation

jimmielin
Copy link
Member

@jimmielin jimmielin commented Sep 5, 2024

Closes #1141

Saves second dimension (dycore formula) of total energy and total water initial and current condition.

Companion PR in CAM-SIMA to read updated snapshot file and updates to standard names:
ESCOMP/CAM-SIMA#296

Copy link
Collaborator

@cacraigucar cacraigucar left a comment

Choose a reason for hiding this comment

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

Good job finding and fixing the bug in cam_snapshot

@cacraigucar cacraigucar added the bug Something isn't working correctly label Sep 5, 2024
@cacraigucar
Copy link
Collaborator

@nusbaume - since the changes morphed beyond snapshot code, I am adding you to the reviewer list as I have not reviewed the other changes

@jimmielin jimmielin changed the title Add missing total energy and total water in physics state from dycore in snapshots Add missing total energy in physics state from dycore in snapshots and cleanup total water Sep 5, 2024
@jimmielin
Copy link
Member Author

jimmielin commented Sep 5, 2024

Thanks @cacraigucar, apologies for repurposing the existing branch/PR. I've updated the title. Changelog update will follow shortly.

@jimmielin jimmielin force-pushed the hplin/fix_total_energy_snapshots branch from ceabf1a to 1ecca08 Compare September 10, 2024 03:49
@jimmielin
Copy link
Member Author

Changes rebased on newest cam_development (cam6_4_031)

Copy link
Collaborator

@nusbaume nusbaume left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this!

@cacraigucar cacraigucar changed the title Add missing total energy in physics state from dycore in snapshots and cleanup total water cam6_4_034: Add missing total energy in physics state from dycore in snapshots and cleanup total water Sep 19, 2024
@jimmielin jimmielin merged commit 4c7c062 into ESCOMP:cam_development Sep 19, 2024
2 checks passed
@jimmielin jimmielin deleted the hplin/fix_total_energy_snapshots branch September 19, 2024 21:31
jimmielin added a commit to ESCOMP/CAM-SIMA that referenced this pull request Sep 26, 2024
…date standard names (#296)

Companion issue in CAM: ESCOMP/CAM#1141

Companion PR in CAM: ESCOMP/CAM#1142
this PR in CAM will update snapshots with new variables.

Updates to CAM-SIMA:
- read ncdata variables te_ini_phys, te_ini_dyn, te_cur_phys, te_cur_dyn
- update to standard names per discussion in
ESCOMP/CAM#1141
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working correctly ccpp-conversion
Projects
Status: Tag
Development

Successfully merging this pull request may close these issues.

3 participants