Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BeamDyn: output summary file in yaml format #1335

Merged
merged 5 commits into from
Nov 30, 2022

Conversation

ebranlard
Copy link
Contributor

@ebranlard ebranlard commented Nov 19, 2022

This pull request is ready to be merged
Builds on top of #1333

Feature or improvement description
Outputs BeamDyn summary file in YAML format so that it can be human and machine readable.

Also fixes #1336, and adds a comment in the summary file related to #1337

Impacted areas of the software
BeamDyn

Test results, if applicable
Results are unchanged, but summary files are. I suggest updating them as .

Additional resources
A sample summary file is given below
solid_beam_BeamDyn.dat.sum.txt

@ebranlard ebranlard added this to the v3.4.0 milestone Nov 19, 2022
@ebranlard ebranlard self-assigned this Nov 19, 2022
@ebranlard ebranlard changed the title F/bd yaml sum BeamDyn: output summary file in yaml format Nov 19, 2022
Copy link
Collaborator

@andrew-platt andrew-platt left a comment

Choose a reason for hiding this comment

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

Looks good to me! We'll want to rerun baselines the BD cases that have summary files.

@ebranlard ebranlard merged commit ae73af4 into OpenFAST:dev Nov 30, 2022
@ebranlard ebranlard deleted the f/bd-yaml-sum branch November 30, 2022 02:30
@rafmudaf rafmudaf mentioned this pull request Dec 19, 2022
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants