RRFS_dev1: Turn on radar tten option and use "upgrade" to deploy.#121
Merged
Conversation
hu5970
commented
May 24, 2021
hu5970
commented
- Add and turn on radar tten option for RRFS_dev1
- use "PREEXISTING_DIR_METHOD=upgrade" to deploy RRFS_dev1
hu5970
requested review from
CurtisAlexander-NOAA,
JeffBeck-NOAA,
TrevorAlcott-NOAA,
christinaholtNOAA,
chunhuazhou,
jaymes-kenyon and
terraladwig
as code owners
Collaborator
|
This needed a newer version of the App, right? We should wait to merge until that is also available? |
|
|
||
| RUN_ENVIR="nco" | ||
| PREEXISTING_DIR_METHOD="rename" | ||
| PREEXISTING_DIR_METHOD="upgrade" |
Collaborator
There was a problem hiding this comment.
Should this not be done for all configs? Seems that we want to have as much consistency across configs as possible for smooth deployment.
Author
There was a problem hiding this comment.
We are trying to deploy dev1. I would like to change others when we need to deploy others. This can be a PR from other colleagues. Just want to make RRFS_dev1 deploy simple as we need to catch 22Z cold start.
Collaborator
There was a problem hiding this comment.
We need to keep these all consistent, especially in a space where everyone is learning. If the proper way to deploy a cycling run and do it smoothly is to use "upgrade" then all the cycling configs should be modified to that now.
Our other colleagues are still learning these things (including me). We were taught to use this setting in the last few sessions as the trusted method, so it should be the default to get a cycling method to run smoothly. Please go ahead and update them.
Author
|
Just updated hash for the app.
…On Mon, May 24, 2021 at 4:34 PM Christina Holt ***@***.***> wrote:
This needed a newer version of the App, right? We should wait to merge
until that is also available?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#121 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABVV23WKYSK2PLJR2NVPITTTPLH6RANCNFSM45OBUZXA>
.
|
christinaholtNOAA
approved these changes
May 24, 2021
hu5970
added a commit
that referenced
this pull request
May 29, 2021
* Feature/rrfs dev1: add retrospective function and changes for running on WCOSS (#104) * Cold start RRFS_dev1 3km CONUS hourly cycles from 09/21Z HRRR analysis. * Changes for running on RRFS cycles on WCOSS. * Copy addtional grid from UPP run directory to comout space. (#105) * Copy addtional grid from UPP run directory to comout space. * Improvements for 3-km NA (#103) * Extending walltime for "run_post" in 3-km NA * Updated settings for layout_x, layout_y to improve run time. * Increase RRFS_AK time step to match CONUS (#110) * Feature/rrfs dev1: add retro and radar tten functions (#108) * Add functions to run retro with staged data on disk. * Add namelist option for radar tten. * Bug fix for using GFS on JET for 3km NA. * Feature/rrfs dev1: delete radar tten option (#112) * delete tten option in namelist to match current USD model. * Turn off cloud analysis for future debug. (#113) * Change the "overwrite" option to "upgrade" (#111) * Change $EXPDIR backup from inside to outside (#117) * RRFS_dev1: Add python maps (#82) * Updating job script to work with maps. * Workflow for Python graphics. * Use a metatask to manage map subsets. Move skewt out of metatask. * Sorting out plotting the wgrib2-produced domains. * Updating configure scripts for appropriate domains. * Adding domains for 3DRTMA * Fixes for undefined variables and logic changes. * RRFS_dev1: Turn on radar tten option and use "upgrade" to deploy. (#121) * Add and turn on radar tten option * Change PREEXISTING_DIR_METHOD from "rename" to "upgrade" for all RRFS_dev runs. * Add configure options to decide how to clean the old cycles. (#123) Change the reference date for clean from current run time to current cycle time Co-authored-by: Jaymes Kenyon <jaymes.kenyon@noaa.gov> Co-authored-by: Trevor Alcott <69324222+TrevorAlcott-NOAA@users.noreply.github.com> Co-authored-by: Guoqing Ge <guoqing.ge@noaa.gov> Co-authored-by: Christina Holt <56881914+christinaholtNOAA@users.noreply.github.com>
EdwardColon-NOAA
pushed a commit
to EdwardColon-NOAA/regional_workflow
that referenced
this pull request
Sep 24, 2023
guoqing-noaa
pushed a commit
to guoqing-noaa/regional_workflow
that referenced
this pull request
Sep 15, 2024
…-GSL#121) Co-authored-by: ming hu <ming.hu@clogin01.cactus.wcoss2.ncep.noaa.gov>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.