-
Notifications
You must be signed in to change notification settings - Fork 508
Allow separate teeter inertia for 2-bladed turbines #2981
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
Conversation
|
I'll look at the history - there may be something missing in the regression test branch (missing merge). |
|
|
This PR changes the computation for 2 bladed turbines slightly. Implemention in this PR: Note that we have switched from g-axes to f-axes for hub inertia in rest of the code. The following plots demonstrate this findings. Labels: For a 3-bladed turbine, as f and g axes are same and the implmentation remains the same: These tests should pass. If you both (@andrew-platt , @jjonkman ) agree with the code changes in the PR, I can upload new .outb files to the few test that are failing. Abhineet |
87a2a5d to
be17f97
Compare
3faedb7 to
195d8a9
Compare
|
As the definition of inputs change with this PR, I have updated the r-test ElastoDyn files with new values for PS: This PR also has contributions from @faisal-bhuiyan. |
jjonkman
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for making this change. See a couple minor comments below.


Feature or improvement description
OpenFAST assumes that hub for a two-bladed turbine is a rod and the hub-inertia about the rotor axis and the teeter axis is the same. This PR addresses this shortcoming allowing users to provide the two inertia's separate.
Computation for three-bladed turbines are not affected.
Related issue, if one exists
Impacted areas of the software
ElastoDyn
Additional supporting information
Test results, if applicable