ardupilot icon indicating copy to clipboard operation
ardupilot copied to clipboard

Plane: Logged rangefinder state

Open Georacer opened this issue 1 month ago • 3 comments

I've recently tried to debug some landing logs and I've always stumbled guessing the rangefinder state information.

This PR adds logging of the whole structure.

Georacer avatar Nov 24 '25 22:11 Georacer

@peterbarker I believe I need to add a case for #if AP_RANGEFINDER_ENABLED in https://github.com/georacer/ardupilot/blob/pr/log_rangefinder_state/Tools/autotest/vehicle_test_suite.py#L2746-L2746, for the failing autotest to pass, correct?

Georacer avatar Nov 27 '25 11:11 Georacer

@tridge the boolean fields are now separate, not a bitfield. Is this what you were looking for, or you meant for them to be one consolidated flag field?

Georacer avatar Dec 04 '25 13:12 Georacer

@IamPete1 ok to approve?

tridge avatar Dec 10 '25 08:12 tridge