Skip to content

MHK inflow calls#3306

Draft
hkross wants to merge 6 commits intoOpenFAST:rc-5.0.1from
hkross:bug/MHK_Inflow_Calls
Draft

MHK inflow calls#3306
hkross wants to merge 6 commits intoOpenFAST:rc-5.0.1from
hkross:bug/MHK_Inflow_Calls

Conversation

@hkross
Copy link
Copy Markdown
Contributor

@hkross hkross commented Apr 13, 2026

This pull request is NOT ready to be merged

Feature or improvement description

  • Addresses minor bugs in calls to MHK inflow (wave+current)
  • Includes a few documentation updates

Impacted areas of the software

  • AeroDyn
  • Documentation

Test results

  • Likely some r-test impacts, still assessing this
  • r-test branch merging required

@andrew-platt andrew-platt self-assigned this Apr 13, 2026
@andrew-platt andrew-platt requested a review from luwang00 April 13, 2026 19:23
@andrew-platt andrew-platt added Module: AeroDyn Feature: MHK Marine Hydro-Kinetic Module: SeaState SeaState module for wave data generation labels Apr 13, 2026
@andrew-platt andrew-platt added this to the v5.0.1 milestone Apr 13, 2026
@hkross hkross marked this pull request as draft April 13, 2026 19:37
if (dvr%MHK /= MHK_None .and. dvr%MHK /= MHK_FixedBottom .and. dvr%MHK /= MHK_Floating) call SetErrStat(ErrID_Fatal, 'MHK switch must be 0, 1, or 2.', ErrStat, ErrMsg, RoutineName)

if (dvr%MHK /= MHK_None .and. dvr%SS_InitInp%CompSeaSt == 1 .and. dvr%IW_InitInp%CompInflow /= 1) call SetErrStat( ErrID_Fatal, 'InflowWind must be activated for MHK turbines when SeaState is used.', ErrStat, ErrMsg, RoutineName )
if (dvr%MHK /= MHK_None .and. dvr%SS_InitInp%CompSeaSt == 1 .and. dvr%IW_InitInp%CompInflow == 0 .and. dvr%IW_InitInp%HWindSpeed > 0) call SetErrStat( ErrID_Fatal, 'Steady Wind option cannot be used for MHK turbines with SeaState.', ErrStat, ErrMsg, RoutineName )
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should we clarify this is the steady wind option in the AeroDyn driver (as opposed to in InflowWind) if that's what you meant?

@hkross hkross closed this Apr 13, 2026
@hkross hkross force-pushed the bug/MHK_Inflow_Calls branch from 2f4ba2d to 2bb3b37 Compare April 13, 2026 21:59
@hkross hkross reopened this Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature: MHK Marine Hydro-Kinetic Module: AeroDyn Module: SeaState SeaState module for wave data generation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants