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

AD: adding linearization tests for aerodyn continuous-states model #1695

Merged
merged 12 commits into from
Jul 19, 2023

Conversation

ebranlard
Copy link
Contributor

@ebranlard ebranlard commented Jul 19, 2023

This pull request ready to be merged

Feature or improvement description
Adds two simple test cases for the continuous-states aerodynamic models DBEMT=3, UAMod=4 and UAMod=6. The test also look at the UserProp linearization. These tests are introduced because these aerodyn linearization features were not previously tested. (Time domain tests are present in the aerodyn_driver).

The tests are introduced in preparation for a couple of AeroDyn updates to UA, DBEMT, and the AeroCompMaps in AeroDyn.

See readmes:

Impacted areas of the software
r-test only, but this is intended towards AeroDyn linearization testing.

Test results, if applicable

The tests are labeled Fake5MW as they only use 4 blade stations, 2 polars, and the last polar is subplemented with an extra table to test the UserProp interpolation. It was verified that the UA and DBEMT models have a decent impact (see figure below for different UA and DBEMT) on the axial induction, and lift, compared to simulations with steady parameters (UA0 DBEMT0). The Jacobian with respect to UserProp are none zero for the 3 sections that use the polar.

image

image

Checklist

@ebranlard ebranlard merged commit 202bfe7 into OpenFAST:rc-3.5.1 Jul 19, 2023
@ebranlard ebranlard deleted the f/adlin-tests branch July 19, 2023 20:20
@andrew-platt andrew-platt mentioned this pull request Oct 19, 2023
19 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