Skip to content

Conversation

@deslaughter
Copy link
Collaborator

This PR is ready to merge

Feature or improvement description

This PR adds an error message if the user attempts to use BeamDyn with AeroMap which is not currently possible. Support for this combination of features will be added in the OpenFAST 5.x series.

Impacted areas of the software

AeroMap (FAST_SS_Subs.f90)

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds validation to prevent using BeamDyn blades with AeroMap functionality, which is not currently supported. The change informs users that this feature combination will be available in a future OpenFAST release.

  • Adds error check after module initialization to detect BeamDyn usage with AeroMap
  • Returns a fatal error with a descriptive message when the incompatible configuration is detected

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@andrew-platt
Copy link
Collaborator

The error with the py_seastate_1 is addressed in #3065, which will be merged soon. So we can merge this despite that failing case.

@deslaughter deslaughter merged commit f90d379 into OpenFAST:dev Dec 23, 2025
11 of 12 checks passed
@deslaughter deslaughter deleted the aeromap_bd_err branch December 23, 2025 01:23
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