Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Floris wrapper clean-up
Previously when using floris to simulate the wind farm, it would write the wind resource data to speed_dir_data.csv. This file is never read or used anywhere else in the code - so this has been removed. All other relevant information for this PR is found in the below section.
Note: this also includes a patch fix for building readthedocs (see more information here)
PR Checklist
RELEASE.md
has been updated to describe the changes made in this PRdocs/
files are up-to-date, or added when necessaryRelated issues
Impacted areas of the software
hopp/simulation/technologies/wind/floris.py
value()
: fixed theset_value
functionality to work as documentedexecute()
: cleaned up how operational losses are applied and to only print 'Simulating wind farm output in FLORIS...' ifverbose
is True (verbose
is a new attribute which defaults to True)parse_resource_data
: removed part that exports wind resource data to speed_dir_data.csv (this was unused and does not impact the functionality at all)Additional supporting information
Test results, if applicable