Andrew Tridgell
Andrew Tridgell
@peterbarker I think there is a less intrusive approach which we can discuss
maybe a SURF log message? with instance number
do you want the mag, baro airspeed data etc? should we do a true IMU_DATA msg, instanced, mavlink2, just IMU?
@loki077 i'm happy for it to be logged in DroneCAN for now. We may move it if we get other protocols that can carry this information
@IamPete1 maybe we could chat during the week and discuss the developer tooling implications?
@durka thanks! minor thing, our code style requires the commit to start with the subsystem, so: AP_InternalError: fix signedness issue with snprintf I can fix your PR if you don't...
@HiRCBro I've fixed up the commits in your PR
can be merged when testing is complete
@Hwurzburg I think @andyp1per has addressed your issues except the back-side photo. I think we could merge? can you check?
@andyp1per nice work! I'd love to see some test logs