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

Selectively invert plot axis via normal vector in CutPlane #384

Merged
merged 1 commit into from
Mar 9, 2022

Conversation

rafmudaf
Copy link
Collaborator

@rafmudaf rafmudaf commented Mar 9, 2022

Feature or improvement description
This pull request adds an option to invert the flow field and rotor plane plots. Specifically, floris currently plots y-planes from the perspective of downstream of the turbine and looking upstream. This pull request inverts the axes when we are plotting a CutPlane with x normal vector.

Below are plots of three turbines in a row with the leading turbine yawed with the latest develop and this pull request. The spanwise plots and the rotor plane plots are inverted.

Related issue, if one exists
None

Impacted areas of the software
Visualization tools and CutPlane class

Current develop branch

up1
up2

This pull request

down
down2

@rafmudaf rafmudaf added enhancement An improvement of an existing feature v3 Label to denote focus on v3 labels Mar 9, 2022
@rafmudaf rafmudaf requested a review from bayc March 9, 2022 18:43
@rafmudaf rafmudaf self-assigned this Mar 9, 2022
@paulf81
Copy link
Collaborator

paulf81 commented Mar 9, 2022

this is great, thanks @rafmudaf

@rafmudaf rafmudaf merged commit 6e4ef3f into NREL:develop Mar 9, 2022
@rafmudaf rafmudaf deleted the reverse_plotting branch March 11, 2022 21:18
@rafmudaf rafmudaf mentioned this pull request Mar 31, 2022
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An improvement of an existing feature v3 Label to denote focus on v3
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants