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
CBRK_FLIGHTTERM: Does not affect takeoff failure safety detection #14972
Conversation
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.
This change looks good but I would also mention something about the takeoff failure detection exception in FD_FAIL_P and FD_FAIL_R parameters as those values are still relevant during the take-off phase regardless of the value of CBRK_FLIGHTTERM.
Thanks @RicardoM17 - have clarified those too. @bresch Can you please review the additional changes. If you're happy please also MERGE. I don't have rights to merge. |
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.
Thanks @hamishwillee !
Thanks for removing the trailing whitespace. @bresch if this is good, I can't merge. |
Improve description. @RicardoM17 - better?