-
Notifications
You must be signed in to change notification settings - Fork 18k
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
AUAV Airspeed Sensor Driver #27802
base: master
Are you sure you want to change the base?
AUAV Airspeed Sensor Driver #27802
Conversation
These actually provide absolute pressure measurement too. |
Nice! Although I'm going to leave that to someone else to implement if they want :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needs extract_features.py also
Ping @Tdogb - also needs to pass CI :-) |
Hey guys, I think it would be super cool to implement the absolute pressure sensing capability. Does anyone have strong opinions on how this is implemented? I don't want someone to unknowingly have it override their baro |
Maybe do that as a second stage? I'll mark this for DevCallEU so we think about your question anyway. We do have some precendents for detecting something as two different sensor types. |
just as an airspeed sensor initially is the way to go |
@Tdogb did you need assistance on this one, or still plugging away at it? |
@peterbarker i tried this pr. Was able to get differential pressure output. But temperature didn't work for me. |
Temperature is compensated for but the real temperature value isn't read because there isn't documentation on it. I had accidentally pushed some other dev code I have to this branch so I removed it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are still changes requested from my previous review
@MallikarjunSE @Tdogb I've force pushed some fixes:
I've bench tested buy not flight tested. Please test! |
Hey @tridge |
@tridge tested on L30D. Works! |
Hey @tridge |
Drivers for the Amphenol AUAV airspeed sensor. This is a relatively new UAV specific airspeed sensor. I don't know why Amphenol didn't write their own Ardupilot drivers, but I did it for them since this looks like a really awesome sensor.
https://www.mouser.com/new/amphenol/amphenol-auav-air-speed-altitude-sensors/