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

Update checksums from #176 #180

Closed
wants to merge 2 commits into from

Conversation

anton-seaice
Copy link
Contributor

@anton-seaice anton-seaice commented Feb 18, 2025

We need to manually trigger updates to historical repro checksums when they change in dev branches (this is only enforced for release branches).

@anton-seaice
Copy link
Contributor Author

!test repro commit

@anton-seaice anton-seaice changed the title Update checksums from #173 Update checksums from #176 Feb 18, 2025
Copy link

❌ The Bitwise Reproducibility Check Failed ❌

When comparing:

  • 274-dev-1deg_jra55do_ryf (checksums created using commit 0ecfee0), against
  • dev-1deg_jra55do_ryf (checksums in commit e45b8f6)

🔧 The checksums will be committed to this PR, as they differ.

Further information

The experiment can be found on Gadi at /scratch/tm70/repro-ci/experiments/access-om3-configs/0ecfee042fb670c490867d1ad3c41ec9a1f91e4d, and the test results at https://github.com/ACCESS-NRI/access-om3-configs/runs/37374705106.

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/13382749239/artifacts/2606653825.

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

Copy link

!test Command Failed ❌ There was a problem committing the result of the reproducibility run. See https://github.com/ACCESS-NRI/access-om3-configs/actions/runs/13382749239

@ACCESS-NRI ACCESS-NRI deleted a comment from github-actions bot Feb 18, 2025
@dougiesquire
Copy link
Collaborator

!test repro commit

Copy link

❌ The Bitwise Reproducibility Check Failed ❌

When comparing:

  • 274-dev-1deg_jra55do_ryf (checksums created using commit 0ecfee0), against
  • dev-1deg_jra55do_ryf (checksums in commit e45b8f6)

🔧 The checksums will be committed to this PR, as they differ.

Further information

The experiment can be found on Gadi at /scratch/tm70/repro-ci/experiments/access-om3-configs/0ecfee042fb670c490867d1ad3c41ec9a1f91e4d, and the test results at https://github.com/ACCESS-NRI/access-om3-configs/runs/37377136445.

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/13383597204/artifacts/2606900457.

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

@dougiesquire
Copy link
Collaborator

@CodeGat, the above workflow committed to the ./testing directory, not ./testing/checksum

@CodeGat
Copy link
Member

CodeGat commented Feb 18, 2025

Thanks @dougiesquire, looking into it ACCESS-NRI/model-config-tests#104

@CodeGat
Copy link
Member

CodeGat commented Feb 18, 2025

I've got an open PR to fix the bug. If you don't mind, would you be able to drop the commit by access-bot before running !test repro commit? (Once the update to the test command is merged)

@dougiesquire dougiesquire force-pushed the 274-dev-1deg_jra55do_ryf branch from 60b8ee0 to 0ecfee0 Compare February 18, 2025 23:18
@dougiesquire
Copy link
Collaborator

I've got an open PR to fix the bug. If you don't mind, would you be able to drop the commit by access-bot before running !test repro commit? (Once the update to the test command is merged)

Done

@dougiesquire
Copy link
Collaborator

!test repro commit

Copy link

❌ The Bitwise Reproducibility Check Failed ❌

When comparing:

  • 274-dev-1deg_jra55do_ryf (checksums created using commit 0ecfee0), against
  • dev-1deg_jra55do_ryf (checksums in commit e45b8f6)

🔧 The checksums will be committed to this PR, as they differ.

Further information

The experiment can be found on Gadi at /scratch/tm70/repro-ci/experiments/access-om3-configs/0ecfee042fb670c490867d1ad3c41ec9a1f91e4d, and the test results at https://github.com/ACCESS-NRI/access-om3-configs/runs/37435757515.

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/13402116646/artifacts/2612946260.

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

@dougiesquire
Copy link
Collaborator

Closing this, as these checksums were updated in the meantime using the "Generate Initial Checksum" workflow - see a706052

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