Skip to content

Conversation

@minghangli-uni
Copy link
Collaborator

@minghangli-uni minghangli-uni commented Jun 5, 2025

1. Summary:

Why was this done?
This is a follow-up PR to update checksums for #562

2. Issues Addressed:

3. Depedencies (e.g. on payu, model or om3-scripts)

This change requires changes to (note required version where true):

  • payu:
  • access-om3:
  • om3-scripts:

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. Documentation

The docs folder has been updated with output from running the model?

  • Yes
  • N/A

A PR has been created for updating the documentation?

  • Yes:
  • N/A

8. Formatting

Changes to MOM_input have been copied from model output in docs/MOM_parameter_docs.short?

  • Yes
  • N/A

9. Merge Strategy

  • Merge commit
  • Rebase and merge
  • Squash

@minghangli-uni
Copy link
Collaborator Author

!test repro commit

@github-actions
Copy link

github-actions bot commented Jun 5, 2025

❌ The Bitwise Reproducibility Check Failed ❌

When comparing:

  • 457-update-checksum (checksums created using commit b4c8f9b), against
  • dev-MC_25km_jra_ryf (checksums in commit c33a731)

🔧 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-om3-configs/b4c8f9b0311642b41a84ecef51d118159a7b351b, and the test results at https://github.com/ACCESS-NRI/access-om3-configs/runs/43527342026.

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

The checksums compared against are found here https://github.com/ACCESS-NRI/access-om3-configs/tree/c33a7310177944300c199cf30e7069831d4abf5b/testing/checksum

@minghangli-uni minghangli-uni merged commit a905c6f into dev-MC_25km_jra_ryf Jun 5, 2025
11 checks passed
@minghangli-uni minghangli-uni deleted the 457-update-checksum branch June 5, 2025 23:03
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.

4 participants