Skip to content

Set UM print status to PrStatus_Normal#321

Merged
blimlim merged 1 commit intodev-preindustrial+concentrationsfrom
292-um_printstatus
Dec 19, 2025
Merged

Set UM print status to PrStatus_Normal#321
blimlim merged 1 commit intodev-preindustrial+concentrationsfrom
292-um_printstatus

Conversation

@blimlim
Copy link
Copy Markdown
Collaborator

@blimlim blimlim commented Dec 19, 2025

1. Summary:

This PR changes the UM PRINT_STATUS setting from PrStatus_Diag to PrStatus_Normal. This reduces the amount of output written to the UM atm.fort6.pe* log files, primarily removing excess oasis logging. See here for details on the changes to the logs: https://github.com/ACCESS-NRI/UM7/issues/195

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:

Requires UM PR: https://github.com/ACCESS-NRI/UM7/pull/198 for the change to take full effect

4. Ad-hoc Testing

What ad-hoc testing was done? How are you convinced this change is correct (plots are good)?
See testing in https://github.com/ACCESS-NRI/UM7/pull/198

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

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 Dec 19, 2025

!test repro

@github-actions
Copy link
Copy Markdown

✅ The Bitwise Reproducibility Check Succeeded ✅

When comparing:

  • 292-um_printstatus (checksums created using commit 383aadc), against
  • dev-preindustrial+concentrations (checksums in commit e23db53)
Further information

The experiment can be found on Gadi at /scratch/tm70/repro-ci/experiments/access-esm1.6-configs/383aadc10a7880eb7bedf0e74a1c31b5056e8ee1, and the test results at https://github.com/ACCESS-NRI/access-esm1.6-configs/runs/58497362054.

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/20357914326/artifacts/4919760730.

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

Test summary:
test_repro_historical
test_repro_determinism

@blimlim blimlim requested a review from MartinDix December 19, 2025 02:59
Copy link
Copy Markdown
Collaborator

@MartinDix MartinDix left a comment

Choose a reason for hiding this comment

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

OK

@blimlim blimlim marked this pull request as ready for review December 19, 2025 04:08
@blimlim blimlim merged commit e5960e7 into dev-preindustrial+concentrations Dec 19, 2025
11 of 12 checks passed
@blimlim blimlim deleted the 292-um_printstatus branch December 23, 2025 01:34
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.

2 participants