-
Notifications
You must be signed in to change notification settings - Fork 98
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
CALSLR_ROEBBER
: Instead of calling BOUND incorrectly, use min and max
#673
CALSLR_ROEBBER
: Instead of calling BOUND incorrectly, use min and max
#673
Conversation
CALSLR_ROEBBER
: Instead of calling BOUND incorrectly, don't call it at allCALSLR_ROEBBER
: Instead of calling BOUND incorrectly, use min and max
I just noticed I added a blank line by accident. The ufs-weather-model regression tests are already running, so I'd rather not remove the accidental blank line. It can't affect anything since the Fortran parser eliminates blank lines before the compiler sees them. |
Apparently the author year is wrong too. That seems somewhat more important than a blank line. |
@SamuelTrahanNOAA Can you sync your branch with the latest UPP develop? |
The UPP RT tests were completed on WCOSS2, Hera and Orion without changed results. This PR is ready for merging. |
Fixes #672 by calling min and max in CALSLR_ROEBBER instead of calling BOUND incorrectly.
Fixes #674 the same way.