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

Bump SHiELD_physics and atmos_drivers #23

Merged
merged 4 commits into from
Dec 21, 2023

Conversation

spencerkclark
Copy link
Member

@spencerkclark spencerkclark commented Dec 20, 2023

This PR bumps the versions of the SHiELD_physics and atmos_drivers repositories following NOAA-GFDL/SHiELD_physics#35, NOAA-GFDL/SHiELD_physics#36, NOAA-GFDL/SHiELD_physics#37, and NOAA-GFDL/atmos_drivers#32; these PRs fix a bug in loading a Q-flux from a file and add surface type fraction diagnostics (like in ai2cm/fv3gfs-fortran#377). In doing so this PR adds regression test coverage for diagnostics manager controlled physics diagnostics, to ensure that these continue operating properly as the SHiELD_physics code evolves.

We first add a diagnostics manager controlled field to the regression test and update the checksums, then update the submodules to ensure that the updates do not modify this existing field, and finally add the surface type fraction fields.

Copy link

@brianhenn brianhenn left a comment

Choose a reason for hiding this comment

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

@spencerkclark
Copy link
Member Author

Thanks @brianhenn — in porting the surface type fraction diagnostics over from FV3GFS I neglected to test compiling things in mixed 32-bit / 64-bit mode, and missed a spot where kind=kind_phys needed to be specified when declaring a real. Thankfully @laurenchilutti caught this quickly. I'll update this PR and merge once NOAA-GFDL/SHiELD_physics#37 is merged upstream.

It's not important for the wrapper at the moment since we do not support wrapping a mixed mode model, but it would make me feel better to have this feature implemented fully correctly in this update.

@spencerkclark spencerkclark merged commit 1e5939d into main Dec 21, 2023
2 checks passed
@spencerkclark spencerkclark deleted the 2023-12-20-bump-submodules branch December 21, 2023 21:51
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.

None yet

2 participants