Skip to content

Add info on atmosphere restart creation tool#604

Merged
Whyborn merged 2 commits intomainfrom
add-land-restart-documentation
Mar 23, 2026
Merged

Add info on atmosphere restart creation tool#604
Whyborn merged 2 commits intomainfrom
add-land-restart-documentation

Conversation

@Whyborn
Copy link
Copy Markdown
Collaborator

@Whyborn Whyborn commented Mar 20, 2026

1. Summary:

Documentation has changed to describe the atmosphere restart creation tool.

2. Issues Addressed:

3. Dependencies (e.g. on payu, or model)

This change requires changes to (note pull request(s) where relevant):

  • workflow manager (payu):
  • model deployment (ACCESS-ESM1.6):
  • model component or library dependency:
  • input workflow:

4. Ad-hoc Testing

What ad-hoc testing was done? How are you convinced this change is correct (plots are good)?

5. CI Testing

  • !test repro has been run

6. Reproducibility

Is this reproducible with the previous commit? (If not, why not?)

  • Yes
  • No - !test repro commit has been run.

7. Performance

Has the model performance (say, throughput of model-years/wall-day) changed?

  • Yes
  • No
  • N/A (if selected, please add a brief explanation why performance testing is not necessary for this PR)

If yes, provide the numbers from your testing. Is the performance better or worse?

8. Manifests

Have you changed the executable, the input files and/or the restart files?

  • Yes
  • No

If yes, have you updated the manifests?

  • Yes
  • No

To update the manifests, run payu setup (in a cloned copy of your feature branch) with reproducibility tests turned off:

manifest:
  reproduce:
    exe: false
    input: false
    restart: false
runlog:
  enable: false

Then commit the newly created manifest files (under manifests/) only to the branch for this PR.

9. Documentation

Is the documentation updated?

  • Yes
  • N/A

10. Merge Strategy

  • Merge commit
  • Rebase and merge
  • Squash

@Whyborn Whyborn requested review from blimlim and ccarouge March 20, 2026 00:21
Copy link
Copy Markdown
Collaborator

@blimlim blimlim left a comment

Choose a reason for hiding this comment

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

Thanks @Whyborn, I think this reads well. Just commented a couple small suggestions and typos.

Comment thread documentation/docs/pages/inputs/restarts.md Outdated
Comment thread documentation/docs/pages/inputs/restarts.md Outdated
@Whyborn
Copy link
Copy Markdown
Collaborator Author

Whyborn commented Mar 23, 2026

Fixed those typos and minor rewording.

Copy link
Copy Markdown
Collaborator

@blimlim blimlim left a comment

Choose a reason for hiding this comment

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

Thanks @Whyborn, looks good!

@Whyborn Whyborn merged commit 642f8f7 into main Mar 23, 2026
22 checks passed
@anton-seaice anton-seaice deleted the add-land-restart-documentation branch March 24, 2026 04:22
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