Skip to content

Conversation

@paulf81
Copy link
Collaborator

@paulf81 paulf81 commented Jan 6, 2026

This PR is meant to address #192. Based on that issue what is needed is:

Some secondary issues to address before merging

  • Any documentation updates required?

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses issue #192 by adding a new example demonstrating the WindFarmSCADAPower component, which allows simulating wind farms using SCADA power data instead of wind speed data.

Key Changes

  • New example directory 00b_wind_farm_scada_power with runscript, configuration, plotting, and documentation
  • Enhanced input generation script to produce SCADA power data alongside wind speed data
  • Updated utility functions to track the new scada_input.ftr file

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
hercules/utilities_examples.py Updated documentation and file tracking to include new scada_input.ftr file
examples/inputs/00_generate_wind_history_small.py Extended to generate turbine power data using cubic wind speed approximation
examples/00b_wind_farm_scada_power/hercules_input.yaml Configuration file for WindFarmSCADAPower example
examples/00b_wind_farm_scada_power/hercules_runscript.py Main simulation script with controller that toggles turbine 0 power setpoint
examples/00b_wind_farm_scada_power/plot_outputs.py Plotting script to visualize simulation outputs
examples/00b_wind_farm_scada_power/README.md Documentation describing the example setup and usage

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

paulf81 and others added 3 commits January 6, 2026 13:39
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant