Skip to content

Comments

generate BD files if fst_vt dictionary BD keys are not empty#2726

Closed
ptrbortolotti wants to merge 60 commits intoOpenFAST:rc-4.0.3from
ptrbortolotti:write_beamdyn
Closed

generate BD files if fst_vt dictionary BD keys are not empty#2726
ptrbortolotti wants to merge 60 commits intoOpenFAST:rc-4.0.3from
ptrbortolotti:write_beamdyn

Conversation

@ptrbortolotti
Copy link
Contributor

In openfast_io, write BeamDyn files if data structure is available in the fast_vt dictionary (and not only if CompElast=2)
@mayankchetan

@mayankchetan mayankchetan self-requested a review April 2, 2025 21:31
@mayankchetan mayankchetan added the Type: IO Readers/Writers Input file readers and writers label Apr 2, 2025
@mayankchetan mayankchetan added this to the v4.0.3 milestone Apr 2, 2025
@mayankchetan
Copy link
Contributor

Thanks Pietro,

We might need extend the ability to write the input files even if the modules are not enabled in the .fst file across to all modules. But, this looks good for v4.0.3.

Tests pass:

1/1 Test #156: openfast_io_library ..............   Passed  417.68 sec

100% tests passed, 0 tests failed out of 1

Label Time Summary:
openfast_io    = 417.68 sec*proc (1 test)
python         = 417.68 sec*proc (1 test)

Total Test time (real) = 417.71 sec

@andrew-platt andrew-platt changed the base branch from dev to rc-4.0.3 April 2, 2025 22:01
deslaughter and others added 26 commits April 2, 2025 16:05
The argument RR0mEta to Calc_RR0mEta_rho was handled inconsistently.  See issue OpenFAST#2641 for details
Co-authored-by: Derek Slaughter <deslaughter@gmail.com>
When running from Simulink, Matlab would end (instead of the DLL). This PR fixes the problem.
@mayankchetan
Copy link
Contributor

I messed up the re-base, Will redo the PR to rc4.0.3.

More things to learn :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: IO Readers/Writers Input file readers and writers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants