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

An issue about average wind speed calculation #88

Open
xxh0523 opened this issue Dec 28, 2021 · 4 comments
Open

An issue about average wind speed calculation #88

xxh0523 opened this issue Dec 28, 2021 · 4 comments

Comments

@xxh0523
Copy link

xxh0523 commented Dec 28, 2021

Merry Christmas and Have a Good Holiday!

First of all, thank you for sharing SOWFA. It helps us a lot and we really appreciate it.

Recently, we have been using SOWFA to calculate average wind speed within a wind farm.
We found a possible bug since the computed average wind speed is much smaller than the instant one, which drew our attention.

After checking the source code in /applications/solvers/incompressible/windEnergy/windPlantSolver.ALM/computeAverageFields.H, we made modification as follows and obtained a result that seems reasonable.

Line 15:
Change
UAvg = (UAvg * avgTimeSum) + (runTime.deltaT().value() * U);
to
UAvg = (UAvg * (avgTimeSum - runTime.startTime().value())) + (runTime.deltaT().value() * U);
Line 27:
Change
UAvg = UAvg / avgTimeSum;
to
UAvg = UAvg / (avgTimeSum - runTime.startTime().value());

We hope to make sure whether it is really a bug, or it is just that we are not using your software in the right way.
Looking forward to hearing from you.

Sincerely,
Tannan

@ZY2xlllz
Copy link

Hello, I also find that the average speed of my calculation is very small. Do you know the reason? Is it a bug? Looking forward to your reply. Thank you very much.

@xxh0523
Copy link
Author

xxh0523 commented Apr 30, 2022

Hello, I also find that the average speed of my calculation is very small. Do you know the reason? Is it a bug? Looking forward to your reply. Thank you very much.

Hi. We think it might be a bug. So we modified the source code as we mentioned in the issue. You could try it and see if the results become reasonable.

@ZY2xlllz
Copy link

Thank you very much for your reply. I will try using your code. Wish you success in your research

@xxh0523
Copy link
Author

xxh0523 commented Apr 30, 2022

Thank you very much for your reply. I will try using your code. Wish you success in your research

You 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