-
Notifications
You must be signed in to change notification settings - Fork 557
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
Some inconsistencies in reported flightmodes #1068
Comments
Can you make a list inconsistencies? Also please report what firmware version you are using. |
I am using Arducopter v3.2_RC9 firmware. I hope that this helps a little. |
@frikzy Thanks, that's very helpful |
@frikzy Are you sure that it's supposed to be "PosHold"? The ArduCopter 3.2 beta testing forum references Hybrid mode twice in the description, but never once mentions "PosHold". |
Hybrid was the first proposal for the new mode (hybrid of "Loiter" and "AltHold"), but later with release of 3.2 beta it ws renamed to "PosHold". That's what I know about it. |
@rmackay9 Is it PosHold or Hybrid? |
PosHold. |
@frikzy okay, you're right. Re: number 6 on that list, I thought RTL stood for Return to Launch (not ' return to land'). |
Yes, that's right. It's "Return to Launch", my mistake. |
Okay, I'll try to fix this tomorrow. |
@arthurbenemann we already have a |
This is the most likely culprit: https://github.com/DroidPlanner/droidplanner/blob/master/Mavlink/src/com/MAVLink/Messages/ApmModes.java#L33 look at the second parameter of the enum. |
A feature is something you deliberately put in to a program. A bug is something you accidentally put into a program. Usually described as a "feature" by the Marketing Department. [grin] |
In Droidplanner v2.7.5 RC1 there are some inconsistencies in reported flightmodes (I belive in other releases too). PosHold is reported as "Hybrid" (also as voice) and RTH is reported as "RTL", but in voice as "Return to Home" which is correct. Other flightmodes that I tested: Stabilize, AltHold, Loiter and Drift are OK.
The text was updated successfully, but these errors were encountered: