Skip to content

Fully disable 2nd tower mode shapes if DOF is disabled#466

Merged
dzalkind merged 1 commit intoNLRWindSystems:developfrom
dzalkind:2nd_tower_mode
Apr 6, 2026
Merged

Fully disable 2nd tower mode shapes if DOF is disabled#466
dzalkind merged 1 commit intoNLRWindSystems:developfrom
dzalkind:2nd_tower_mode

Conversation

@dzalkind
Copy link
Copy Markdown
Collaborator

@dzalkind dzalkind commented Mar 27, 2026

Purpose

In cases where the 2nd tower mode is not critical, it's helpful to just disable it. OpenFAST still checks the mode shapes, and if they are ill-conditioned (usually very large, and still need to sum to 1), an error ensues. This update fills those mode shapes with a placeholder input when the 2nd tower DOF is disabled.

Type of change

What types of change is it?
Select the appropriate type(s) that describe this PR

  • 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

Explain the steps needed to test the new code to verify that it does indeed address the issue and produce the expected behavior.

Checklist

Put an x in the boxes that apply.

  • 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

@dzalkind dzalkind requested a review from ptrbortolotti April 6, 2026 16:42
@dzalkind dzalkind merged commit 84ea9a2 into NLRWindSystems:develop Apr 6, 2026
9 checks passed
@dzalkind dzalkind deleted the 2nd_tower_mode branch April 6, 2026 19:43
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