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

Add utility to build turbine yaml from absolute power, thrust curves #729

Merged
merged 14 commits into from Dec 13, 2023

Conversation

misi9170
Copy link
Collaborator

@misi9170 misi9170 commented Nov 2, 2023

Users have reported that often turbine data is provided in terms of absolute power (often paired with thrust coefficient) per wind speed.

This utility converts tables of wind speed coupled with absolute power and/or absolute thrust to a turbine input yaml file, containing the power and thrust coefficient tables.

Checklist:

  • Code runs
  • Tests built
  • Tests pass
  • Example to demonstrate usage. Is this needed/warranted?

@misi9170
Copy link
Collaborator Author

misi9170 commented Nov 7, 2023

I've altered the functionality a little to have the utility return a turbine dictionary (which can be passed to turbine_type in a fi.reinitialize() call), and optionally create a matching turbine yaml file (if file_path is specified).

I've also added an example.

@misi9170
Copy link
Collaborator Author

@rafmudaf some of this functionality (but not all of it) will become redundant when we switch to an absolute power curve in 4.0. I think there will still be useful aspects though. I think this could be put in for 3.6 as is, or if you'd like we can wait until 4.0/4.1 and I can revamp it a bit at that point.

@rafmudaf rafmudaf merged commit da61128 into NREL:develop Dec 13, 2023
8 checks passed
@misi9170 misi9170 mentioned this pull request Apr 5, 2024
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-feature A new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants