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
AP_Baro: New AP_Baro_Keller device driver #6787
Conversation
great stuff. |
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.
I don't like the semaphore waiting forever on"flying" code, but it has been done in other places. Otherwise, looks good to me.
I've marked @tridge for review as he is more used to drivers.
_sem->give(); | ||
|
||
uint16_t raw_pressure_avg = ((float)sum_pressure) / num_samples; | ||
uint16_t raw_temperature_avg = ((float)sum_temperature) / num_samples; |
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.
Why the float cast? Those are floats already
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.
whoops, dev remnants. Thanks.
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.
this looks good to me, thanks!
This is a line of pressure transducers that can measure up to 200 bar (2000 meters water depth). The current MS5837 device is rated to only 30 bar. Let's go deeper with ArduSub!