Skip to content

Conversation

@lejeunemax
Copy link

This PR introduce FLORIS support in WIFA:

WindIO is not yet supported by FLOIS main branch.
Currently refer to lejeunemax/floris/windIO.

Some features may not be fully supported, these include:

  • Subsetting of wind conditions (times_run, wind_speeds_run, directions_run)
  • Only power, thrust_coefficient, axial_induction, and turbulence_intensity
    are currently supported as turbine output variables.
  • Some advanced windIO properties may not be fully supported. These include:
    blockage_model, rotor_averaging, and axial_induction_model.

Warnings will be raised if some of these inputs are encountered.

def test_floris_multiple_turbines():
"""Test FLORIS with multiple turbine types"""
wes_dir = test_path / "../examples/cases/windio_4turbines_multipleTurbines/wind_energy_system/"
_run_floris(wes_dir)
Copy link
Member

Choose a reason for hiding this comment

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

It would be ideal to manually run floris to confirm that each test creates the expected output, like we do in pywake tests

Copy link
Author

Choose a reason for hiding this comment

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

This has been addressed by the latest commit.

@kilojoules
Copy link
Member

FYI pre-commit is failing

@lejeunemax
Copy link
Author

Change log:

  • Ensure consistent definition of the DTU10MW across files for smoother testing;
  • Making all paths relative;
  • Comparing WIFA results to these obtained running FLORIS directly.

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.

2 participants