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 bugs in GCC vs GCHP transport tracer benchmark mass tables #300

Merged
merged 2 commits into from
Feb 5, 2024

Conversation

lizziel
Copy link
Contributor

@lizziel lizziel commented Feb 5, 2024

Name and Institution (Required)

Name: Lizzie Lundgren
Institution: Harvard University

Describe the update

This PR fixes two bugs that were preventing generation of the GCC vs GCHP mass tables in the transport tracer benchmark. One of the problems was the reference restart path used the GCHP run directory rather than GCC. This bug went undetected because of the second problem, which omitted the call to generate the GCC vs GCHP mass tables altogether.

These problems also went under the radar because GCC vs GCHP global mass tables are not posted as part of the 1-year benchmark tables per version. The tables can now be generated and so should be included on the GEOS-Chem version pages.

Expected changes

GCC vs GCHP global mass tables with now be generated in the 1-yr transport tracer benchmark.

Reference(s)

None

Related Github Issue(s)

None

@lizziel lizziel added category: Bug Something isn't working topic: Benchmark Plots and Tables Issues pertaining to generating plots/tables from benchmark output labels Feb 5, 2024
@lizziel lizziel requested a review from yantosca February 5, 2024 16:29
The call to make the benchmark mass tables was missing from the transport
tracer run script.

Signed-off-by: Lizzie Lundgren <[email protected]>
@lizziel lizziel force-pushed the bugfix/tt_benchmarK_gcc_vs_gchp_mass_tables branch from 4007bc6 to 9940b8f Compare February 5, 2024 16: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 @lizziel! I'll bring this into dev.

@yantosca yantosca merged commit a55badc into dev Feb 5, 2024
0 of 3 checks passed
@yantosca yantosca deleted the bugfix/tt_benchmarK_gcc_vs_gchp_mass_tables branch February 5, 2024 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Bug Something isn't working topic: Benchmark Plots and Tables Issues pertaining to generating plots/tables from benchmark output
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants