-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Rotwing guidance bank fix #3271
Conversation
The setting in the GCS is still in radians.
You have |
ac1cda0
to
68faaf1
Compare
PR updated! |
Thanks @dewagter and @gautierhattenberger for looking into this, didn't know the setting could also be configured with an alt_unit. Like this it works! |
Some stuff seem to be merged with #3274 already. Can you rebase on master to check ? Or we just believe in git... |
Co-authored-by: Gautier Hattenberger <gautier.hattenberger@enac.fr>
e733c04
to
7bc65a5
Compare
Rebased on master works fine. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bypass of preflight in NPS is OK.
Setting MAX_BANK is now done the clean way.
Guidance bank bug is resolved.
@EwoudSmeur, please verify