-
Notifications
You must be signed in to change notification settings - Fork 3
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
Companion PR to NOAA-EMC/jcb/pull/7 #15
Conversation
* develop: Update atmosphere-lgetkf satwnd yamls (#13)
* develop: Yaml updates for use with updated GDASApp submodules (#8)
model/atmosphere/atmosphere_background_error_hybrid_gsibec_bump.yaml.j2
Outdated
Show resolved
Hide resolved
model/atmosphere/atmosphere_background_error_hybrid_gsibec_bump.yaml.j2
Outdated
Show resolved
Hide resolved
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.
Approve.
@danholdaway: Can we merge this PR into |
@RussTreadon-NOAA I'm not sure about merging because it will confuse what is being tested in g-w. If ultimately we need jcb and gdasapp on develop in order to test and merge NOAA-EMC/global-workflow#2665 then merging this will then mean testing these changes in that PR. I would prefer to merge that PR (and associated jcb/gdas PRs) first. Unless I'm missing something if we merge these now then we may have to end up testing them with NOAA-EMC/global-workflow#2665. |
* feature/no_sub_mods: add comment action debugging action debugging action debugging action debugging action debugging action debugging action debugging action debugging action debugging action debugging Make sure to init clients in GH test pyyaml Adjust testing for new path checker file
Companion PR: NOAA-EMC/jcb/pull/7
atmosphere_<class>.yaml.j2
{{atmosphere_<key>}}