Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update HEMCO interface for changes to HEMCO error and log handling #2643

Merged

Conversation

lizziel
Copy link
Contributor

@lizziel lizziel commented Dec 16, 2024

Name and Institution (Required)

Name: Lizzie Lundgren
Institution: Harvard University

Describe the update

Updates in this PR:

  1. Update argument lists for HCO_LOGFILE_OPEN, HCO_MSG, and HCO_ERROR due to changes in Improve log flexibility in HEMCO HEMCO#298.
  2. Remove print messages passed to GC_ERROR in the HEMCO interface instructing users to see the HEMCO log. GEOS-Chem no longer users a separate HEMCO log by default.
  3. Suppress doing Linoz and Linear chemistry prints unless verbose.
  4. Restrict KPP standalone disabled print to root only.

NOTE: This PR should be merged at the same time as geoschem/HEMCO#298.

Expected changes

This is a no diff update.

Reference(s)

None

Related Github Issue

geoschem/HEMCO#298

@lizziel lizziel added category: Feature Request New feature or request topic: HEMCO Submodule Related to HEMCO no-diff-to-benchmark This update will not change the results of fullchem benchmark simulations labels Dec 16, 2024
@lizziel lizziel self-assigned this Dec 16, 2024
This update requires an analogous update in HEMCO

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
GEOS-Chem no longer users a separate HEMCO log by default.

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
@lizziel lizziel force-pushed the feature/clean_up_hemco_log_and_error_handling branch from 2f21b00 to 54b4a81 Compare December 17, 2024 14:48
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
@lizziel lizziel added the topic: Structural Modifications Related to GEOS-Chem structural modifications (as opposed to scientific updates) label Dec 17, 2024
@lizziel lizziel marked this pull request as ready for review December 17, 2024 21:24
@lizziel lizziel requested a review from yantosca December 17, 2024 21:24
@lizziel lizziel added this to the 14.5.1 milestone Dec 17, 2024
Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
@lizziel lizziel force-pushed the feature/clean_up_hemco_log_and_error_handling branch from 3721468 to 344a4f5 Compare December 17, 2024 21:25
@lizziel lizziel marked this pull request as draft December 17, 2024 21:28
amIRoot is included within verbose so is not needed.

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
@lizziel lizziel marked this pull request as ready for review December 17, 2024 21:30
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.

Looks good to merge alongside geoschem/HEMCO#298. Thanks @lizziel!

@lizziel lizziel merged commit f4af613 into dev/no-diff-to-benchmark Dec 19, 2024
@lizziel lizziel deleted the feature/clean_up_hemco_log_and_error_handling branch December 19, 2024 18:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
category: Feature Request New feature or request no-diff-to-benchmark This update will not change the results of fullchem benchmark simulations topic: HEMCO Submodule Related to HEMCO topic: Structural Modifications Related to GEOS-Chem structural modifications (as opposed to scientific updates)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants