-
Notifications
You must be signed in to change notification settings - Fork 1
Test using oasis3-mct master branch #187
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
base: dev-preindustrial+concentrations
Are you sure you want to change the base?
Test using oasis3-mct master branch #187
Conversation
pr123-1 uses oasis3-mct built from the master branch and is otherwise identical to 2025.07.000
|
!test repro |
|
✅ The Bitwise Reproducibility Check Succeeded ✅ When comparing:
Further informationThe experiment can be found on Gadi at The checksums generated by this The checksums compared against are found here https://github.com/ACCESS-NRI/access-esm1.6-configs/tree/9fa7b7f4545bc6764bb175dcdf4b1b835f0fe8cf/testing/checksum Test summary: |
pr123-2 uses the oasis3-mct access-om2 spack version
|
!test repro |
|
❌ The Bitwise Reproducibility Check Failed ❌ When comparing:
Further informationThe experiment can be found on Gadi at The checksums generated by this The checksums compared against are found here https://github.com/ACCESS-NRI/access-esm1.6-configs/tree/9fa7b7f4545bc6764bb175dcdf4b1b835f0fe8cf/testing/checksum Test summary: |
|
This is a known issue that occurs randomly. Will rerun. |
|
!test repro |
|
✅ The Bitwise Reproducibility Check Succeeded ✅ When comparing:
Further informationThe experiment can be found on Gadi at The checksums generated by this The checksums compared against are found here https://github.com/ACCESS-NRI/access-esm1.6-configs/tree/9fa7b7f4545bc6764bb175dcdf4b1b835f0fe8cf/testing/checksum Test summary: |
|
I've done some basic testing of the performance impact of this change. I've run two experiments:
The experiments were run at the same time. Times below are the
So moving to using the
@manodeep, @harshula, @blimlim, @penguian, I think we agreed that with the above confirmed we are happy to use |
This PR is to test a prerelease build using the oasis3-mct master branch and demonstrate that this does not change answers.