Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

waves_example notebook was broken with dev code #16

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

nRiccobo
Copy link

There were a few errors that needed to be fixed to run the waves_example notebook.

dmulas and others added 8 commits December 2, 2024 16:33
* energy_loss update

* fix line lengths, typos, typing, and add  input.

* environmental_loss_ratio, detailed wake_losses, eliminate with_losses

* add missing docstring and run pre-commit

* fix error message

* fix verbose bool check

* fix typo in cable naming

* update for ORBIT v1.1 compatibility

* fix missing turbine potential bug in timeseries

* apply intended changes and bug fixes to wake_losses

* fix remaining floris energy potential/production method issues

* update error message

* Addressed feedback from RHammond2: energy_losses = potential * total_loss_ratio and descriptive note for total_loss_ratio method

* udpate pre-commit configuration

* shorten argument and run pre-commit

* remove redundancies and fix small calculation issues

* fix small computation and aggregation bugs

* update wake losses for potential

* update technical losses for unavailable land option

* update changelog

* add input validation and common floris energy capture method

* update remaining metrics with validator decorator

* update changelog

---------

Co-authored-by: RHammond2 <[email protected]>
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.

4 participants