-
Notifications
You must be signed in to change notification settings - Fork 25
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Bugfix to account for rotor yaw control in solve statics
The rotor orientation is set internally depending on yaw_mode, but this was done only after computing the inertial properties of the system in solveStatics(). Consequently, the weight of the RNA was acting as if the turbine was at its initial position regardless of yaw_mode. This commit fixes that, and now rotor orientation is updated before computing the inertial properties of the system. I'm still not sure if this fix is 100% correct, so this modification needs to be revised before merging rc1.3.2 into main. The affected tests are the ones with wind in test_model.py. The corresponding true values were updated.
- Loading branch information
1 parent
c3e9df7
commit b5d2591
Showing
5 changed files
with
45 additions
and
54 deletions.
There are no files selected for viewing
This file contains 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 file contains 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
Binary file not shown.
Binary file not shown.
This file contains 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