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

Fix missing headers in aerosol table #341

Merged
merged 1 commit into from
Sep 13, 2024

Conversation

yuanjianz
Copy link
Contributor

Name and Institution (Required)

Name: Yuanjian Zhang
Institution: WashU

Describe the update

Fix missing headers for aerosol burden table.

Expected changes

Missing headers in aerosol burden table now will be printed into the file.

Reference(s)

Related Github Issue

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 is a simple PR that looks OK to merge. I will add the CHANGELOG.md entry.

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 is a simple PR that redirects output to the file instead I will add the changelog update.

yantosca added a commit that referenced this pull request Sep 13, 2024
CHANGELOG.md
- Added a note about the header line now being written to file
  in the aerosol burden table

Signed-off-by: Bob Yantosca <[email protected]>
yantosca added a commit that referenced this pull request Sep 13, 2024
This merge brings PR #341 (Fix missing headers in aerosol table,
by @yuanjianz) into the GCPy 1.6.0 development stream.

This PR makes sure that the line containing the header for the
benchmark aerosol burden table is written to file.

Signed-off-by: Bob Yantosca <[email protected]>
@yantosca yantosca merged commit 08bcbc2 into geoschem:dev Sep 13, 2024
14 checks passed
@yuanjianz yuanjianz deleted the bugfix/aerosol_table_headers branch September 13, 2024 19:41
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.

2 participants