-
Notifications
You must be signed in to change notification settings - Fork 163
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
Link both global-nest fix files and non-nest ones at the same time #2632
Link both global-nest fix files and non-nest ones at the same time #2632
Conversation
@aerorahul and @WalterKolczynski-NOAA Thanks a lot for the discussion in today's meeting!
Where FYI, here are the output from
|
@guoqing-noaa I think changing it is still the way to go. Changing UFS Utils may require additional changes beyond scripts we use. @GeorgeGayno-NOAA will know for sure @GeorgeGayno-NOAA Short version: We're looking to go back to using |
That change was done in this PR: ufs-community/UFS_UTILS#899. Does the entire PR need to be reverted or just the |
Thanks, @WalterKolczynski-NOAA and @GeorgeGayno-NOAA Currently, the idea is to revert I will work on this. |
I will open a UFS_UTILS issue? Should I assign it to you or myself? |
Thanks, @GeorgeGayno-NOAA! It will be great if you can make changes for UFS_UTILS. And I can focus on making changes to the global workflow. |
@guoqing-noaa @GeorgeGayno-NOAA |
Great. I opened this issue: ufs-community/UFS_UTILS#955
|
Thanks, @aerorahul. I will update this PR sometime today and get ready for a review. |
@guoqing-noaa |
- link nest fix files when needed, i.e. keep the --nest option - change $FIXgfs/orog to $FIXorog - change $FIXgfs/ugwd to $FIXugwd
|
@aerorahul @WalterKolczynski-NOAA This PR is ready for review. Let me know if you have any comments. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good except the UFS_UTILS version needs to be updated. Let us know if you need guidance on how to update a submodule hash.
@WalterKolczynski-NOAA Thanks for the feedback. I just updated the UFS_UTILS hash. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Conditionally approved pending successful completion of CI tests.
CI Passed Hercules at |
|
Automated global-workflow Testing Results:
|
Experiment C48_ATM_49dcee8b SUCCESS on Wcoss2 at 06/11/24 10:48:11 PM |
Experiment C48_S2SW_49dcee8b SUCCESS on Wcoss2 at 06/11/24 11:06:09 PM |
Experiment C96C48_ufs_hybatmDA_49dcee8b SUCCESS on Wcoss2 at 06/11/24 11:57:11 PM |
Experiment C96C48_hybatmDA_49dcee8b SUCCESS on Wcoss2 at 06/12/24 12:03:23 AM |
CI Passed Hera at |
Experiment C96_atm3DVar_extended_49dcee8b SUCCESS on Wcoss2 at 06/12/24 05:42:29 AM |
All CI Test Cases Passed on Wcoss2:
|
2e6f1fc
into
NOAA-EMC:develop
…bal-workflow into feature/move_jcb * 'feature/move_jcb' of https://github.com/danholdaway/global-workflow: Add COM template for JEDI obs (NOAA-EMC#2678) Link both global-nest fix files and non-nest ones at the same time (NOAA-EMC#2632) Update ufs-weather-model (NOAA-EMC#2663) Add ability to process ocean/ice products specific to GEFS (NOAA-EMC#2561) Update cleanup job to use COMIN/COMOUT (NOAA-EMC#2649) Add overwrite to creat experiment in BASH CI (NOAA-EMC#2676) Add handling to select CRTM cloud optical table based on cloud scheme and update calcanal_gfs.py (NOAA-EMC#2645) Update RDHPCS Hera resource for `eupd` task (NOAA-EMC#2636)
* origin/develop: Add observation preparation job for aerosols DA to workflow (NOAA-EMC#2624) Remove ocean daily files (NOAA-EMC#2689) Update Jenkinsfile Add Hercules-EMC to the Jenkins configurable parameter list (NOAA-EMC#2685) Update gdas.cd and gsi_utils hashes (NOAA-EMC#2641) Add ability to use GEFS replay ICs (NOAA-EMC#2559) Replace `sleep` with `wait_for_file` (NOAA-EMC#2586) Add COM template for JEDI obs (NOAA-EMC#2678) Link both global-nest fix files and non-nest ones at the same time (NOAA-EMC#2632) Update ufs-weather-model (NOAA-EMC#2663) Add ability to process ocean/ice products specific to GEFS (NOAA-EMC#2561) Update cleanup job to use COMIN/COMOUT (NOAA-EMC#2649) Add overwrite to creat experiment in BASH CI (NOAA-EMC#2676) Add handling to select CRTM cloud optical table based on cloud scheme and update calcanal_gfs.py (NOAA-EMC#2645) Refs NOAA-EMC#2475
Description
This PR enables linking both global-nest fix files and non-nest ones at the same time and users can run both nesting and non-nesting experiments at the same time without worries about what fix files to be linked.
Resolves #2631
Type of change
Change characteristics
How has this been tested?
Checklist