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

Populate mid-run checkpoints with non-species data every timeset #1442

Merged

Conversation

lizziel
Copy link
Contributor

@lizziel lizziel commented Oct 17, 2022

The update in this PR fixes a GCHP bug in which non-species entries in mid-run restart files (aka checkpoint files) were incorrect. All internal state data is now updated every timestep, with the except of area which is only updated in the first timestep since it is time-invariant.

This update will have no impact on GCHP 1-month benchmarks or end-of-run restart files. It will correct a problem in 1-year GCHP benchmarks which used mid-run checkpoint files to retrieve area for generating mass tables. It will also correct error introduced when restarting a GCHP simulation from a previously generated mid-run checkpoint file to continue a series of runs. This would happen if, for example, a run crashed mid-way through the configured run duration due to a compute cluster issue rather than a model issue.

yantosca and others added 5 commits October 11, 2022 14:49
run/GCClassic/README
- removed

run/GCClassic/README.md
- Added w/ updated links for 14.0.0

Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
We have now added the link to GEOS-Chem Supporting Guidelines.

Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
run/GCClassic/geoschem_config.yml.templates/geoschem_config.yml.CH4
run/GCClassic/geoschem_config.yml.templates/geoschem_config.yml.tagCH4
- Set the "analytical_inversion/activate:true" flag to false.  The
  analyical inversion is only used for the Integrated Methane Inversion
  (aka IMI).  The IMI will toggle this to true.

Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
…mestep

Previously non-species data was only updated in the internal state at
the end of the run.

Signed-off-by: Lizzie Lundgren <elundgren@seas.harvard.edu>
@lizziel lizziel added category: Bug Something isn't working topic: GCHP Related to GCHP only labels Oct 17, 2022
@lizziel lizziel added this to the 14.0.1 milestone Oct 17, 2022
@lizziel lizziel self-assigned this Oct 17, 2022
@lizziel lizziel marked this pull request as ready for review October 17, 2022 18:31
@msulprizio msulprizio changed the base branch from dev to patch/14.0.1 October 18, 2022 13:56
@msulprizio msulprizio merged commit db479fc into patch/14.0.1 Oct 18, 2022
@msulprizio msulprizio deleted the bugfix/set_all_intstate_ptrs_for_midrun_checkpoints branch October 18, 2022 13:58
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: GCHP Related to GCHP only
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants