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

Enable linearization with MHK turbines #2431

Merged
merged 2 commits into from
Sep 25, 2024

Conversation

andrew-platt
Copy link
Collaborator

Ready to merge

Feature or improvement description
When linearization for MHK turbines was added (PRs #1882, #2014, and #2181), it was not enabled. This PR turns on this feature, but adds a warning.

While the MHK linearization is now possible, there are several key things missing that will reduce the accuracy of the results:

  • Added mass is not included in any AeroDyn calculations (pending PR for v5.0.0: MHK wave-current superposition, added mass, inertia #2213)
  • Buoyancy calculations are not included in perturbation. This may not change results much as buoyancy is included in the operating point and is likely a small effect in the linearization compared to the missing added mass.

It should also be noted that linearization of the MHK RM1 floating turbine is extremely slow. This will be improved on significantly with v5.0.0 when MHK linearization is fully supported.

Related issue, if one exists
#1882
#2014
#2181

Impacted areas of the software
Linearization with MHK turbines.

There are pieces still missing for good MHK linearization:
- buoyancy is not included in the perturbations
- added mass is not included in AeroDyn at all
@andrew-platt andrew-platt merged commit a5c11f2 into OpenFAST:dev Sep 25, 2024
21 checks passed
@andrew-platt andrew-platt deleted the b/enable_MHK_lin branch November 1, 2024 20:16
@andrew-platt andrew-platt mentioned this pull request Dec 24, 2024
35 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants