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 GCHPctmEnv small numerical bugs #456

Conversation

yuanjianz
Copy link

@yuanjianz yuanjianz commented Oct 30, 2024

Name and Institution (Required)

Name: Yuanjian Zhang
Institution: WashU

Describe the update

  1. Correct dry mass flux derivation.
  2. Correct upwardsMassflux sign.

Expected changes

  1. Dry mass flux will be slightly smaller than before but is expected.
  2. UpwardsMassflux diagnostic will correctly be positive = "up"

Related Github Issue

closes #445
closes #455

@lizziel
Copy link
Contributor

lizziel commented Nov 12, 2024

Update #1 affects only GCHP runs that import mass fluxes from offline meteorology which makes this a zero diff update for benchmarking. I will therefore put milestone 14.5.1.

@lizziel lizziel added this to the 14.5.1 milestone Nov 12, 2024
@lizziel lizziel added topic: Runtime Related to runtime issues (e.g. simulation stops with error) category: Bug Fix Fixes a previously-reported issue topic: Diagnostics Related to output diagnostic data labels Nov 12, 2024
@yuanjianz yuanjianz force-pushed the bugfix/gchp-advection-small-numerical-bugs branch from bf79029 to 4023750 Compare November 15, 2024 17:40
@lizziel lizziel changed the base branch from main to dev/no-diff-to-benchmark November 15, 2024 19:11
@lizziel lizziel self-requested a review November 15, 2024 19:12
Copy link
Contributor

@lizziel lizziel left a comment

Choose a reason for hiding this comment

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

Please also update the changelog with the updates.

@yuanjianz yuanjianz requested a review from lizziel November 15, 2024 20:02
@lizziel lizziel self-assigned this Nov 22, 2024
@lizziel lizziel force-pushed the bugfix/gchp-advection-small-numerical-bugs branch from 85979ae to 47f4160 Compare December 4, 2024 15:47
@lizziel
Copy link
Contributor

lizziel commented Dec 4, 2024

@yuanjianz, I am rebasing to avoid merge conflicts and streamline the git history when I bring this into dev/no-diff-to-benchmark.

@lizziel lizziel force-pushed the bugfix/gchp-advection-small-numerical-bugs branch from 47f4160 to 2641547 Compare December 5, 2024 15:41
@lizziel lizziel merged commit afd03c1 into geoschem:dev/no-diff-to-benchmark Dec 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Bug Fix Fixes a previously-reported issue topic: Diagnostics Related to output diagnostic data topic: Runtime Related to runtime issues (e.g. simulation stops with error)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Opposite sign of UpwardsMassflux diagnostic GCHP mass flux wrong moisture correction
2 participants