Feature: farm orientation Fix(north up) and matching plot turbine with fi#533
Feature: farm orientation Fix(north up) and matching plot turbine with fi#533dhcho347 wants to merge 8 commits intoNatLabRockies:developfrom
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