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

Feature: Expand capabilities of UncertaintyInterface #428

Merged
merged 1 commit into from
May 26, 2022

Conversation

Bartdoekemeijer
Copy link
Collaborator

This PR is ready to be merged.

Feature or improvement description
Expands functionality of UncertaintyInterface: everything that is in FlorisInterface besides the visualizations. We also streamline the no_wake implementation from FlorisInterface, which had changed in a recent pull request.

Related issue, if one exists
Issue #426.

Impacted areas of the software
tools.uncertainty_interface

Additional supporting information
N/A

Test results, if applicable
Example 05, with and without uncertainty:
image

Where uncertainty spreads the wake laterally, and thus we see a higher power production at the second turbine.

Example 07 works if we replace FlorisInterface with UncertaintyInterface, and the results are sensible. With FlorisInterface:

Farm AEP (default options): 57.900 GWh
Farm AEP (with cut_in/out specified): 57.895 GWh
Farm AEP (no_wake=True): 58.472 GWh

With UncertaintyInterface:

Farm AEP (default options): 57.820 GWh
Farm AEP (with cut_in/out specified): 57.816 GWh
Farm AEP (no_wake=True): 58.396 GWh

Also, example 17:
image

@Bartdoekemeijer Bartdoekemeijer added bug Something isn't working enhancement An improvement of an existing feature v3 Label to denote focus on v3 labels Apr 22, 2022
@Bartdoekemeijer Bartdoekemeijer added this to the v3.2.0 milestone Apr 22, 2022
@Bartdoekemeijer Bartdoekemeijer self-assigned this Apr 22, 2022
@Bartdoekemeijer Bartdoekemeijer merged commit 0697c37 into NREL:develop May 26, 2022
@Bartdoekemeijer Bartdoekemeijer deleted the feature/246 branch May 26, 2022 17:11
@rafmudaf rafmudaf mentioned this pull request Sep 12, 2022
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement An improvement of an existing feature v3 Label to denote focus on v3
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature request: write 'assign_hub_height_to_ref_height' for 'UncertaintyInterface'
2 participants