-
Notifications
You must be signed in to change notification settings - Fork 155
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
Adding the TurbOPark wake velocity deficit model #396
Merged
Merged
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
bayc
added
enhancement
An improvement of an existing feature
v3
Label to denote focus on v3
labels
Mar 17, 2022
Closed
rafmudaf
reviewed
Mar 31, 2022
# Conflicts: # floris/simulation/solver.py
@bayc we also need to update the TurbOPark solver to use the |
* Use sorted flow field velocities * getting additional _sorted parameters Co-authored-by: bayc <christopher.j.bay@gmail.com>
rafmudaf
approved these changes
Mar 31, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
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.
Feature or improvement description
This pull request add the TurbOPark wake velocity deficit model, respective solver, and tests to the FLORIS package. For more details on the TurbOPark model, see: https://github.com/OrstedRD/TurbOPark.
Related issue, if one exists
Closes #297
Impacted areas of the software
Most of the
simulation
directory and parts ofFlorisInterface
.Additional supporting information
This is an initial implementation of the TurbOPark model. Due to the nature of the model, it does not currently support plotting of the wakes (flow field points are assumed to be at turbine locations). Additionally, the current implementation does not support wake steering. Either plotting or setting of yaw angles will cause an error to be issued when using this model. Also, the model is currently only meant to be used with the linear (FLS) wake combination method.