Skip to content

Conversation

@minghangli-uni
Copy link
Collaborator

@minghangli-uni minghangli-uni commented May 19, 2025

1. Summary:

The provided MOM6 parameters apply only to the 0.25deg configuration. This PR is intended ONLY to trigger the CI workflow for repro tests

DO NOT MERGE

2. Issues Related:

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

 Update MEKE
module MOM_thickness_diffuse
 no GM parameterisation
 Update horizontal mixing coefficients
 Update background viscosity to 0
 VELOCITY_TOLERANCE set to default
 Disable Leith and enable biharmonic Smagorinsky eddy viscosity
 Change to Bodner et al. 2023 formulation of the restratifying mixed-layer restratification parameterisation.
module MOM_diabatic_driver
 Implicit energetics PBL scheme to determine diffusivity and viscosity in the PBL
module MOM_CVMix_KPP
 Disabled
module MOM_CVMix_conv
 Disabled
module MOM_CVMix_shear
 Disabled
module MOM_CVMix_ddiff
 Disabled
module MOM_kappa_shear
 Jackson's shear-driven turbulence
module MOM_energetic_PBL
 Enable ePBL and disable KPP
 Add SIMPLE_TKE_TO_KD, but can not be used reliably with USE_REGRIDDING?
 Reduce BBL_EFFIC from default 0.2 to 0.01
 Add background KD, KD_MIN, Henyey_igw_background latitude scaling for near surface background diffusivity
 Add double diffusion
 Remove KHTR_MIN & Remove MAX_TR_DIFFUSION_CFL
 Disable nuetral diffusion
 Disable horizontal diffusion of tracers near boundaries
 Reduce SEA_ICE_RIGID_MASS from 1000 to 100
 Enable max limit of surface pressure
 Remove constant unresolved background gust for ustar
 Disable surface pressure in EOS
 Enable a dynamic pressure due to viscous ice shelves
@minghangli-uni
Copy link
Collaborator Author

!test repro

@github-actions
Copy link

❌ The Bitwise Reproducibility Check Failed ❌

When comparing:

  • CI-check-after-#556 (checksums created using commit d9a7555), against
  • dev-MC_100km_jra_ryf (checksums in commit 3724a14)
Further information

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

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/15105067373/artifacts/3148969993.

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

@minghangli-uni
Copy link
Collaborator Author

!test repro

@github-actions
Copy link

❌ The Bitwise Reproducibility Check Failed ❌

When comparing:

  • CI-check-after-#556 (checksums created using commit 0060071), against
  • dev-MC_100km_jra_ryf (checksums in commit 3724a14)
Further information

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

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/15105195127/artifacts/3149009926.

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

@anton-seaice
Copy link
Collaborator

test_repro_restart (test-venv.lib.python3.10.site-packages.model_config_tests.test_bit_reproducibility.TestBitReproducibility) failed

@minghangli-uni
Copy link
Collaborator Author

!test repro

@github-actions
Copy link

❌ The Bitwise Reproducibility Check Failed ❌

When comparing:

  • CI-check-after-#556 (checksums created using commit 7dc5a88), against
  • dev-MC_100km_jra_ryf (checksums in commit 3724a14)
Further information

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

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/15287778487/artifacts/3208466960.

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

@minghangli-uni
Copy link
Collaborator Author

minghangli-uni commented May 27, 2025

7dc5a88b320764d5e0e9e88d231c5e418c90dbaf]$ git diff --no-index restart-1d-1-checksum.json restart-2d-0-checksum.json
diff --git a/restart-1d-1-checksum.json b/restart-2d-0-checksum.json
index fa2809e..c119eda 100644
--- a/restart-1d-1-checksum.json
+++ b/restart-2d-0-checksum.json
@@ -83,7 +83,7 @@
       "9ACD58C5D22AE195"
     ],
     "v": [
-      "BF83DB1845E3AE14"
+      "3F83DB1845E3AE14"
     ],
     "v2": [
       "CA8D417951F37114"

Okay, by reducing the cpl timestep from 3600s to 1800s which is now same as mom dynamic timestep, problem solved. Now we have a new bug(?) for the restart repro?

@dougiesquire dougiesquire deleted the CI-check-after-#556 branch September 24, 2025 22:43
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