ardupilot icon indicating copy to clipboard operation
ardupilot copied to clipboard

AP_Airspeed: ND210 pressure sensor, airspeed library

Open zzaurak opened this issue 10 months ago • 4 comments

I've rebased this pull request https://github.com/ArduPilot/ardupilot/pull/23300, did some fixes and testet it with real hardware. It seems to work well on the bench, I will do a few real word flight test & comparisons next week and report as well.

zzaurak avatar Apr 21 '24 09:04 zzaurak

Hi @zzaurak,

Thanks for this. Just a small admin thing, it would be good to change the commit title (not the PR title but the individual commit titles) to start with "AP_Airspeed: ".

rmackay9 avatar Apr 22 '24 00:04 rmackay9

Hi @rmackay9 thanks for the feedback, I changed the commit titles :)

zzaurak avatar Apr 22 '24 06:04 zzaurak

As promised I did a real-world comparison test flight with the ND210 airspeed sensor and this PR. The top tube is the ND210, the bottom one is the Matek ASPD-DLVR-CAN. Setup: Screenshot 2024-04-30 at 20 12 39

I've attached the log 00000028-anon.log.zip. In the flight before this logfile, airspeed autocal was on for the second airspeed sensor [1].

ASPD[0] is the DLVR ASPD[1] is the ND210

I would say pretty much comparable, but especially in the lower speed ranges and while in QLoiter and FBWA it looks a bit smoother. Screenshot 2024-04-30 at 20 16 15 Screenshot 2024-04-30 at 20 17 24 Screenshot 2024-04-30 at 20 17 51

zzaurak avatar Apr 30 '24 18:04 zzaurak

@rmackay9 anything else that would be blocking a merge?

zzaurak avatar May 03 '24 12:05 zzaurak