Bug fix for building height in LCZ urban parameter table #1867
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Synchronize the develop branch from official WRF GitHub
sync up develop branch
|
The regression test results: |
dudhia
reviewed
May 23, 2023
dudhia
reviewed
May 23, 2023
|
I believe the previous values are from a specific study by Andrea Zonato. Now, we are updating these values based on the the middle-range global mean values of the urban LCZs from Stewart and Oke (2012), which is more applicable for future studies as suggested by Alberto Martilli. |
|
@cenlinhe Do you want this to be in 4.5.1? |
|
Yes, we want to include this bug fix as soon as possible. |
dudhia
approved these changes
Jul 5, 2023
smileMchen
approved these changes
Jul 5, 2023
vlakshmanan-scala
pushed a commit
to scala-computing/WRF
that referenced
this pull request
Apr 4, 2024
) Bug fix for building height in LCZ urban parameter table TYPE: bug fix KEYWORDS: urban, BEP, BEP_BEM, parameter table, building height SOURCE: Cenlin He (NCAR), Alberto Martilli (CIEMAT, Spain) DESCRIPTION OF CHANGES: Problem: The BEP and BEP_BEM urban schemes (urban_physics = 2,3) require the building height distribution to be a multiple of 5-m because of the hard-coded dz_u = 5 parameter in BEP and BEP_BEM source code. However, currently the urban parameter table (URBPARM_LCZ.TBL) has a building height distribution as a multiple of 3-m, which will cause model issues. Solution: Change the building height distribution in URBPARM_LCZ.TBL to be a multiple of 5-m. In addition to the model code requirement as mentioned above, the justifications of this change also include: (1) the NUDAPT database has building heights as a multiple of 5m. (2) the updated mean building heights in URBPARM_LCZ.TBL aer equal to the middle range values of the urban LCZs from Stewart and Oke (2012): Stewart, I. D., & Oke, T. R. (2012). Local climate zones for urban temperature studies. Bulletin of the American Meteorological Society, 93(12), 1879-1900. LIST OF MODIFIED FILES: run/URBPARM_LCZ.TBL TESTS CONDUCTED: The regression tests have passed. The mods fixed the problem. RELEASE NOTE: This is a bug fix for building height in LCZ urban parameter table.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bug fix for building height in LCZ urban parameter table
TYPE: bug fix
KEYWORDS: urban, BEP, BEP_BEM, parameter table, building height
SOURCE: Cenlin He (NCAR), Alberto Martilli (CIEMAT, Spain)
DESCRIPTION OF CHANGES:
Problem:
The BEP and BEP_BEM urban schemes (urban_physics = 2,3) require the building height distribution to be a multiple of 5-m because of the hard-coded dz_u = 5 parameter in BEP and BEP_BEM source code. However, currently the urban parameter table (URBPARM_LCZ.TBL) has a building height distribution as a multiple of 3-m, which will cause model issues.
Solution:
Change the building height distribution in URBPARM_LCZ.TBL to be a multiple of 5-m. In addition to the model code requirement as mentioned above, the justifications of this change also include: (1) the NUDAPT database has building heights as a multiple of 5m. (2) the updated mean building heights in URBPARM_LCZ.TBL aer equal to the middle range values of the urban LCZs from Stewart and Oke (2012): Stewart, I. D., & Oke, T. R. (2012). Local climate zones for urban temperature studies. Bulletin of the American Meteorological Society, 93(12), 1879-1900.
LIST OF MODIFIED FILES:
run/URBPARM_LCZ.TBL
TESTS CONDUCTED:
RELEASE NOTE: This is a bug fix for building height in LCZ urban parameter table