bluerobotics / bluerobotics/ms5837-python

check out ian black's work

Đang mở
#14 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

Ngôn ngữ chính
Python
Star
33
Fork
41
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

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.

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Bắt đầu bằng việc xem xét triển khai Ian Black được liên kết và so sánh sensor API cũng như hành vi áp suất/độ sâu của nó với repository này. Kiểm tra xem sáu thay đổi được liệt kê có phù hợp với các interface MS5837-30BA và MS5837-02BA hiện có hay không; hoàn thành nghĩa là thống nhất về một phạm vi cụ thể và xác thực hành vi thu được đối với các lệnh gọi độc lập cho nhiệt độ, áp suất, độ sâu và độ cao.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
python
Lĩnh vực
embedded-iot
Loại issue
Tính năng
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Cần làm rõ
Mức phù hợp với người mới
20/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.