Possible minor error in the code #166
Replies: 1 comment
-
Maybe a bug. This seems to be the old waypoints implementation (only supported on some DJI drones). IMO, this old implementation is too tight dependent and limited by the DJI waypoints structure definition and should be merged in the future into the MissionManager in order to support the complete MAVLink waypoint definitions. The big difference is that the old code stores the waypoint missions using DJI format (a compatible limited subset of the MAVLink mission information) instead of using MAVLink format. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
Hello,
I've already asked some question about this project as I'm working on something similar to this, anyway I was taking a look at the code when I found this in the MAVLinkReceiver.java file, basically at line 738 (but also some line before) param2 is intended as the yaw angle, while on the MAVLink wiki I found that param2 should be the accept radius, while param4 rapresent the yaw angle, am I correct or am I missing something?
Thanks in advance for your help!
Beta Was this translation helpful? Give feedback.
All reactions