FF: align initial wake grid with the blades #3031
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This avoids a mesh mapping error we were occasionally seeing. No calculations are done with this mesh until after it has been reset, so we really don't care if it is "correct" or not.
andrew-platt
force-pushed
the
b/FF_initMeshMapError
branch
from
b558af5 to
ccf66c4
Compare
jjonkman
reviewed
Oct 10, 2025
jjonkman
approved these changes
Oct 10, 2025
Collaborator
jjonkman
left a comment
jjonkman
left a comment
There was a problem hiding this comment.
Thanks for correcting this implementation. This makes more sense to me now.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ready to merge
Feature or improvement description
We have seen occasional mesh mapping errors during initialization of FAST.Farm. This may be due to the initial position of the
m%ADRotorDiskmeshes getting set based only on the wake grid sitting on the ground. However, the wake grid should be centered on the hub as it is in later calculations. If the grid is less than the distance from the ground to the top of whatever blade is pointed vertically (blade 1 typically), a mesh mapping error was occurring. This change now aligns the wake grid meshes with the blade axis during init - not technically correct, but will avoid mesh mapping issues.Related issue, if one exists
#2027
#1613
#1044
Impacted areas of the software
FAST.Farm initialization in the AWAE module only.
Additional supporting information
This PR came about after some internal discussions about an issue with a tall model where the number of wake radii was set too low.
Test results, if applicable
No test results are affected.