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

Add capability for TOMAS simulations in GCHP #359

Merged
merged 14 commits into from
Dec 6, 2023
Merged

Conversation

msulprizio
Copy link
Contributor

Name and Institution (Required)

Name: Melissa Sulprizio
Institution: Harvard / GCST

Describe the update

This pull request adds the option to run GCHP with TOMAS as originally implemented by @BettyCroft in:

This pull request should be merged alongside:

Copy link
Contributor

@yantosca yantosca left a comment

Choose a reason for hiding this comment

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

This looks mostly good to merge, pending minor fixes as noted.

Also: The HEMCO submodule only has 2 files changed but the HEMCO PR #251 contains changes 3 files. We'll need to point to the updated commit in the HEMCO submodule.

CHANGELOG.md Outdated Show resolved Hide resolved
Also bring dev/tomas branches in the GEOS-Chem and HEMCO submodules
up-to-date.

Signed-off-by: Melissa Sulprizio <[email protected]>
@msulprizio msulprizio requested review from yantosca and removed request for lizziel December 6, 2023 13:46
Copy link
Contributor

@yantosca yantosca left a comment

Choose a reason for hiding this comment

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

Now good to merge!

@msulprizio msulprizio merged commit a0aa482 into dev/14.3.0 Dec 6, 2023
1 check passed
@msulprizio msulprizio deleted the dev/tomas branch December 6, 2023 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Feature Request New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants