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

Linearization with AD15 and IfW #2014

Conversation

andrew-platt
Copy link
Collaborator

@andrew-platt andrew-platt commented Jan 29, 2024

This is ready for merging.

Feature or improvement description
When InflowWind was converted to use pointers, significant changes were required to AeroDyn to support this with linearization (InflowWind outputs are no longer inputs to AeroDyn). This is the second of 3 PR's to address this.

Related issue, if one exists
PR #1882 is the first PR in the sequence.

Impacted areas of the software

Linearization in the following modules is affected:

  • AeroDyn
  • InflowWind
  • glue code

Additional supporting information

Test results, if applicable

  • The following regression tests have changed:
  • 5MW_Land_BD_Linear_Aero - same results, different matrix size. Extended outputs from IfW to AD15
  • 5MW_Land_BD_Linear - slight change in results
  • 5MW_Land_Linear_Aero - same results, different matrix size. Extended outputs from IfW to AD15
  • 5MW_OC3Spar_Linear - same results, different matrix size. Extended outputs from SS to HD
  • 5MW_OC4Semi_Linear - same results, different matrix size. Extended outputs from SS to HD
  • 5MW_OC4Semi_MD_Linear - same results, different matrix size. Extended outputs from SS to HD
  • Fake5MW_AeroLin_B1_UA4_DBEMT3 - same results as dev-unstable-pointers branch, different matrix size. Extended outputs from IfW to AD15.
  • Fake5MW_AeroLin_B3_UA6 - same results as dev-unstable-pointers branch, different matrix size. Extended outputs from IfW to AD15.
  • Ideal_Beam_Fixed_Free_Linear - same results, different matrix size
  • Ideal_Beam_Free_Free_Linear - same results, different matrix size

Note: This branch will be rebased soon to resolve conflicts and simplify comparisons.

Swap ordering from
   - AD14
   - AD15
   - ExtLoads
   - InflowWind
   - SuperController
   - SeaState
to
   - AD14
   - InflowWind
   - SeaState
   - AD15
   - ExtLoads
   - SuperController
Also modify call order in the ADI library
Not sure aero settings are correct, but should give a reasonable estimate if things are working right or not.  So far not.
- original results generated by 3.5.2
…AD15_IfW_Linearization

Some regression tests need updating
added -j1 flag so that this test always runs sequential.  Otherwise the 5MW_OC3Mnpl_DLL_WTurb_WavesIrr_Restart may kick off before 5MW_OC3Mnpl_DLL_WTurb_WavesIrr has any input files generated
This was removed at some point during the linearization work, which broke all AD14 cases.
Added values from IfW/FlowField: HWindSpeed/PLexp/PropagationDir
@andrew-platt andrew-platt force-pushed the f/AD15_IfW_Linearization branch 2 times, most recently from d0c2301 to 6c757fc Compare April 29, 2024 22:19
@andrew-platt
Copy link
Collaborator Author

See andrew-platt#23 for a temporary PR for comparing what changes in this PR relative to #1882.

@andrew-platt andrew-platt marked this pull request as ready for review April 30, 2024 21:41
@andrew-platt andrew-platt merged commit 33ec68b into OpenFAST:dev-unstable-pointers May 9, 2024
21 checks passed
@andrew-platt andrew-platt deleted the f/AD15_IfW_Linearization branch June 6, 2024 16:57
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.

3 participants