-
Notifications
You must be signed in to change notification settings - Fork 161
UFS-dev PR#195 #1153
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
UFS-dev PR#195 #1153
Conversation
… into a subdir v1
… the associated files
… into feature/gfdlmpv3
… into feature/gfdlmpv3
… into feature/gfdlmpv3
This reverts commit 3115d2c.
… into feature/gfdlmpv3
… into feature/gfdlmpv3
…pp-physics into feature/gfdlmpv3
… into feature/gfdlmpv3
… into feature/gfdlmpv3
… into feature/gfdlmpv3
…-physics into feature/gfdlmpv3
Combo PR (merge from NCAR/main, ECMWF GWD, maxmf bugfix, NEPTUNE reproducibility, mersenne bugfix)
physics/Interstitials/UFS_SCM_NEPTUNE/GFS_suite_interstitial_4.F90
Outdated
Show resolved
Hide resolved
@climbfuji All comments have been addressed. Since the ufs-dev PR has already been merged, these changes will make their way back during the next time we sync. |
@dustinswales was the author for a lot of this, so he probably doesn't need to (re)approve. Rhaesung also approved when this was going into ufs/dev. |
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.
Thanks for addressing my review comments!
Description of Changes:
Identical to ufs-community#195 - adds TEMPO and GFDL MP v3
Tests Conducted:
UFS RTs were run as part of merging Identical to ufs-community#195; all tests passed except new tests added
Dependencies:
None
Documentation:
Added some SCM documentation regarding downloading the TEMPO data for that model.
Issue (optional):
None
Contributors (optional):
@dustinswales @AndersJensen-NOAA @RuiyuSun