bluerobotics / bluerobotics/tsys01-python
check out ian black's work
- 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