Update Noah-MP driver with new variables related to WRF-urban-nbs#229
Conversation
Update driver with new WRF-urban-nbs related variables
cenlinhe
left a comment
cenlinhe
left a comment
There was a problem hiding this comment.
The changes look good. This PR is to make the old non-refactored noahmp driver module that calls urban subroutine work with the new urban tree schemes added to WRF which includes 8 newly added urban variables. Nothing is changed from the NoahMP side. All changes are related to urban.
|
@tslin2 could you please take a quick look at this? This also needs a quick approval so that the WRF PR is able to be updated. Also, once our refactored coupling with WRF is approved, I will update these urban interface subroutine call in our refactored NoahMP code as well. For now, to make the non-refactored WRF PR pass regression test, we have to do this non-refactored code updates first, because the refactored coupling is delayed. |
|
@chenghaow Did you test the compilation with these Noah-MP code changes for your coupled WRF code? Is it successful? |
3 participants
This PR is related to a recent WRF PR based on v4.7.1: WRF-urban-NBS development. Updating module_sf_noahmpdrv.F with new WRF-urban-nbs related variables.
SOURCE: Yuqi Huang (University of Oklahoma), Chenghao Wang (University of Oklahoma)