Skip to content

Change MOM6_OUTPUT_FH variable name to MOM6_HISTFREQ_N + Update MOM6 with inline-CDEPS capability #3033#3044

Merged
gspetro-NOAA merged 14 commits intoufs-community:developfrom
dpsarmie:fix/standardize_mom6_output_freq_var
Jan 13, 2026
Merged

Change MOM6_OUTPUT_FH variable name to MOM6_HISTFREQ_N + Update MOM6 with inline-CDEPS capability #3033#3044
gspetro-NOAA merged 14 commits intoufs-community:developfrom
dpsarmie:fix/standardize_mom6_output_freq_var

Conversation

@dpsarmie
Copy link
Collaborator

@dpsarmie dpsarmie commented Jan 9, 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:

This PR will change the variable MOM6_OUTPUT_FH to MOM6_HISTFREQ_N.

This is for two reasons.

  1. This will standardize the MOM6 variable to the naming convention used by the CICE equivalent (CICE_HISTFREQ_N).
  2. There was a variable in Global Workflow that was called MOM6_OUTPUT_FH that was being used for other applications, so there was a conflict when the UFS hash was being updated in GW. This will minimize the work needed by GW to update the version of UFS in their workflow.

UFS PR #3033 :

Updates MOM6 to include CDEPS inline capability.

Commit Message:

* UFSWM - Change MOM6_OUTPUT_FH name to add consistency 
* MOM6 - enable CDEPS inline 

Priority:

  • High: PR also resolves conflict with Global Workflow

Git Tracking

UFSWM:

Sub component Pull Requests:

UFSWM Blocking Dependencies:

  • Blocked by #
  • None

Documentation:

  • Documentation update required.
    • Relevant updates are included with this PR.
    • A WM issue has been opened to track the need for a documentation update; a person responsible for submitting the update has been assigned to the issue (link issue).
  • Documentation update NOT required.

Changes

Regression Test Changes (Please commit test_changes.list):

  • PR Adds New Tests/Baselines.
  • PR Updates/Changes Baselines.
  • No Baseline Changes.

Input data Changes:

  • None.
  • PR adds input data.
  • PR changes existing input data.

Library Changes/Upgrades:

  • Required
    • Library names w/versions:
    • Git Stack Issue (JCSDA/spack-stack#)
  • No Updates

Testing Log:

  • RDHPCS
    • Hera
    • 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 Jan 9, 2026
@dpsarmie dpsarmie requested a review from DeniseWorthen January 9, 2026 14:23
@dpsarmie dpsarmie marked this pull request as ready for review January 9, 2026 14:28
DeniseWorthen
DeniseWorthen previously approved these changes Jan 9, 2026
@JessicaMeixner-NOAA
Copy link
Collaborator

Thanks @dpsarmie !

@gspetro-NOAA gspetro-NOAA moved this from Evaluating to Review in PRs to Process Jan 9, 2026
@gspetro-NOAA gspetro-NOAA moved this from Review to Schedule in PRs to Process Jan 12, 2026
@dpsarmie
Copy link
Collaborator Author

@gspetro-NOAA , brought in #3033 changes and tested on Hercules. I'll upload logs and test_changes.list when #3040 is merged.

I'll update the description in a bit.

@dpsarmie dpsarmie changed the title Change MOM6_OUTPUT_FH variable name to MOM6_HISTFREQ_N Change MOM6_OUTPUT_FH variable name to MOM6_HISTFREQ_N + Update MOM6 with inline-CDEPS capability #3033 Jan 12, 2026
@gspetro-NOAA
Copy link
Collaborator

@dpsarmie Please sync w/develop, and then we can start processing this PR. :)

@dpsarmie
Copy link
Collaborator Author

@gspetro-NOAA Ok, should be good to go. Thanks!

@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 Jan 12, 2026
@FernandoAndrade-NOAA FernandoAndrade-NOAA added the jenkins-ort run ORT testing label Jan 12, 2026
@epic-cicd-jenkins epic-cicd-jenkins removed the jenkins-ort run ORT testing label Jan 13, 2026
@gspetro-NOAA gspetro-NOAA removed the In Testing The PR that is currently in testing stages label Jan 13, 2026
@gspetro-NOAA
Copy link
Collaborator

@dpsarmie The WW3 sub-PR has been merged at hash 5b3f120. Please revert .gitmodules and update the hash.

@dpsarmie
Copy link
Collaborator Author

@gspetro-NOAA Done, should be ready for final reviews.

@gspetro-NOAA gspetro-NOAA merged commit 7b637a8 into ufs-community:develop Jan 13, 2026
9 checks passed
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 Priority: High 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.

Changing MOM6 variable to add consistency Add CDEPS inline capability to MOM6 NUOPC cap

7 participants