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

Tropical Blended Rain #41

Merged
merged 3 commits into from
Aug 9, 2016
Merged

Tropical Blended Rain #41

merged 3 commits into from
Aug 9, 2016

Conversation

bdolan44
Copy link
Contributor

@bdolan44 bdolan44 commented Aug 8, 2016

Added tropical, oceanic blended logic. New 2DVD-derived relationships
at X, C, and S-band and convective /stratiform logic in Z-R branch.

Added tropical, oceanic blended logic. New 2DVD-derived relationships
at X, C, and S-band and convective /stratiform logic in Z-R branch.
Fixed formatting.
I added the tropical function to the main csu_blended_rain function.
@tjlang tjlang merged commit ff9172d into master Aug 9, 2016
@tjlang tjlang mentioned this pull request Aug 9, 2016
@tjlang
Copy link
Contributor

tjlang commented Aug 9, 2016

I had to make several fixes to get this to work correctly. The if statement logic was messed up so r_blended and method were being returned as NoneType if fhc was not None. Also, I changed the missing rain data return to 0 rather than -9999. Philosophically, the other routines return 0 in ice and related non-raining regions.

These changes were all implemented in #42. @bdolan44 and @pfhein please check my changes to ensure they match your intended logic.

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.

2 participants