Skip to content

Fix premature finalizes and retrieval of is_moving info in ATM cap (#3093) // Aquaplanet test case (#3013)#3093

Merged
gspetro-NOAA merged 57 commits intoufs-community:developfrom
DeniseWorthen:feature/fixes_finalize_and_ismoving
Feb 20, 2026
Merged

Fix premature finalizes and retrieval of is_moving info in ATM cap (#3093) // Aquaplanet test case (#3013)#3093
gspetro-NOAA merged 57 commits intoufs-community:developfrom
DeniseWorthen:feature/fixes_finalize_and_ismoving

Conversation

@DeniseWorthen
Copy link
Copy Markdown
Collaborator

@DeniseWorthen DeniseWorthen commented Feb 13, 2026

Commit Queue Requirements:

  • This PR addresses a relevant WM issue (if not, create an issue).
  • All subcomponent pull requests (if any) have been reviewed by their code managers.
  • Run the full Intel+GNU RT suite (compared to current baselines), preferably on Ursa (Derecho or Hercules are acceptable alternatives). Exceptions: documentation-only PRs, CI-only PRs, etc.
    • Commit log file w/full results from RT suite run (if applicable).
    • Verify that test_changes.list indicates which tests, if any, are changed by this PR. Commit test_changes.list, even if it is empty.
  • Fill out all sections of this template.

Description:

#3093

Adds two small changes in UFSATM cap which resolve two edge use cases.

#3013

Within HSD project, new test case is developed called "Aquaplanet". We added that test to the ufs-weather-model/tests-dev/ directory.

Commit Message:

* UFSWM - added aquaplanet test under ufs-weather-model/tests-dev/ directory
  * UFSATM - add a barrier in the UFS ATM cap and reorder the retrieval of the is_moving info when intitializing the WCG

Priority:

  • Normal

Git Tracking

UFSWM:

Sub component Pull Requests:

UFSWM Blocking Dependencies:

  • None

Documentation:


Changes

Regression Test Changes (Please commit test_changes.list):

  • No Baseline Changes.

Input data Changes:

  • None.

Library Changes/Upgrades:

  • No Updates

Testing Log:

  • RDHPCS
    • Orion
    • Hercules
    • GaeaC6
    • Derecho
    • Ursa
  • WCOSS2
    • Dogwood/Cactus
    • Acorn
  • CI
  • opnReqTest (complete task if unnecessary)

@gspetro-NOAA
Copy link
Copy Markdown
Collaborator

gspetro-NOAA commented Feb 20, 2026

@DeniseWorthen The UFSATM PR has been merged. Please revert .gitmodules and update the UFSATM hash to 01bcf1d.

@gspetro-NOAA gspetro-NOAA removed the In Testing The PR that is currently in testing stages label Feb 20, 2026
@DeniseWorthen
Copy link
Copy Markdown
Collaborator Author

Why are we approving PRs where the checks are failing?

@FernandoAndrade-NOAA
Copy link
Copy Markdown
Collaborator

repo_check is failing due to AQM merging 124 and 125 without an accompanying UFS WM PR. The super linter failures are temporary and will need follow up changes after needing to update the version due to a security issue noted in #3072

@gspetro-NOAA
Copy link
Copy Markdown
Collaborator

gspetro-NOAA commented Feb 20, 2026

Why are we approving PRs where the checks are failing?

The Super Linter is something that I believe @BrianCurtis-NOAA is looking into a fix for. In PR #3072 he commented that "a security risk in older superlinter forced our hand a bit getting a newer version used in the CI. It can fail for now, but we'll want to work on a PR that adjusts things to pass again."
The Repo_check is failing because AQM just updated its develop branch outside of the PR process, so it's temporarily broken that CI.
Therefore, these two failures do not indicate a problem with the PR itself. I'll file issues though so that they are tracked more publicly.

@jkbk2004
Copy link
Copy Markdown
Collaborator

we should catch up AQM hash in next PR.

@gspetro-NOAA gspetro-NOAA merged commit 753c42f into ufs-community:develop Feb 20, 2026
12 of 16 checks passed
@DeniseWorthen DeniseWorthen deleted the feature/fixes_finalize_and_ismoving branch February 20, 2026 19:06
@gspetro-NOAA gspetro-NOAA mentioned this pull request Feb 20, 2026
32 tasks
@DeniseWorthen DeniseWorthen self-assigned this Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

No Baseline Change No Baseline Change UFSATM There are changes to the UFSATM repository.

Projects

Archived in project

9 participants