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

uStar calculation #120

Open
bkelley9 opened this issue Dec 19, 2023 · 1 comment
Open

uStar calculation #120

bkelley9 opened this issue Dec 19, 2023 · 1 comment

Comments

@bkelley9
Copy link

I believe the code for checking if the wind height is above 10m should be if (wndHeight <= 10), not if (wndHeight != 10). In its current state it only checks if it is exactly equal to 10m.

Also, if a correction is being made to the wind speed if the above is true, shouldn't Cd be calculated after this correction, as it's value is dependent on wind speed being less than 5 m/s?

@jzwart
Copy link
Member

jzwart commented Jan 2, 2024

Hey @bkelley9 , thanks for the comment. I think the code is correct but the comment here:

# -- correct for wind measurement height if < 10 m (Amorocho and DeVries, 1980)

should be changed to correct for wind measurement if height does not equal 10 m (Amorocho and DeVries, 1980) . rLakeAnalyzer is based on MatLab version Lake-Analyzer, which has the same code implementation and correct comment - see here.

Good question about when to calculate Cd. The Cd calculation is based off of Hicks 1972 which says there isn't much of a dependence for wind speed height on the drag coefficient Cd. So I don't think it matters much that Cd is calculated prior to wind correction. @hdugan @riwoolway @jordansread might want to weigh in too

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

No branches or pull requests

2 participants