-
Notifications
You must be signed in to change notification settings - Fork 27
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
Conversation
Signed-off-by: Melissa Sulprizio <[email protected]>
Signed-off-by: Melissa Sulprizio <[email protected]>
Resolved conflicts in: CHANGELOG.md src/GCHP_GridComp/GEOSChem_GridComp/CMakeLists.txt src/GCHP_GridComp/GEOSChem_GridComp/geos-chem src/GCHP_GridComp/HEMCO_GridComp/HEMCO Signed-off-by: Melissa Sulprizio <[email protected]>
Signed-off-by: Melissa Sulprizio <[email protected]>
Signed-off-by: Melissa Sulprizio <[email protected]>
There was a problem hiding this 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.
Also bring dev/tomas branches in the GEOS-Chem and HEMCO submodules up-to-date. Signed-off-by: Melissa Sulprizio <[email protected]>
There was a problem hiding this 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!
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: