NREL / hercules

Hercules

https://nrel.github.io/hercules/

Apache License 2.0

3 stars 7 forks source link

[BUGFIX] Reenable AMR-Wind by setting up empty power_setpoints #85

Closed misi9170 closed 6 months ago

misi9170 commented 6 months ago

76 Enabled passing power_setpoints to the `FlorisStandin`; however, @jfrederik-nrel noticed that this created a bug when running with AMR-Wind, because the helics connection in AMR-Wind was not expecting extra elements in the `"control"` message being passed to it.

This PR fixes that bug by handling variable length control signals (without power_setpoints for AMR-Wind; and (optionally) with power_setpoints for the FlorisStandin).

misi9170 commented 6 months ago

Confirmed that it handles turbine power from AMR-Wind properly now

Thanks!

NREL / hercules

[BUGFIX] Reenable AMR-Wind by setting up empty power_setpoints #85

76 Enabled passing power_setpoints to the FlorisStandin; however, @jfrederik-nrel noticed that this created a bug when running with AMR-Wind, because the helics connection in AMR-Wind was not expecting extra elements in the "control" message being passed to it.

76 Enabled passing power_setpoints to the `FlorisStandin`; however, @jfrederik-nrel noticed that this created a bug when running with AMR-Wind, because the helics connection in AMR-Wind was not expecting extra elements in the `"control"` message being passed to it.