-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Fix SHT3xd fails sometimes in 2024.4.0 #6592
Conversation
1) read serial number using clock stretching enabled register and if that fails try alternative clock stretching disabled register 2) add error codes to setup show in dump_config()
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #6592 +/- ##
==========================================
- Coverage 53.70% 53.43% -0.28%
==========================================
Files 50 50
Lines 9408 9537 +129
Branches 1654 1685 +31
==========================================
+ Hits 5053 5096 +43
- Misses 4056 4130 +74
- Partials 299 311 +12 ☔ View full report in Codecov by Sentry. |
Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍 |
Co-authored-by: Jesse Hills <3060199+jesserockz@users.noreply.github.com>
@jesserockz - I committed your suggested changes. This should bring things back to where they were. I am seeing if I can resolve issue#5303 - hopefully ready for May. Thanks |
Thanks for this!! I rolled back to 2024.3 as I had failures at random times with the sensor. Looking forward to the fix! |
What does this implement/fix?
Update to this component in release 2024.4.0 caused failure for some users on reading serial number.
This fix reinstates original register for reading serial number and if this fails the alternate is used. Error codes have been added to setup and are shown in dump_config
Types of changes
Related issue or feature (if applicable): fixes esphome/issues issue#5710
User that raised issue has tested fix successfully plus another user
Pull request in esphome-docs with documentation (if applicable): NA
Test Environment
Example entry for
config.yaml
:NA
Checklist:
tests/
folder).If user exposed functionality or configuration variables are added/changed: