Skip to content

Point to a UKCA tag instead of a commit hash#33

Closed
Whyborn wants to merge 1 commit intomainfrom
pin-ukca-to-tag
Closed

Point to a UKCA tag instead of a commit hash#33
Whyborn wants to merge 1 commit intomainfrom
pin-ukca-to-tag

Conversation

@Whyborn
Copy link
Copy Markdown
Collaborator

@Whyborn Whyborn commented Apr 7, 2026

Points to the UKCA tag which is the current head of the um13.1_blackcarbon_cable_deposition branch, which contains the changes required for CABLE. There is no functional change- the 2026.03.0 tag points to the same commit as is used currently.


🚀 The latest prerelease access-am3/pr33-1 at 7e36627 is here: #33 (comment) 🚀

@Whyborn Whyborn requested a review from bschroeter April 7, 2026 01:07
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 7, 2026

The model version in the spack.yaml has not been updated.
You can update it manually.
Alternatively, comment the following to have it updated and committed automatically:

  • !bump major for feature releases
  • !bump minor for bugfixes

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 7, 2026

🚀 Attempted to deploy access-am3 Prerelease pr33-1 with commit 7e36627

🖥️ Gadi Deployment ❌

@Whyborn
Copy link
Copy Markdown
Collaborator Author

Whyborn commented Apr 7, 2026

Thoughts on upping the version- there is no functional change here. The UKCA change is also dated based on when the tag was created, rather than when the last modification was, so it's currently at a "higher" dated version than AM3 itself, which would be a bit strange. Thoughts @bschroeter? Might just be easier to up the AM3 version

@Whyborn
Copy link
Copy Markdown
Collaborator Author

Whyborn commented Apr 7, 2026

Actually, I have this PR which is ready to go- confirmed bitwise compatibility between builds using this configuration. I can just add the UKCA reference change there?

@MartinDix
Copy link
Copy Markdown
Collaborator

Should the tag be called am3_2026.03.0? Just the date seems too much like a release version rather than a branch.

@Whyborn
Copy link
Copy Markdown
Collaborator Author

Whyborn commented Apr 7, 2026

Fair point- especially if CM3/rAM3 may be using the main branch for some time before CABLE is included. I'll make that change.

@Whyborn
Copy link
Copy Markdown
Collaborator Author

Whyborn commented Apr 8, 2026

I've decided to merge this change into this PR for simplicity. This is now superseded.

@Whyborn Whyborn closed this Apr 8, 2026
@Whyborn Whyborn temporarily deployed to Gadi Prerelease April 8, 2026 00:25 — with GitHub Actions Inactive
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