Skip to content
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

Improve Ardupilot AC_PrecLand logging #101

Open
fnoop opened this issue Aug 6, 2018 · 5 comments
Open

Improve Ardupilot AC_PrecLand logging #101

fnoop opened this issue Aug 6, 2018 · 5 comments
Milestone

Comments

@fnoop
Copy link
Member

fnoop commented Aug 6, 2018

Improve AC_PrecLand dataflash logging to include new LANDING_TARGET fields, and better data.

@fnoop
Copy link
Member Author

fnoop commented Aug 6, 2018

Should add: uint64_t sensor_time_us;, but depends on #9020, so add after that PR is merged.

@fnoop
Copy link
Member Author

fnoop commented Aug 6, 2018

Currently, log fields:

        pos_x           : target_pos_rel.x,
        pos_y           : target_pos_rel.y,
        vel_x           : target_vel_rel.x,
        vel_y           : target_vel_rel.y

data is obtained from _target_pos_rel_out_NE and _target_pos_rel_out_NE which are the final calculated target position after estimation, offsets and predictions, in body frame.
Would be helpful for debugging where the original sensor data is not immediately available (ie. IRLock) to also log _target_pos_rel_meas_NED which is the target vector before estimation.

@fnoop
Copy link
Member Author

fnoop commented Aug 6, 2018

Distance and alt should also be logged but again that's dependent on another PR #9125.

@fnoop
Copy link
Member Author

fnoop commented Aug 6, 2018

PR raised: ArduPilot/ardupilot#9130

@fnoop fnoop added the ArduPilot label Aug 6, 2018
@fnoop fnoop added this to the 1.0 milestone Aug 6, 2018
@fnoop
Copy link
Member Author

fnoop commented Aug 22, 2018

PR #9130 closed, need to add target_type and target_num to logging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant