Skip to content

Conversation

@paulf81
Copy link
Collaborator

@paulf81 paulf81 commented Mar 12, 2024

Add ti-mixing to EMG

This may not be an actual pull request, but may be convenient if it yields some changes. The purpose of this is to look at making a version of EMG with atmospheric_ti_gain re-enabled. To this end I started with a small pseudo-example that compares the power output of columns of a wind farm from the current default EMG to one which uses atmospheric_ti_gain.

I had in mind a step by step criteria, ideally:

  • The row averaged power ratios will be similar when 6% TI is used
  • The row averaged power ratios will also be similar when IEC-reference TI per wind speed (provided using the Iref function in WindRose), or not, this might need something thinking
  • The wakes will be deeper when TI < 6%
  • The wakes will be shallower when TI > 6%

Note this is just a start but thought it a convenient way to collaborate. Note also this is very messy until the namespace changes go into v4. The only file worth checking is: examples/xx_add_ti_to_emg.py

Tagging: @misi9170 @jfrederik-nrel

rafmudaf and others added 30 commits March 5, 2024 11:06
# Conflicts:
#	floris/core/core.py
# Conflicts:
#	examples/01_opening_floris_computing_power.py
#	examples/02_visualizations.py
#	examples/03_making_adjustments.py
#	examples/04_sweep_wind_directions.py
#	examples/32_plot_velocity_deficit_profiles.py
#	floris/floris_model.py
#	floris/flow_visualization.py
#	floris/layout_functions.py
#	floris/layout_visualization.py
#	floris/tools/__init__.py
#	floris/tools/visualization.py
#	floris/uncertainty_interface.py
#	floris/visualization.py
#	tests/floris_model_integration_test.py
@paulf81 paulf81 self-assigned this Mar 12, 2024
@paulf81 paulf81 mentioned this pull request Mar 13, 2024
4 tasks
@paulf81
Copy link
Collaborator Author

paulf81 commented Mar 13, 2024

Closing this one in favor of #841

@paulf81 paulf81 closed this Mar 13, 2024
@paulf81 paulf81 deleted the add_ti_to_emg branch March 13, 2024 15:35
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