ED: update blade file parser to allow arbitrary column locations#3253
Merged
andrew-platt merged 3 commits intoOpenFAST:devfrom Mar 11, 2026
Merged
ED: update blade file parser to allow arbitrary column locations#3253andrew-platt merged 3 commits intoOpenFAST:devfrom
andrew-platt merged 3 commits intoOpenFAST:devfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates ElastoDyn’s blade input file parsing so the distributed blade properties table can contain additional columns (e.g., PitchAxis) and arbitrary column ordering, by selecting the required 5 columns based on header names and ignoring the rest.
Changes:
- Add header parsing (
GetBldTbleCols) to locate required table columns by name. - Update table reading to parse an arbitrary number of numeric columns per row and map required fields via the discovered column indices.
- Remove the previous hard failure when extra columns (like
PitchAxis) are present.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
deslaughter
approved these changes
Mar 11, 2026
luwang00
approved these changes
Mar 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ready to merge
Feature or improvement description
Some of the IEA turbine models include the
PitchAxiscolumn as well as additional columns that ElastoDyn does not use. Simply deleting thePitchAxiscolumn from the table is insufficient as the parsing will fail due to more than 5 colunns still remaining. This PR generalizes the reading of the blade properties table to select the 5 columns needed based on the names in the header, and ignore any others. Columns also don't need to be in order anymore.Related issue, if one exists
None. Reported internally.
Impacted areas of the software
ElastoDyn blade file parsing only.
Generative AI usage
None
Test results, if applicable
No test results change