-
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
ins_ekf2 is incompatible with fixedwing firmware #2865
Comments
The waypoints are only included for the The second part of rotorcraft guidance functions is there when the EKF2 decides to reset it's heading. You can disable it with |
The structure for the waypoints of fixedwing and rotorcraft is completely different and not not compatible (and of course, not easy to merge, that would be too simple). Even the "nav_common" is not common at all. |
Currently including the module
ins_ekf2
will only compile for arotorcraft
airframe, due to a couple of conflicts:Waypoints
I'm not 100% that
nav.h
is included for all fixedwing airframes, but when it is it will also includenav_common.h
, which has a number of conflicting defines withwaypoints.h
paparazzi/sw/airborne/modules/nav/waypoints.h
Lines 42 to 47 in e82db25
paparazzi/sw/airborne/modules/nav/waypoints.h
Line 51 in e82db25
paparazzi/sw/airborne/modules/nav/common_nav.h
Lines 39 to 43 in e82db25
paparazzi/sw/airborne/modules/nav/common_nav.h
Line 57 in e82db25
The definition of the variable
waypoints
as well as a number of macros that perform operations on waypoints are both double and use a different structure. I think it would make sense to move all waypoints and functions on waypoints towaypoints.h
and include this file incommon_nav.h
.Rotocraft guidance functions
Certain variables that are used in ekf2 seem to only be defined in rotorcraft stabilization and guidance files,
stab_att_sp
,nav_heading
,guidance_h_read_rc
,stabilization_attitude_enter
The text was updated successfully, but these errors were encountered: