bluerobotics / bluerobotics/tsys01-python

check out ian black's work

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

Description

https://github.com/IanTBlack/br_sensors

1) Changed variable/function names to more closely match TSYS01 manual.
2) Combined read() and temperature() functions into one.
3) Split _calculate into two functions (_read_adc and _adc2temp).
4) Added multiple selection options to give user options for selecting temperature conversion units.
5) Changed Kelvin conversion from 273 to 273.15.
6) Added experimental burst_avg_temperature. Rapidly takes X number of samples, drops the first and last, and averages the rest. Need to run tests to determine if it provides any benefit.
7) Added serial number function. I'm still trying to understand how bit manipulation works, so I may have gotten this wrong. I assumed it could be manipulated the same way as _calculate.

Contributor guide

No contributing guide indexed for this repository

Research direction

Review the implementation at https://github.com/IanTBlack/br_sensors and compare its TSYS01 changes with this repository. Start by checking the listed read, temperature, _calculate, _read_adc, _adc2temp, burst_avg_temperature, and serial-number behavior; done means deciding which changes belong here and validating the accepted behavior with tests.

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
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.