Skip to content

Develop#699

Open
ptrbortolotti wants to merge 111 commits intomasterfrom
develop
Open

Develop#699
ptrbortolotti wants to merge 111 commits intomasterfrom
develop

Conversation

@ptrbortolotti
Copy link
Contributor

Purpose

Prep release 4.1.0

  • Integration of WOMBAT for the prediction of OpEx costs of wind farms
  • Update to windIO v2.1
  • New sizing tool for electric converter
  • Rename GitHub organization from WISDEM to NLRWindSystems

We should wait to merge this as @dzalkind tests this branch on WEIS

Type of change

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (non-backwards-compatible fix or feature)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Documentation update
  • Maintenance update
  • Other (please describe)

Testing

Checklist

  • I have run existing tests which pass locally with my changes
  • I have added new tests or examples that prove my fix is effective or that my feature works
  • I have added necessary documentation

RHammond2 and others added 23 commits February 11, 2026 09:16
Feature: WOMBAT integration for optional OpEx calculation
Converter model from Sam Seo and Bang Nguyen at NLR
Copy link
Member

@gbarter gbarter left a comment

Choose a reason for hiding this comment

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

THank you for pushing this forward!

@gbarter
Copy link
Member

gbarter commented Feb 24, 2026

@dzalkind - Good to go here?


if 'ROSCO' in modeling_options: # Will only be there if called by WEIS
if modeling_options['ROSCO']['ps_percent'] != control["pitch"]["ps_percent"]:
wt_opt["control.V_in"] = min(control["min_pitch_table"]["wind_speed"])
Copy link
Contributor

Choose a reason for hiding this comment

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

I suggest we incorporate a specific cut-in/cut-out wind speed because I made these tables in ROSCO, which uses some defined cut-in wind speed, sometimes coming from wisdem/weis, so it's a bit circular and unclear now where these should be set.

I prototyped an addition to the control geometry schema if we want it there, but I also think it could go under assembly. Thoughts? @ptrbortolotti

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