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

Add no_nsst CCPP suites #69

Merged

Conversation

climbfuji
Copy link
Collaborator

Based on #67, but use different names (no_nsst instead of no_NST).

Associated PR: ufs-community/ufs-weather-model#61 - see there for testing.

@climbfuji
Copy link
Collaborator Author

I believe it is safe to merge this PR already (but not yet the ufs-weather-model). But we can also wait until all tests are completed.

@junwang-noaa
Copy link
Collaborator

I believe it is safe to merge this PR already (but not yet the ufs-weather-model). But we can also wait until all tests are completed.

I believe so, but let's get these suite files tested before we commit them.

@climbfuji
Copy link
Collaborator Author

I need to create a new baseline on hera first and ask you to move it. Will take a bit. Cheyenne+GNU is running.

@climbfuji
Copy link
Collaborator Author

All tests pass, this PR is ready to merge.

@junwang-noaa junwang-noaa merged commit b9554ae into NOAA-EMC:release/public-v1 Feb 20, 2020
@junwang-noaa
Copy link
Collaborator

junwang-noaa commented Feb 20, 2020 via email

@climbfuji
Copy link
Collaborator Author

Done, please go ahead and merge this PR - thanks, Jun!

SamuelTrahanNOAA referenced this pull request in SamuelTrahanNOAA/fv3atm Jun 13, 2022
…elop_20210331

Update gsl/develop from develop 2021/03/31
LarissaReames-NOAA pushed a commit to LarissaReames-NOAA/fv3atm that referenced this pull request Nov 17, 2023
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.

4 participants