Skip to content

Remove bulk aoflux scheme from CMEPS and remove tests#3155

Merged
FernandoAndrade-NOAA merged 13 commits into
ufs-community:developfrom
DeniseWorthen:feature/remove_bulkaoflux
Mar 24, 2026
Merged

Remove bulk aoflux scheme from CMEPS and remove tests#3155
FernandoAndrade-NOAA merged 13 commits into
ufs-community:developfrom
DeniseWorthen:feature/remove_bulkaoflux

Conversation

@DeniseWorthen
Copy link
Copy Markdown
Collaborator

@DeniseWorthen DeniseWorthen commented Mar 20, 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:

Removes un-used "bulk" aoflux option in CMEPS and remove related DATM tests

Commit Message:

* UFSWM - remove tests using flux scheme -1
  * CMEPS - remove ocean_surface_flux_scheme -1 option

Priority:

  • Normal

Git Tracking

UFSWM:

Sub component Pull Requests:

UFSWM Blocking Dependencies:

  • None

Documentation:

  • Documentation update NOT required.
    • Explanation: RT update

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)

@github-project-automation github-project-automation Bot moved this to Evaluating in PRs to Process Mar 20, 2026
@DeniseWorthen DeniseWorthen linked an issue Mar 20, 2026 that may be closed by this pull request
@DeniseWorthen DeniseWorthen self-assigned this Mar 20, 2026
@DeniseWorthen DeniseWorthen marked this pull request as ready for review March 20, 2026 18:32
@gspetro-NOAA gspetro-NOAA added No Baseline Change No Baseline Change CMEPS There are changes to the CMEPS component repository. labels Mar 23, 2026
@gspetro-NOAA gspetro-NOAA moved this from Evaluating to Review in PRs to Process Mar 23, 2026
@gspetro-NOAA gspetro-NOAA moved this from Review to Schedule in PRs to Process Mar 23, 2026
@gspetro-NOAA
Copy link
Copy Markdown
Collaborator

@DeniseWorthen Could you please sync w/develop? We may process this one next since Lisa hasn't pushed her logs yet.

@gspetro-NOAA gspetro-NOAA added Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked. In Testing The PR that is currently in testing stages labels Mar 23, 2026
@FernandoAndrade-NOAA FernandoAndrade-NOAA removed the In Testing The PR that is currently in testing stages label Mar 24, 2026
@FernandoAndrade-NOAA FernandoAndrade-NOAA merged commit 382f0b3 into ufs-community:develop Mar 24, 2026
9 of 10 checks passed
@DeniseWorthen DeniseWorthen deleted the feature/remove_bulkaoflux branch April 24, 2026 12:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CMEPS There are changes to the CMEPS component repository. No Baseline Change No Baseline Change Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Remove "bulk" DATM tests

4 participants