Skip to content

FAST.Farm Interface#94

Open
abhineet-gupta wants to merge 64 commits intoNatLabRockies:developfrom
abhineet-gupta:FFInterface
Open

FAST.Farm Interface#94
abhineet-gupta wants to merge 64 commits intoNatLabRockies:developfrom
abhineet-gupta:FFInterface

Conversation

@abhineet-gupta
Copy link
Collaborator

@abhineet-gupta abhineet-gupta commented Feb 26, 2026 •
edited
Loading

This PR integrates ROSCO based WFC into hycon (e.g FAST.Farm).

misi9170 added 30 commits April 23, 2025 16:20
@misi9170
Add interface for Hercules in LongSim mode.
d88ee46
@misi9170
Reorganize Hercules interfaces library
5428209
@misi9170
Add ramp rate option. Not yet tested.
e3e3930
@misi9170
Merge changes from develop
70dcf3a
@misi9170
Rename as v1 and v2 in anticipation of upcoming Hercules release
495885c
@misi9170
Reorganize Hercules interface tests and add v2 Hercules tests.
0ed29b7
@misi9170
Building out hybrid interface and test
bcea874
@misi9170
Fix import ordering
a2d59ab
@misi9170
Update hybrids interface and add tests/
54a5d28
@misi9170
Merge develop and resolve conflicts.
d2e3183
@misi9170
Update imports in examples
e85b061
@misi9170
Explicit naming for wind-only interface
c439dc1
@misi9170
Create base Hercules interface and save controller and plant paramete…
f7b4d59 
…rs there
@misi9170
Ruff formatting
d64a23c
@misi9170
Add in outline of controller; not yet tested
18f5b11
@misi9170
Rename to respect conventions
9eef950
@misi9170
Merge branch 'feature/long-run-interface' into feature/split-hybrid-c…
c599f21 
…ontrol
@misi9170
Add to controllers import
35c3ba4
@misi9170
Pull in most changes to v2 interfaces from paulf81 branch
c251bc2
@misi9170
Pull in changes from paulf81 branch
3be2fef
@misi9170
Comments added to note possible issues
a8abf3a
@misi9170
Can safely remove yaw_angles since HerculesADInterface will set these…
d623f8f 
… to default if it doesnt receive them
@misi9170
dt as a variable on interface
4958404
@misi9170
Reignore input csvs that are generated at runtime
d456c47
@misi9170
Update tests and implement fixes
a4a42e3
@misi9170
Temp update controller to work with LongRun
c07eb17
@misi9170
Get component capacities from plant_parameters and size mismatch erro…
e61cc4e 
…r explicit.
@misi9170
num_turbines -> n_turbines, power_setpoints -> wind_power_setpoints
49500e3
@misi9170
Fix tests where appropriate
3bfb2b7
@misi9170
power_references update (not complete, likely breaks examples and tests)
8be9dc8
@abhineet-gupta abhineet-gupta changed the base branch from develop to main February 26, 2026 10:02
@abhineet-gupta abhineet-gupta changed the base branch from main to develop February 26, 2026 10:03
@abhineet-gupta abhineet-gupta changed the base branch from develop to main February 26, 2026 10:03
@abhineet-gupta abhineet-gupta changed the base branch from main to develop February 26, 2026 16:21
@abhineet-gupta abhineet-gupta added the new-feature A new feature label Feb 27, 2026
@abhineet-gupta abhineet-gupta marked this pull request as ready for review March 1, 2026 02:01
@abhineet-gupta abhineet-gupta requested a review from misi9170 March 1, 2026 02:01
@abhineet-gupta
Copy link
Collaborator Author

abhineet-gupta commented Mar 1, 2026

The code starts the FAST.Farm simulation but errors out during controller calculation:

hycon/controllers/lookup_based_wake_steering_controller.py", line 99, in wake_steering_angles           
yaw_setpoint = (np.array(wind_directions) - yaw_offsets).tolist()                                                                                                                                                                                                                                  
ValueError: operands could not be broadcast together with shapes (3,) (2,)

I think the controller expects the wind directions in a differnt format but I am unable to run the other example lookup-based_wake_steering_florisstandin to debug this.

Need some help with this error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@misi9170 misi9170 Awaiting requested review from misi9170

Assignees

No one assigned

Labels

new-feature A new feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@abhineet-gupta @misi9170 @paulf81