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

added support for CICE6 thermo state rewind for replay #854

Merged
merged 3 commits into from
Nov 29, 2023

Conversation

zhaobin74
Copy link
Contributor

@zhaobin74 zhaobin74 commented Nov 15, 2023

This PR adds a customized entry point method to support restoring CICE6 state variables such that they can be reset before corrector step starts. The reason for saving and restoring CICE6 states is that predictor step updates these states via callbacks even though the seaice_gridcomp is not running.

This PR also fixed a bug in Record method which doesn't handle dumping MAPL states properly.

This PR is zero diff and it depends on GEOS-ESM/GEOSgcm#702

@zhaobin74 zhaobin74 added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Nov 15, 2023
@zhaobin74 zhaobin74 marked this pull request as ready for review November 15, 2023 13:56
@zhaobin74 zhaobin74 requested review from a team as code owners November 15, 2023 13:56
Copy link
Collaborator

@tclune tclune left a comment

Choose a reason for hiding this comment

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

cmake change ok.

@sanAkel sanAkel self-requested a review November 15, 2023 16:57
@sanAkel sanAkel removed their request for review November 27, 2023 18:08
@sanAkel
Copy link
Contributor

sanAkel commented Nov 27, 2023

@sdrabenh I got a confirmation from @zhaobin74 that he verified in both configurations:

  • With replay: it worked for him to rewind.
  • Without replay: answers preserved.

I'll not be reviewing this PR.

@sdrabenh
Copy link
Collaborator

@sdrabenh I got a confirmation from @zhaobin74 that he verified in both configurations:

  • With replay: it worked for him to rewind.
  • Without replay: answers preserved.

I'll not be reviewing this PR.

Sounds good. My AGCM tests passed... but obviously not a test of CICE

@sdrabenh sdrabenh merged commit f65854b into develop Nov 29, 2023
3 checks passed
@sdrabenh sdrabenh deleted the feature/zhaobin74/rewind-thermo-states branch November 29, 2023 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0 diff The changes in this pull request have verified to be zero-diff with the target branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants