bluerobotics / bluerobotics/BlueRobotics_MS5837_Library
Allow setting the ground level atmospheric pressure
- Dominant language
- C++
- Stars
- 74
- Forks
- 70
- PR merge metrics
- No merged PRs in 30d
Description
We currently offset by mean sea level pressure when estimating depth or altitude, but most applications get more useful data by using an explicit measured pressure value to offset by. Library users currently need to do that offsetting manually, which is inconvenient and somewhat unintuitive.
It would be preferable if we provided a suitable method for setting the current pressure as the basis to measure depth/altitude from, and simply use MSL as the _default_ offset value.
Issue mentioned in [this forum post](https://discuss.bluerobotics.com/t/bm30-mistake-sensor/13843), along with a variety of others previously (e.g. [this comment](https://discuss.bluerobotics.com/t/i2c-level-converter-with-bar30-pressure-sensor/797/43) from 2017).
This kind of offsetting / calibration is [already done in ArduPilot vehicle firmwares](https://github.com/ArduPilot/ardupilot/blob/Sub-4.1/libraries/AP_Baro/AP_Baro.cpp#L844-L853), and used regularly to set the zero-point for ArduSub vehicle depth, and the launch altitude for aerial vehicles.
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.