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

GitFlow: Merge Develop into Main for a release #22

Merged
merged 7 commits into from
Aug 3, 2023
Merged

GitFlow: Merge Develop into Main for a release #22

merged 7 commits into from
Aug 3, 2023

Conversation

mathomp4
Copy link
Member

@mathomp4 mathomp4 commented Aug 1, 2023

This pulls in the change from #21 into main for a tag

mathomp4 and others added 7 commits June 8, 2023 11:56
This PR updates the CI to point to Baselibs 7.13.0 which is the version used by GEOSgcm on `main`
This PR updates the CircleCI config to default to using the Baselibs in
the CircleCI orb rather than explicitly specifying the versions. In most
cases, this is the "best" way as a change to the orb can fix this for
projects without the need for a PR.

Note that we keep commented the explicit anchors in case needed.
@mathomp4 mathomp4 added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Aug 1, 2023
@mathomp4 mathomp4 requested a review from a team as a code owner August 1, 2023 19:09
@mathomp4 mathomp4 self-assigned this Aug 1, 2023
@mathomp4
Copy link
Member Author

mathomp4 commented Aug 2, 2023

Let me do a quick GEOSgcm run to make sure it's zero-diff. I mean, it seems it obviously is, but...for safety's sake :)

@dr0cloud
Copy link
Collaborator

dr0cloud commented Aug 2, 2023

Should be 0diff if running standard RRTMG. But if running RRTMGP it may not be 0diff if one selects those new AGCM.rc parameters different from the default values of 10.

@mathomp4
Copy link
Member Author

mathomp4 commented Aug 3, 2023

And it is! Zero-diff. Merging and will make release.

@mathomp4 mathomp4 merged commit 0d8e5e7 into main Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 diff The changes in this pull request have verified to be zero-diff with the target branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants