Skip to content

Update phenology file#133

Merged
blimlim merged 3 commits intodev-preindustrial+concentrationsfrom
109-PI-phenology-file
Jun 11, 2025
Merged

Update phenology file#133
blimlim merged 3 commits intodev-preindustrial+concentrationsfrom
109-PI-phenology-file

Conversation

@blimlim
Copy link
Copy Markdown
Collaborator

@blimlim blimlim commented Jun 10, 2025

Closes preindustrial half of #109 (AMIP changes to follow).

This PR points the configuration to the updated phenology file, allowing the number of phenology types to be read from the file rather than hardcoded.

Also includes changes to input manifests which were skipped in #131.

@blimlim
Copy link
Copy Markdown
Collaborator Author

blimlim commented Jun 10, 2025

!test repro

@github-actions
Copy link
Copy Markdown

✅ The Bitwise Reproducibility Check Succeeded ✅

When comparing:

  • 109-PI-phenology-file (checksums created using commit 9da3d22), against
  • dev-preindustrial+concentrations (checksums in commit 8331316)
Further information

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

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/15572393067/artifacts/3301150284.

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

@blimlim blimlim marked this pull request as ready for review June 10, 2025 23:35
Comment thread manifests/input.yaml
Comment thread manifests/restart.yaml
Copy link
Copy Markdown

@JhanSrbinovsky JhanSrbinovsky left a comment

Choose a reason for hiding this comment

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

LGTM

@blimlim blimlim merged commit 7a6bf9d into dev-preindustrial+concentrations Jun 11, 2025
11 checks passed
@access-hive-bot
Copy link
Copy Markdown

This pull request has been mentioned on ACCESS Hive Community Forum. There might be relevant details there:

https://forum.access-hive.org.au/t/coordinating-development-threads-for-next-phase-of-control-experiment/4523/1

@blimlim blimlim deleted the 109-PI-phenology-file branch August 7, 2025 06:46
anton-seaice pushed a commit that referenced this pull request Aug 7, 2025
* Update exe and filepath for phenology changes
anton-seaice pushed a commit that referenced this pull request Aug 7, 2025
* Update exe and filepath for phenology changes
anton-seaice pushed a commit that referenced this pull request Aug 15, 2025
* Update exe and filepath for phenology changes
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