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

Dtc script mod #308

Merged
merged 2 commits into from
Apr 29, 2021
Merged

Dtc script mod #308

merged 2 commits into from
Apr 29, 2021

Conversation

hertneky
Copy link
Contributor

Fixes #307

Fixing a couple issues in run_upp script. This is purely a modification to the DTC run_upp script and there are no changes to code.

Fixes and issue with renaming files related to how the forecast hour was being defined.
Updates output filenames for LAM to be consistent with change in develop (NATLEV/PRSLEV)

Copy link
Contributor

@fossell fossell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Script looks good to me and has been tested. No changes to source code.

@WenMeng-NOAA WenMeng-NOAA merged commit e1c4098 into NOAA-EMC:develop Apr 29, 2021
@hertneky hertneky deleted the dtc_script_mod branch May 10, 2022 20:33
EricJames-NOAA pushed a commit to EricJames-NOAA/UPP that referenced this pull request Dec 14, 2022
## DESCRIPTION OF CHANGES:
This PR removes the USE_CCPP variable from all scripts and other files.  The workflow only supports running the FV3 model with CCPP, so USE_CCPP is deprecated.

## TESTS CONDUCTED: 
Ran one WE2E test (regional_002) on hera.  Succeeded.

## ISSUE (optional):
This resolves Issue NOAA-EMC#196.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DTC run_upp script modifications
3 participants