-
Notifications
You must be signed in to change notification settings - Fork 162
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
Fix NST_SPINUP being ignored by CROW #350
Fix NST_SPINUP being ignored by CROW #350
Conversation
The NSST config file was setting NST_SPINUP to use an environment variable rather than using the value set by the yaml, so values set in the case file were ignored. Refs: NOAA-EMC#337
Note: I didn't actually run the model, I just confirmed that the config.nsst file is now exporting the value set in the yaml. |
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 correct based on my understanding of CROW yaml.
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.
See also commit d16630f
Going to merge this one, since it is clearly a bug, then we can deal with any additional changes needed for NSST separately. |
GitHub Issue NOAA-EMC/GSI#341. Update the use of MW radiances for GFS v16.3 implementation
GitHub Issue NOAA-EMC/GSI#341. Update the use of MW radiances for GFS v16.3 implementation
…ve snet from noah lsm, fix time dimension in restart files (NOAA-EMC#350) - Update of submodule pointers for ccpp-physics for the changes listed below - Thompson inner loop from @RuiyuSun - Thompson subcycling bugfix from @climbfuji - Noah/NoahMP bugfix/remove snet from Noah LSM from @HelinWei-NOAA - Fix time dimension in restart files from @climbfuji Co-authored-by: Helin Wei <Helin.Wei@noaa.gov> Co-authored-by: Ruiyu Sun <Ruiyu.Sun@noaa.gov>
The NSST config file was setting NST_SPINUP to use an environment variable rather than using the value set by the yaml, so values set in the case file were ignored.
Refs: #337