Skip to content

Update preindustrial+concentrations restart#699

Merged
blimlim merged 2 commits intodev-preindustrial+concentrationsfrom
698-update-pi-restart
Apr 15, 2026
Merged

Update preindustrial+concentrations restart#699
blimlim merged 2 commits intodev-preindustrial+concentrationsfrom
698-update-pi-restart

Conversation

@blimlim
Copy link
Copy Markdown
Collaborator

@blimlim blimlim commented Apr 15, 2026

1. Summary:
This PR updates the preindustrial+concentrations configuration to use a copy of /g/data/p73/archive/CMIP7/ACCESS-ESM1-6/production/Ndep2-PI-CNP-concentrations/restart170, with the date changed to 1/1/1.

This change is required for the new preindustrial simulation.

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.

Change in restart will change the checksums.

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)

Change in restart won't affect performance.
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

@blimlim
Copy link
Copy Markdown
Collaborator Author

blimlim commented Apr 15, 2026

!test repro commit

@blimlim blimlim changed the title Update restart to restart170 of Ndep 2 test run Update preindustrial+concentrations restart Apr 15, 2026
@github-actions
Copy link
Copy Markdown

❌ The Bitwise Reproducibility Check Failed ❌

When comparing:

  • 698-update-pi-restart (checksums created using commit 2f6da30), against
  • dev-preindustrial+concentrations (checksums in commit 37d5312)

🔧 The new checksums will be committed to this PR, if they differ from what is on this branch.

Further information

The experiment can be found on Gadi at /scratch/tm70/repro-ci/experiments/access-esm1.6-configs/pr699/698-update-pi-restart/2f6da309a1dc5231de2478f83f5f1fc5722c4ed6, and the test results at https://github.com/ACCESS-NRI/access-esm1.6-configs/runs/71382819111.

The checksums generated by this !test command are found in the testing/checksum directory of https://github.com/ACCESS-NRI/access-esm1.6-configs/actions/runs/24432877771/artifacts/6442005502.

The checksums compared against are found here https://github.com/ACCESS-NRI/access-esm1.6-configs/tree/37d5312847642f1ea574306d200449aa8b44fd39/testing/checksum

Test summary:
test_repro_historical
test_repro_determinism

@blimlim blimlim requested a review from JhanSrbinovsky April 15, 2026 02:40
Copy link
Copy Markdown

@JhanSrbinovsky JhanSrbinovsky left a comment

Choose a reason for hiding this comment

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

thanks

@blimlim blimlim merged commit aa8e59a into dev-preindustrial+concentrations Apr 15, 2026
12 checks passed
@blimlim blimlim deleted the 698-update-pi-restart branch April 15, 2026 02:47
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.

3 participants