Add min and max airspeed alarms #7467
Merged
Conversation
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
|
Why closed? Seems to be quite useful |
igor-eremyashev
force-pushed
the
airspeed-alarms
branch
from
6d08fb6 to
381a3d7
Compare
Sorry about that, noticed I used wrong github account to commit changes - fixed it |
|
@igor-eremyashev if you still need a tester, I am ready to help. |
|
@Kernelscr1 That would be awesome! I didn't get a chance to try it myself yet. |
|
@igor-eremyashev I need help to compile this branch. How can we solve this problem? Can we chat in telegram? |
|
@Kernelscr1 Sure, how can I find you on telegram? |
|
@igor-eremyashev acetheranger |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Maintaining and keeping an eye on airspeed is crucial for fixed wings, so low and high airspeed alarms would be very helpful. For now I'm still waiting for airspeed sensor to test it, so it would be great if anyone could try it out.
Related issue: #5923