-
Notifications
You must be signed in to change notification settings - Fork 156
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: farm orientation Fix(north up) and matching plot turbine with fi #533
Conversation
Hi @rafmudaf, |
Hi @dhcho347 Yes, its best to work from the develop branch. As far as I can tell, you've done the git workflow correctly. |
Hi, @rafmudaf, How can I change this pull request with new one (fromDevelop branch)? |
Hi @dhcho347 I rebased this branch onto git fetch
git checkout visual_chk
# Optional: if you have any uncommitted changes, stash them and we'll reapply them later
git stash
# Check <remote> to the name of your repository in your local repository
git reset --hard <remote> visual_chk
# Optional: if you stashed changes, reapply them
# Note that you may have merge conflicts to resolve at this point
git stash pop |
@dhcho347 My apologies for all the noise here. |
Results plot with original layout (not rotate to west)
horizontal cut plane with north up farm layout infor with yaw, avg speed, Ct, power & Farm power
example for visual check chg : plot_turbine_with_fi
plot farm layout with info plot 3d for turbine grid with rotor swept circle plot flowfieldplanar grid
update visualization.py from develop branch
I believe this is superseded by #578 and #633. See #530 (comment). |
I'm closing this pull request. @dhcho347 please reopen or update it if you find I've closed it in error. |
Feature or improvement description
Related issue, if one exists
Fix orientation: #348, #357, #372
non-matching plot turbine with fi: #448, #451, #455, #512
Impacted areas of the software
v grid.py: class flowfiledplanar Grid method set_grid (Braoden flowfield, rotate mesh grid)
v utilities.py: ftn rotate_coordinates_rel_west() adding 1 parameter (inv_rot)
v floris_interface.py: method get_plane_of_points() (reverse rotating to north up)
v visualization.py: ftn plot_turbines_with_fi, plot_turbines
Additional supporting information
Test results, if applicable