bluerobotics / bluerobotics/ms5837-python

check out ian black's work

Open
#14 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
33
Forks
41
PR merge metrics
No merged PRs in 30d

Description

https://github.com/IanTBlack/br_sensors

1) Used TEOS-10 pressure to depth conversion to compute depth.
2) Changed variable/function names to more closely match MS5837-30BA manual.
3) Combined read and temperature/pressure/depth/altitude functions into one.
4) temperature/pressure/depth/altitude are called independently of one another. They do rely on one another, but I formatted in a way that doesn't require users to call pressure before calling depth.
5) absolute_pressure() give the absolute pressure reading in user selected units
6) pressure() removes atmospheric pressure if given. if not it uses standard atmospheric pressure.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing the linked Ian Black implementation and comparing its sensor API and pressure/depth behavior with this repository. Check whether the six listed changes fit the existing MS5837-30BA and MS5837-02BA interfaces; done means agreeing on a concrete scope and validating the resulting behavior for independent temperature, pressure, depth, and altitude calls.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
embedded-iot
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
20/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.