Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Land IAU parameters to default_vars / control_c48_lnd_iau modifications // #2585 Update for WW3 initialization #2587

Merged
merged 18 commits into from
Feb 5, 2025

Conversation

dpsarmie
Copy link
Collaborator

@dpsarmie dpsarmie commented Feb 1, 2025

Commit Queue Requirements:

  • Fill out all sections of this template.
  • All sub component pull requests have been reviewed by their code managers.
  • Run the full Intel+GNU RT suite (compared to current baselines) on either Hera/Derecho/Hercules
  • Commit 'test_changes.list' from previous step

Description:

This PR will add the Land IAU default values to default_vars.sh. The control_c48_lnd_iau test was using a separate version of the global namelist. The test will be changed to use the global_control.nml.IN namelist. The regression test will not have its baselines changed with these modifications.

This PR also combined the following from #2585:
When using PIO restarts, we needed to add a capability to read binary restarts when using runtype=intial. For example, this is used in the high resolution (HR) experiments.

Commit Message:

* UFSWM - Add Land IAU variables to default_vars and allow control_c48_lnd_iau to run using the global_control.nml.IN template.
   * WW3 -  Updated hash for WW3 initialization  (#2585)

Priority:

  • Normal

Git Tracking

UFSWM:

Sub component Pull Requests:

UFSWM Blocking Dependencies:

  • None

Changes

Regression Test Changes (Please commit test_changes.list):

  • No Baseline Changes.

Input data Changes:

  • None.

Library Changes/Upgrades:

  • No Updates

Testing Log:

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

This allows the control_c48_lnd_iau test to be run
off of the global_control.nml.IN namelist and adds
the Land IAU defaults to default_vars.sh

Even though PERT_CLDS was set in the original test config,
the namlist did not have this so the option was not turned on.
The test settings reflect the original version of the test and
no baseline changes have been made.
@dpsarmie dpsarmie added the No Baseline Change No Baseline Change label Feb 1, 2025
@dpsarmie dpsarmie changed the title Add Land IAU parameters to default_vars / control_c48_lnd_iau modifications Add Land IAU parameters to default_vars / control_c48_lnd_iau modifications // #2585 Update for WW3 initialization Feb 3, 2025
@dpsarmie dpsarmie mentioned this pull request Feb 3, 2025
15 tasks
@dpsarmie
Copy link
Collaborator Author

dpsarmie commented Feb 3, 2025

WW3 changes from 2585 have been added and PR description was modified.

@jkbk2004 jkbk2004 added the jenkins-ort run ORT testing label Feb 4, 2025
@DeniseWorthen
Copy link
Collaborator

Why is the Hercules log so odd? It's the "old style" log file, with the list of files from each test appearing w/ an 'OK' etc.

@jkbk2004 jkbk2004 removed the jenkins-ort run ORT testing label Feb 5, 2025
@DeniseWorthen
Copy link
Collaborator

Why is the Hercules log so odd? It's the "old style" log file, with the list of files from each test appearing w/ an 'OK' etc.

@jkbk2004

@DeniseWorthen
Copy link
Collaborator

@jkbk2004 Explanation?

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Feb 5, 2025

@jkbk2004 Explanation?

pushed wrong hercules log and fixed with correct one

@DeniseWorthen
Copy link
Collaborator

But how did you generate a log like that? As far as I know, rt.sh will no longer generate a log like that.

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Feb 5, 2025

But how did you generate a log like that? As far as I know, rt.sh will no longer generate a log like that.

rocoto and eclfow are not fully functional on hercules. a few jobs continue to be dropped and rt.sh jusy hangs. @RatkoVasic-NOAA please, re-install rocoto on hercules.

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Feb 5, 2025

But how did you generate a log like that? As far as I know, rt.sh will no longer generate a log like that.

rocoto and eclfow are not fully functional on hercules. a few jobs continue to be dropped and rt.sh just hangs. and manually recovered log on hercules. but all the rt.conf jobs finished ok. @RatkoVasic-NOAA please, re-install rocoto on hercules.

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Feb 5, 2025

All tests are done We can start merging process.

@DeniseWorthen
Copy link
Collaborator

What does running ecflow or rocoto have to do w/ generating a log in the older style?

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Feb 5, 2025

What does running ecflow or rocoto have to do w/ generating a log in the old

@jkbk2004 jkbk2004 closed this Feb 5, 2025
@jkbk2004 jkbk2004 reopened this Feb 5, 2025
@jkbk2004
Copy link
Collaborator

jkbk2004 commented Feb 5, 2025

What does running ecflow or rocoto have to do w/ generating a log in the older style?

if ecflow/rocoto doesn't exit successfully, rt.sh hangs and no log is produced.

@DeniseWorthen
Copy link
Collaborator

a) An issue should be created if that is the case. b) it should be noted that you manually created the log.

@JessicaMeixner-NOAA
Copy link
Collaborator

WW3 has been merged: NOAA-EMC/WW3@8e67627

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Feb 5, 2025

It looks ecflow runs ok on hercules: #2454. We will double check in next pr.

@jkbk2004
Copy link
Collaborator

jkbk2004 commented Feb 5, 2025

@dpsarmie can you update ww3 hash and revert changes in gitmodules ?

@dpsarmie
Copy link
Collaborator Author

dpsarmie commented Feb 5, 2025

@dpsarmie can you update ww3 hash and revert changes in gitmodules ?

Done, let me know if you need anything else.

@RatkoVasic-NOAA
Copy link
Collaborator

RatkoVasic-NOAA commented Feb 5, 2025

But how did you generate a log like that? As far as I know, rt.sh will no longer generate a log like that.

rocoto and eclfow are not fully functional on hercules. a few jobs continue to be dropped and rt.sh just hangs. and manually recovered log on hercules. but all the rt.conf jobs finished ok. @RatkoVasic-NOAA please, re-install rocoto on hercules.

@jkbk2004 rocoto on Orion and Hercules are maintained by Chris Harrop:

hercules: /apps/contrib/rocoto> ll
total 0
drwxr-x--- 10 charrop noaa-hpc 268 May 11  2023 1.3.5
drwxr-x--- 10 charrop noaa-hpc 268 Feb  1  2024 1.3.6
drwxr-x--- 10 charrop noaa-hpc 268 Jun  3  2024 1.3.7
lrwxrwxrwx  1 charrop noaa-hpc   5 Jun 21  2024 default -> 1.3.7

@jkbk2004 jkbk2004 merged commit ee3d52d into ufs-community:develop Feb 5, 2025
4 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 Ready for Commit Queue The PR is ready for the Commit Queue. All checkboxes in PR template have been checked.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Land IAU values to default_vars Git ignore list missing some build products
9 participants