Andrew Tridgell
Andrew Tridgell
@ZeroOne-Aero there are a few build errors (see CI logs with clang for example) Can you supply a flight log, with this IMU and another IMU, with LOG_DISARMED=1 and LOG_REPLAY=1...
@ZeroOne-Aero also needs to pass CI of course
> This is the flight log of our X6 as the flight controller. thanks, the log looks reasonable to me
@ZeroOne-Aero for future flight logs can you set EK3_IMU_MASK so we get EKF data for the 3rd IMU? Also please set LOG_DISARMED=1 and LOG_REPLAY=1 What we're looking for is lower...
looks nice!
@ZeroOne-Aero I've fixed the commit msg to have "Tools: " prefix, and marked this PR for merge once CI passes
@IamPete1 ok to approve?
@rmackay9 needs your review as well I think looks good for plane
> No functional change yet. Hopefully this cleanup is appreciated. I'm a bit sceptical of the benefit of this change. Can you point at some examples of the ultimate benefit?...
@robustini as I discussed with you privately, the performance justification for the binary mode is misplaced. A laptop can very easily run at 6k FPS with text JSON while not...