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

Fix PET. #352

Merged
merged 2 commits into from
Jan 20, 2016
Merged

Fix PET. #352

merged 2 commits into from
Jan 20, 2016

Conversation

tbohn
Copy link
Contributor

@tbohn tbohn commented Jan 20, 2016

This addresses issues #276 and #346. This replaces 6 output variables OUT_PET_* with a single OUT_PET. This replaces an internal array of 6 flavors of PET, handled as separate veg tiles, with a single scalar PET per veg tile (and elevation band). This PET corresponds most closely to the old OUT_PET_NATVEG in that it uses Penman-Monteith with the architectural resistance and LAI of the current vegetation, but it is now an area-weighted sum of potential transpiration from the current vegetation and potential soil evaporation.

Note that uncrustify resulted in minor changes to a couple of files that were otherwise not involved in this pull request.

This addresses issues UW-Hydro#276 and UW-Hydro#346.  This replaces 6 output variables OUT_PET_* with a single OUT_PET.  This replaces an internal array of 6 flavors of PET, handled as separate veg tiles, with a single scalar PET per veg tile (and elevation band).  This PET corresponds most closely to the old OUT_PET_NATVEG in that it uses Penman-Monteith with the architectural resistance and LAI of the current vegetation, but it is now an area-weighted sum of potential transpiration from the current vegetation and potential soil evaporation.
jhamman pushed a commit that referenced this pull request Jan 20, 2016
@jhamman jhamman merged commit 0cc67a0 into UW-Hydro:develop Jan 20, 2016
@tbohn tbohn deleted the feature/fix_pet_2 branch January 21, 2016 21:04
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