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

Use pointers to couple InflowWind and FAST.Farm #1624

Merged
merged 1 commit into from
Jun 14, 2023

Conversation

deslaughter
Copy link
Collaborator

This pull request is ready to merge.

Feature or improvement description

This commit is the initial attempt to use pointers to couple the wind velocities generated by AWAE in FAST.Farm to the InflowWind FlowField which is used by AeroDyn to calculate the wind velocities. The Vdist_High data in AWAE has been changed to a pointer and FlowField%Grid4D%Vel is associated with this member to alleviate the need to copy the data between the modules.

Impacted areas of the software

  • FAST.Farm
  • AWAE
  • InflowWind
  • Openfast Library

Many of these areas were impacted because pointers were added to InitInput structures to make use of standard initialization functions. It would have been possible to skip these changes and directly associate the pointers in FAST.Farm, but I thought doing it this way would be clearer.

Test results, if applicable
FAST.Farm test results are unchanged though the regression tests run slightly faster.

This commit is the initial attempt to use pointers to couple the wind
velocities generated by AWAE in FAST.Farm to the InflowWind
FlowField which is used by AeroDyn to calculate the wind velocities.
The Vdist_High data in AWAE has been changed to a pointer and
FlowField%Grid4D%Vel is associated with this data to alleviate the
need to copy the data between the modules.
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! I'll run a couple of cases I have for WAT and let you know if I find anything.

@deslaughter deslaughter merged commit b5ea7e9 into OpenFAST:dev-unstable-pointers Jun 14, 2023
@deslaughter deslaughter deleted the f/FFptr branch June 14, 2023 19:50
@andrew-platt andrew-platt mentioned this pull request Dec 24, 2024
38 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