New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add support for HT1 #20
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #20 +/- ##
==========================================
+ Coverage 84.81% 87.61% +2.80%
==========================================
Files 2 2
Lines 79 113 +34
Branches 16 24 +8
==========================================
+ Hits 67 99 +32
- Misses 5 6 +1
- Partials 7 8 +1 ☔ View full report in Codecov by Sentry. |
0339acb
to
8e3a921
Compare
8e3a921
to
402bb07
Compare
Hey @bdraco I believe all comments have been addressed and I've added a couple more tests to improve the code coverage |
Looks good. I added a few more review comments above |
I ordered an HT1 to test with. Should be here sometime after the new year |
481415b
to
6d2df78
Compare
The ci is really out of date for this library. I'm going to need to fix that before this can be released |
Thanks @ashleigh-hopkins |
https://pypi.org/project/sensorpush-ble/ 1.6.1 published |
Hey, I reverse engineered the calculation for the HT1 and thought it might be useful for some people with this older device. Tested with the single HT1 I own (I also have a HT.w).
I wrote some tests, but I'm not actually sure how to execute them - I'm pretty new to python 😅
related issue home-assistant/core#76638