Skip to content

Conversation

@pcarruscag
Copy link
Member

Related Work

#1298
#1301
#1171 <- remove dead code

Copy link
Contributor

@TobiKattmann TobiKattmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this fix and chore 💐 So take my approval, although I would like to have the original discussion author @nikenuaa report that it fixes his problem.

Vel_Infty[2] = sin(AoA*PI_NUMBER/180.0)*cos(Beta*PI_NUMBER/180.0)*Vel_Infty_Mag;
else {
Vel_Infty[0] = cos(AoA)*cos(AoS)*Vel_Infty_Mag;
Vel_Infty[1] = sin(AoS)*Vel_Infty_Mag;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Service comment: This line is the bugfix of discussion #1298 and subsequent issue #1301, the remainder of this commit is just nice clean-up 🧹 (please correct me if i am wrong)

Co-authored-by: TobiKattmann <31306376+TobiKattmann@users.noreply.github.com>
@pcarruscag pcarruscag linked an issue Jun 13, 2021 that may be closed by this pull request
Co-authored-by: TobiKattmann <31306376+TobiKattmann@users.noreply.github.com>
@pcarruscag pcarruscag merged commit 295485c into develop Jun 15, 2021
@pcarruscag pcarruscag deleted the fix_fixed_cl branch June 15, 2021 13:00
@TobiKattmann
Copy link
Contributor

TobiKattmann commented Jun 15, 2021

So take my approval, although I would like to have the original discussion author @nikenuaa report that it fixes his problem.

chronicler duty: that just happened, see #1298

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.

Fixed CL mode does not work if SIDESLIP != 0

3 participants