Skip to content
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

airspeed sensor driver #307

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

airspeed sensor driver #307

wants to merge 2 commits into from

Conversation

gellings
Copy link

I saw you all were working on fixed wing stuff and I know a good airspeed sensor might be useful. I've written a simple driver for the ms4525 pressure sensor that autopilots like the pixhawk used. Here is the url (http://www.meas-spec.com/product/t_product.aspx?id=11659) where you can look at the documentation for the sensor. I've only tested it with the DS5AI_011D version that comes with the pixhawk. Right now everything is raw measurements with no calibration. I think it could be added really simply to the drv_ms4525.c file but then you are dealing with floats. I'll let someone else figure that out because the raw measurement is what I want for now. I hope this is helpful!

@ghost
Copy link

ghost commented Jan 18, 2016

Can one of the admins verify this patch?

@trollcop
Copy link
Member

ok to test
On Jan 19, 2016 7:05 AM, "baseflight-jenkins" [email protected]
wrote:

Can one of the admins verify this patch?


Reply to this email directly or view it on GitHub
#307 (comment).

@ghost
Copy link

ghost commented Jan 19, 2016

Your changes violate the baseflight coding style guidelines: http://dev.baseflight.net:8080/job/Baseflight%20Pull%20Requests/ws/git_diff_358.html

@ghost
Copy link

ghost commented Jan 19, 2016

Building this PR Failed :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants