-
Notifications
You must be signed in to change notification settings - Fork 10
FLOWERS subpackage integration #169
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
Open
cfrontin
wants to merge
179
commits into
NLRWindSystems:develop
Choose a base branch
from
cfrontin:feature/full_flowers_integration
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
179 commits
Select commit
Hold shift + click to select a range
43443c5
first commit
locascio-m 183e673
project completed
locascio-m f99f867
before debugging floris optimization
1fc74f8
floris optimization working
3bb1ef6
updating multistart code for eagle
47b2105
eagle setup
9dcd3d3
eagle output
939b8bf
parallel multistart
13dc28e
multistart setup
42096b4
new setup
b5dc63b
adjust multistart inputs
118612f
Merge branch 'main' of https://github.com/locascio-m/flowers
7088ed2
clean up folders
1b6b790
initial results completed
672b440
Merge branch 'main'
7df8071
adjusted inputs
1dbbcde
delete folder
9c3932d
fixed input path
82a5d7a
SNOPT configuration
713a611
fixes
e0548fb
split optimizations
37f49cc
scaling analysis setup
a12fd9a
added time limit
ab3dbbb
gauss optimization setup
f6e4bf2
added constraint scaling
2a6116e
fixes
d8fdb99
Merge branch 'main' of https://github.com/locascio-m/flowers into main
48d0305
new wind roses
locascio-m 25b4572
added wind roses
locascio-m 4934066
added aep comparison after vectorization
2d3fcf6
new code organization
6a688c9
scaling analysis
141aca4
snopt scaling
9af9ace
include verbose file
b32d127
flowers multistart
c8201d5
fix model
2bdff8f
boundary constraint fix
fa5fe43
working on multistart comparison
8f48914
stash model
858b179
Merge branch 'main' of https://github.com/locascio-m/flowers into main
924e45e
clean up multistart
78993ee
updated ignore
cb4fe92
adjusted floris opt setup
e294231
Merge branch 'main' of https://github.com/locascio-m/flowers into main
8d6250f
added file documentation
f54e646
merge conflict
29e3458
after multistart bug
44c7db6
Merge branch 'main' of https://github.com/locascio-m/flowers into main
5e13b5e
added resolution analysis
ed98a4b
Merge branch 'main' of https://github.com/locascio-m/flowers
39f6b1c
pyoptsparse debugging
f81d170
fourier resolution plots
7c878b4
testing branch
e029b19
testing branch again
87b2f5b
removed floris pyoptsparse wrapper
9762376
updated documentation
aa9be2c
multi analysis
ecd90fc
constraint visualization
666be3e
separate spacing constraint
6f78a4a
snopt workaround
501ea8f
Merge branch 'main' of https://github.com/locascio-m/flowers into main
b6f1088
snopt workaround
600c757
separating spacing constraint and setting up debug
6155c4e
update gitignore
5b73678
fixing discontinuity in aep
bae190d
troubleshoot
fc1b43b
model adjustment within rotor area
24458aa
update ignore
4d09261
constraint troubleshooting
cab64cf
Merge branch 'main' of https://github.com/locascio-m/flowers
5fce4b2
new multistart study with improved convergence
2b2c6d2
working on figures
1e2b000
working on figures
1493433
periodic fourier transform
locascio-m 3963e2a
Merge branch 'feature/pyopt'
locascio-m b10f4b2
new power calculation
locascio-m 7ad52c8
new power formulation and calibration
locascio-m ee889ba
found issue with power integral
locascio-m bf92999
park comparisons
locascio-m 1f01e86
added new comparison
locascio-m 49463ca
fixed case 4
locascio-m a90ab45
aep solution space visualization
locascio-m de958a8
Merge branch 'feature/power' of https://github.com/locascio-m/flowers…
locascio-m dc3d9f6
modified Park comparisons
locascio-m 44ae7f8
new feature plots
locascio-m 6af91ab
Merge branch 'feature/power' of https://github.com/locascio-m/flowers…
locascio-m 77bbf22
updated plotting
locascio-m 13f5896
new aep comparisons
locascio-m 6de3982
new aep comparisons
locascio-m 32389f3
updated aep comparison plots
locascio-m 229e798
larger range of conditions
locascio-m 51525c5
fixed saving
locascio-m f0ae803
changed gauss superposition
locascio-m 04a006d
Merge branch 'feature/power' of https://github.com/locascio-m/flowers…
locascio-m c23d29e
changed gauss superposition
locascio-m b00843e
validation/comparison work
locascio-m c8d2232
added flowers gradients, rebuilt code interface
locascio-m b9fa6c3
implemented analytic grads
locascio-m bfad3c8
snopt working and fixed constraint bugs
locascio-m ae85d65
optimization multistart
locascio-m e89fda5
clean up files and merge branches
locascio-m b1dd252
debugging floris imports
locascio-m f24047f
hpc multistart workflow
locascio-m 7861a7f
eagle multistart code
811c5fb
small, medium, large cases
locascio-m d8ca55e
fixed optimization convergence, added layouts
locascio-m 0b8c574
fixed slurm time limit
locascio-m a9f4c5c
deleted testing output files
locascio-m f4163e6
opt results plots
locascio-m c31011c
fixed stored layouts
locascio-m 1755fe3
locally copy layouts
locascio-m a8b61ec
delete layouts
locascio-m 2901822
new layouts
e487197
fixed time series flag
locascio-m 0d0ad3a
small convergence tolerance
locascio-m 6e5456d
medium tolerance and scaling
locascio-m 299ff28
multistart typo
locascio-m 04482a3
parameter sweep setup
locascio-m ed36d7a
large case scaling
locascio-m 77105e9
faster opt post-processing
locascio-m 46de06a
added aep and opt figs
locascio-m dcb729f
updated figures
locascio-m ee2cf8f
updated scripts for paper figures
locascio-m da5523e
convert FLOWERS project repo into python package
bayc 2c23f76
add initial aep test
bayc 4a79859
some changes to make the turbine more malleable
cfrontin 42efe9d
making density something that can be specified
cfrontin 0f7d3e2
Merge pull request #147 from WISDEM/develop
jaredthomas68 1d81242
Bump version from 0.1.0-beta0 to 0.1.0-beta1
cfrontin bcc1be4
Merge pull request #151 from WISDEM/develop
jaredthomas68 49b12da
Bump version from 0.1.0-beta1 to 0.1.0-beta2
cfrontin d6f0e38
my changes
cfrontin f24221a
move files into a configuration compatible with ard
cfrontin b429750
prep for flowers combination
cfrontin 92cb237
Merge remote-tracking branch 'flowers_for_merge/prep_for_ard_move' in…
cfrontin efadd40
adjust pyproject.toml to ensure flowers is installable
cfrontin 7ba6793
reorganize old tests to prepare for new ones
cfrontin aa8504d
black reformat plus refactoring test/example structure
cfrontin e4e1ccd
black reformat of the test problem
cfrontin 2f96007
reconfigure testing, add FLOWERS/FLORIS comparison sandbox for system…
cfrontin 3361bc5
added statistical test for FLORIS comparison to FLOWERS system tests.
cfrontin fb637a2
rename for nicer test printout
cfrontin 817ba02
Merge branch 'main' of github.com:WISDEM/Ard into develop
cfrontin b16bdaf
update for WISDEM/ORBIT pyrite standard value changes
cfrontin 58468d8
Merge branch 'develop' of github.com:WISDEM/Ard into develop
cfrontin 3046222
black v2026 reformat
cfrontin 3ffc6a6
Merge branch 'develop' of github.com:WISDEM/Ard into feature/full_flo…
cfrontin de6f004
Merge branch 'develop' into feature/full_flowers_integration
cfrontin 60c4737
get unit tests renamed w/ files and update unit testing script
cfrontin 923d7b8
updated unit test bonus script
cfrontin 8c26d82
Merge branch 'develop' of github.com:WISDEM/Ard into feature/full_flo…
cfrontin d96e6c4
clean unit test script
cfrontin 6fb0602
Merge branch 'develop' of github.com:WISDEM/Ard into feature/full_flo…
cfrontin cbbd4fd
add system test too now
cfrontin abcc0d9
update coverage omits and comments to pull legacy FLOWERS code out of…
cfrontin a44084b
black reformat
cfrontin 56a5749
add tools test but it doesn't do anything and errors out ha
cfrontin 16d6a27
testing progress
cfrontin 2cabc52
tools tested
cfrontin e9d8030
remove old github actions runner
cfrontin ee2b078
add derivative test to iea3.4mw
cfrontin 28712c8
break out the derivative test
cfrontin 87c0931
consolidated derivative test
cfrontin 63246f3
black reformat...
cfrontin 151c931
first tranche of copilot suggestions
cfrontin dfb02d4
second tranche of copilot suggestions
cfrontin f5dac07
adjust consolidated test script to run again, no new changes except m…
cfrontin 703de40
added flowers unit and system test to gh action
cfrontin ae98712
add wind rose pickle
cfrontin 11c75e5
adjust gha script for correct dependencies
cfrontin 50b3a3d
attempt fix for copilot import space pollution suggestion
cfrontin 76b10ce
remove == where isclose is more appropriate...
cfrontin 25f0cf2
remove legacy viz from coverage
cfrontin e707fe5
actually viz was already set aside...
cfrontin b676fe8
improved system tests
cfrontin 75b9b97
black reformat...
cfrontin 6cc86f4
cleanup system test
cfrontin 2adc1b8
walk back the house_style change which is now in its own branch
cfrontin 3e5a82b
Merge branch 'develop' into feature/full_flowers_integration
cfrontin File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,7 @@ | ||
| [run] | ||
| omit = | ||
| omit = | ||
| ard/utils/test_utils.py | ||
| ard/farm_aero/placeholder.py | ||
|
|
||
| flowers/visualization.py | ||
| flowers/optimization/optimization_interface.py | ||
| flowers/optimization/model_interface.py | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,5 +1,5 @@ | ||
|
|
||
| ### ARD DEVELOPMENT IGNORES | ||
| ### ARD DEVELOPMENT IGNORES | ||
|
|
||
| .vscode | ||
| case_files | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| import numpy as np | ||
| import pandas as pd | ||
|
|
||
| from flowers import FlowersModel | ||
|
|
||
| # Create generic layout | ||
| D = 126.0 | ||
| layout_x = D * np.array( | ||
| [0.0, 0.0, 0.0, 7.0, 7.0, 7.0, 14.0, 14.0, 14.0, 21.0, 21.0, 21.0, 28.0, 28.0, 28.0] | ||
| ) | ||
| layout_y = D * np.array( | ||
| [0.0, 7.0, 14.0, 0.0, 7.0, 14.0, 0.0, 7.0, 14.0, 0.0, 7.0, 14.0, 0.0, 7.0, 14.0] | ||
| ) | ||
|
|
||
| # Load in wind data | ||
| df = pd.read_csv("../data/HKW_wind_rose.csv") | ||
|
|
||
| # Setup FLOWERS model | ||
| flowers_model = FlowersModel(df, layout_x, layout_y) | ||
|
|
||
| # Calculate the AEP | ||
| aep = flowers_model.calculate_aep() | ||
| print(aep) |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why is model_interface.py omitted?