-
Notifications
You must be signed in to change notification settings - Fork 13.2k
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
Depth and ms8537 #20089
base: main
Are you sure you want to change the base?
Depth and ms8537 #20089
Conversation
…UM to differentiate between air, fresh water and salt water in depth calculation; closes PX4#20073
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.
Great!
Would be cool if we can propagate this to the simulated ROV setups:
- SDF file: https://github.com/PX4/PX4-SITL_gazebo/blob/bd941689b6e9d3e628a1e786d1d042d48005dc7b/models/uuv_bluerov2_heavy/uuv_bluerov2_heavy.sdf#L539-L543
- pressure calculations: https://github.com/PX4/PX4-SITL_gazebo/blob/bd941689b6e9d3e628a1e786d1d042d48005dc7b/src/gazebo_barometer_plugin.cpp#L148-L157
changed param name BARO_MEDIUM to SENS_BARO_MEDIUM; added value comments to param SENS_BARO_MEDIUM;
I don't particularly like that we're doing this within "air data", but it seems relative inoffensive so far. |
Thanks for the contribution! However the CIs are failing indicating that there's a compile error. Could you just check on that & rebase to |
@spiderkeys had some interesting thoughts on #20073 so you might not want to merge this pull request entirely, but just the sensor bug fixes. I don't work at the same company (for UUVs) anymore so I can't spend time on this at the moment. |
Describe problem solved by this pull request
fixes #20049
fixes #20051
fixes #20073
Test
Tested on a pixhawk4 with a MS5837-30BA in a 1.5m deep fresh water pool.
Accuracy ±1cm