Skip to content

Upgrade WM to spack-stack 2.1.x#368

Draft
gspetro-NOAA wants to merge 10 commits intoufs-community:ufs/devfrom
gspetro-NOAA:feature/ss2.x
Draft

Upgrade WM to spack-stack 2.1.x#368
gspetro-NOAA wants to merge 10 commits intoufs-community:ufs/devfrom
gspetro-NOAA:feature/ss2.x

Conversation

@gspetro-NOAA
Copy link
Copy Markdown

@gspetro-NOAA gspetro-NOAA commented Apr 2, 2026

Description of Changes:

Contains ccpp/physics code changes from EPIC that are required to upgrade the WM to spack-stack v2.1.0

Tests Conducted:

In progress.
We will be testing the RTs and troubleshooting failing tests with developers. This may lead to separate PRs from those developers to ensure that their tests work with the new stack, esp. LLVM compilers.

Dependencies:

Add any links to parent PRs (e.g. SCM and/or UFS PRs) or submodules (e.g. rte-rrtmgp). For example:

Documentation:

Does this PR add new capabilities that need to be documented or require modifications to the existing documentation? If so, brief supporting material can be provided here. Contact the CODEOWNERS if your PR requires extensive updates to the documentation. See https://github.com/NCAR/ccpp-doc for Technical Documentation or https://dtcenter.org/community-code/common-community-physics-package-ccpp/documentation for the latest Scientific Documentation.

Issue (optional):

If this PR is resolving or referencing one or more issues, in this repository or elsewhere, list them here. For example, Related to ufs-community/ufs-weather-model#3067

Contributors (optional):

@RatkoVasic-NOAA @rickgrubin-noaa

Comment thread CMakeLists.txt Outdated
target_link_libraries(ccpp_physics PRIVATE MPI::MPI_Fortran)
target_link_libraries(ccpp_physics PUBLIC w3emc::w3emc_d
sp::sp_d
ip::ip_d
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Didn't @grantfirl or someone from DTC make a similar (albeit more flexible) change recently in NCAR ccpp-physics main?

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.

Yes, I was just testing a combined PR syncing NCAR/main (which already contains the IP changes) back to this branch. See #369 for the draft PR.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Awesome, thanks! I will take a look and incorporate here. This is mostly a placeholder PR at the moment; it may be that this PR will get closed out if others take care of the work. I'm trying to replicate Rick and Ratko's WM RT results and then go from there to see what else needs to be done. I was just getting the potential sub-PRs & issues open for visibility/tracking.

Comment thread CMakeLists.txt
@gspetro-NOAA gspetro-NOAA changed the title Upgrade WM to spack-stack 2.1.0 Upgrade WM to spack-stack 2.1.x Apr 29, 2026
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.

5 participants