Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Release 2025.05.001#106

Merged
dougiesquire merged 2 commits into
mainACCESS-NRI/ACCESS-OM3:mainfrom
release-2025.05.001ACCESS-NRI/ACCESS-OM3:release-2025.05.001Copy head branch name to clipboard
Jun 4, 2025
Merged

Release 2025.05.001#106
dougiesquire merged 2 commits into
mainACCESS-NRI/ACCESS-OM3:mainfrom
release-2025.05.001ACCESS-NRI/ACCESS-OM3:release-2025.05.001Copy head branch name to clipboard

Conversation

@dougiesquire

@dougiesquire dougiesquire commented May 29, 2025 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Collaborator

This PR includes changes for the ACCESS-OM3 2025.05.001 release:

  • spack-packages 2025.05.002
  • access3/access3-share 2025.03.1
    • Set earth radius consistent with UM
  • access-cice CICE6.6.0-3
    • Add ability to write grid history file
  • access-mom6 2025.02.001
    • Add gtracers library to CMake build
  • Add access-generic-tracers git.dev-2025.05.001 and access-mocsy git.2017.12.0
  • Use spack versions where sensible; explicity set spack version where gitref is necessary
  • Set compiler optimization flags for improved performance

🚀 The latest prerelease access-om3/pr106-3 at ed23475 is here: #106 (comment) 🚀

@dougiesquire
dougiesquire requested a review from anton-seaice May 29, 2025 04:48
@dougiesquire

dougiesquire commented May 29, 2025

Copy link
Copy Markdown
Collaborator Author

@minghangli-uni can you please push the optimization flags to this branch?

@github-actions

github-actions Bot commented May 29, 2025

Copy link
Copy Markdown

🚀 Attempted to deploy access-om3 Prerelease pr106-1 with commit 592c6b2

🖥️ Gadi Deployment ✔️

Usage Instructions

This access-om3 model will be deployed to Gadi as:

  • 2025.05.001 as a Release (when merged).
  • pr106-1 as a Prerelease (during this PR).

This Prerelease is accessible on Gadi using:

module use /g/data/vk83/prerelease/modules
module load access-om3/pr106-1

When using the above modules, the binaries shall be on your $PATH.

For advanced users, this Prerelease is also accessible on Gadi via /g/data/vk83/prerelease/apps/spack/0.22/spack in the access-om3-pr106-1 environment.

Configuration Information

This Prerelease is deployed using:

  • access-nri/spack on branch 0.22
  • access-nri/spack-packages version 2025.05.002
  • access-nri/spack-config version 2025.02.2

If the above was not what was expected, commit changes to config/versions.json in this PR.

@github-actions

github-actions Bot commented May 29, 2025

Copy link
Copy Markdown

🚀 Attempted to deploy access-om3 Prerelease pr106-2 with commit 3fb7e58

🖥️ Gadi Deployment ✔️

Usage Instructions

This access-om3 model will be deployed to Gadi as:

  • 2025.05.001 as a Release (when merged).
  • pr106-2 as a Prerelease (during this PR).

This Prerelease is accessible on Gadi using:

module use /g/data/vk83/prerelease/modules
module load access-om3/pr106-2

When using the above modules, the binaries shall be on your $PATH.

For advanced users, this Prerelease is also accessible on Gadi via /g/data/vk83/prerelease/apps/spack/0.22/spack in the access-om3-pr106-2 environment.

Configuration Information

This Prerelease is deployed using:

  • access-nri/spack on branch 0.22
  • access-nri/spack-packages version 2025.05.002
  • access-nri/spack-config version 2025.02.2

If the above was not what was expected, commit changes to config/versions.json in this PR.

dougiesquire and others added 2 commits June 3, 2025 16:21
@dougiesquire
Update packages for ACCESS-OM3 2025.05.001
9fa7340 
- spack-packages 2025.06.000
- access3/access3-share 2025.03.1
  - Set earth radius consistent with UM
- access-cice CICE6.6.0-3
  - Add ability to write grid history file
- access-mom6 2025.02.001
  - Add gtracers library to CMake build
- Add access-generic-tracers git.dev-2025.05.001 and access-mocsy git.2017.12.0
- Use spack versions where sensible; explicity set spack version where gitref is necessary
@minghangli-uni @dougiesquire
Compiler optimisation flags for improved performance
ed23475 
---------

Co-authored-by: Dougie Squire <42455466+dougiesquire@users.noreply.github.com>
@dougiesquire
dougiesquire force-pushed the release-2025.05.001 branch from 3fb7e58 to ed23475 Compare June 3, 2025 06:23
@dougiesquire

dougiesquire commented Jun 3, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator Author

@minghangli-uni I also added the optimisation flags to access3-share, access-generic-tracers and access-mocsy and (crudely) checked that performance was not negatively impacted. @minghangli-uni could you check you're happy with what I've done.

@anton-seaice would you be able to review?

@github-actions

github-actions Bot commented Jun 3, 2025

Copy link
Copy Markdown

🚀 Attempted to deploy access-om3 Prerelease pr106-3 with commit ed23475

🖥️ Gadi Deployment ✔️

Usage Instructions

access-om3, defined in ./spack.yaml, will be deployed to Gadi as:

  • 2025.05.001 as a Release (when merged).
  • pr106-3 as a Prerelease (during this PR).

This Prerelease is accessible on Gadi using:

module use /g/data/vk83/prerelease/modules
module load access-om3/pr106-3

When using the above modules, the binaries shall be on your $PATH.

For advanced users, this Prerelease is also accessible on Gadi via /g/data/vk83/prerelease/apps/spack/0.22/spack in the access-om3-pr106-3 environment.

Configuration Information

This Prerelease is deployed using:

  • access-nri/spack on branch 0.22
  • access-nri/spack-packages version 2025.06.000
  • access-nri/spack-config version 2025.02.2

If the above was not what was expected, commit changes to config/versions.json in this PR.

minghangli-uni
minghangli-uni approved these changes Jun 3, 2025
View reviewed changes

@minghangli-uni minghangli-uni left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @dougiesquire The flags look good to me.

I tested them on both the cascadelake (normal) and sapphirerapids (normalsr) queues and they work on both architectures. However on the CL queue, performance is ~5% lower because the flags are tuned for SR maximal throughput. Since in production, all configurations will ultimately run on SR for its lower communication overhead.

Some notes:

  1. fms: I didn’t add flags here because I found that it actually degrades runtime performance. This is still a WIP, and I may revisit the compiler flags for fms later in the beta release.

  2. ACCESS-WW3: I haven’t tested this yet, so no flags are being added at the moment.

  3. Other dependencies: boost wasn’t detected, so no flags were added for those components.

anton-seaice
anton-seaice approved these changes Jun 3, 2025
View reviewed changes

@anton-seaice anton-seaice left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @dougiesquire and @minghangli-uni

@dougiesquire
dougiesquire merged commit 220e1ef into main Jun 4, 2025
@dougiesquire
dougiesquire deleted the release-2025.05.001 branch June 4, 2025 00:05
This was referenced Jun 4, 2025
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@minghangli-uni minghangli-uni minghangli-uni approved these changes
@anton-seaice anton-seaice anton-seaice approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dougiesquire @minghangli-uni @anton-seaice
Morty Proxy This is a proxified and sanitized view of the page, visit original site.