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

Update config.*sh file templates for v2 release #714

Closed
jwolff-ncar opened this issue Mar 25, 2022 · 8 comments
Closed

Update config.*sh file templates for v2 release #714

jwolff-ncar opened this issue Mar 25, 2022 · 8 comments
Assignees
Labels
enhancement New feature or request high priority
Milestone

Comments

@jwolff-ncar
Copy link
Contributor

Description

Currently, config.*.sh files in regional_workflow/ush use GFSv15p2 physics suite - this will not be supported for v2 release

The path to the staged data (example of Hera is used here) in config*.sh points to
/scratch2/BMC/det/UFS_SRW_app/v1p0/model_data/FV3GFS
The purpose of having a static version (rather than pointing to develop) is that as things change going forward it will not break what is needed for the specific release.

Solution

Change GFSv15p2 to GFSv16 (and test!)

The v1p0 should be updated using one of two options 1) a copy of develop should be used to create a v2p0 or 2) point the example to develop.

@natalie-perlin
Copy link
Collaborator

@jwolff-ncar - is there a specific repo for the GFSv15p16 physics option?.. It is not found anywhere in the code when searching through the "develop" branch of ufs-weather-model/FV3/ccpp/physics/.

@gspetro-NOAA
Copy link
Collaborator

gspetro-NOAA commented Apr 20, 2022

@natalie-perlin I think you just need to update the config shell scripts in ufs-srweather-app/regional_workflow/ush and also any machine-specific config scripts in the SRW App repo to say GFSv16 instead of GFSv15p2. (Ideally, you'd also update anything else for the release, like new data/file paths and such, I believe.) But @jwolff-ncar can confirm.

@JeffBeck-NOAA
Copy link
Collaborator

@natalie-perlin, setting CCPP_PHYS_SUITE in your config.sh file tells the App to go grab that specific XML in the ufs-weather-model source directory and copy it to the run directory during the workflow generation step. It also determines the namelist structure and certain flags for pre-processing.

@JeffBeck-NOAA
Copy link
Collaborator

JeffBeck-NOAA commented Apr 27, 2022

This issue was resolved in PR #746, but it still needs to be merged into the release branch.

@natalie-perlin
Copy link
Collaborator

config.community.sh, config.nco.sh are updated.

@jwolff-ncar
Copy link
Contributor Author

@JeffBeck-NOAA It doesn't look like config.nco.sh is updated in the other PR so we need to make sure that also gets included prior to merge.

@JeffBeck-NOAA
Copy link
Collaborator

Thanks @jwolff-ncar, I just left a note in the PR conversation after seeing your comment. Hopefully @natalie-perlin can take a look.

@natalie-perlin
Copy link
Collaborator

Closing the issue, all updated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request high priority
Projects
None yet
Development

No branches or pull requests

6 participants