-
-
Notifications
You must be signed in to change notification settings - Fork 250
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
Ruuvitag sensors - Testers wanted #410
Comments
Hi Ernst, |
Thanks, I’ve responded in the other topic. |
Me neither, but my coding skills are much more limited than yours ;) |
3.4.0-beta has been released with a new parser for Ruuvitag sensors, which should not have the Influxdb issue anymore. Please let me know if this version is working, if one of you have this sensor. |
I updated to 3.4.1-beta and added my two Ruuvitag sensors (they are reporting with v5 format) but no devices/entities are being created in Home Assistant. The Ruuvitag sensors work fine (I am receiving data with the "RuuviTag Discovery" add-on) and I am using ble_monitor with some Xiaomi temperature sensors since quite some time, just the Ruuvitag sensors are not working with it yet. |
@rkallensee Thanks for taking the time to test. I hope you are willing to help. Could you enable debug logging at info level by putting this in your configuration.yaml.
and enable Another, perhaps easier, option is to make a hcidump with the following command.
But this doesn't work on Home Assistant OS. |
Here are two advertisements each from both of my sensors in the log:
|
Thanks, the parser on my local PC works fine for this advertisement. result {'mac': 'EE43F48DE955', 'type': 'Ruuvitag', 'rssi': -86, 'data': True, 'temperature': 19.47, 'humidity': 81.94, 'pressure': 1007.0, 'acceleration': 1023.8281105732544, 'acceleration x': -596, 'acceleration y': -832, 'acceleration z': -28, 'voltage': 2.947, 'tx power': 4, 'motion': 0, 'motion timer': 0, 'packet': 53090, 'firmware': 'Ruuvitag V5', 'battery': 94.7} I will check if I made some other mistake somewhere. No errors in the log? |
No, only these lines, no errors. |
These lines are ok, but they should not have gotten to the point "report_unknown". Can you try to comment out (with a #) the following lines in /custom_components/ble_monitor/ble_parser/ruuvitag.py (from line 103). I think it fails on that part. # Check for duplicate messages
try:
prev_packet = self.lpacket_ids[ruuvitag_mac]
except KeyError:
# start with empty first packet
prev_packet = None
if prev_packet == packet_id:
# only process new messages
return None
self.lpacket_ids[ruuvitag_mac] = packet_id
if prev_packet is None:
# ignore first message after a restart
return None |
Commenting out these lines didn't change anything. Logging a bit in the parser, I found out that |
Where do you see |
I was logging the value of
|
Ah, thanks. You gave enough hints 😄 In the previous version I had the full data passed in, but I should have passed adstruct On line 78 in
|
3.4.3-beta is released, should fix the issue (also for V2/V4). Please let me know if it all works after the update and thanks for the help. |
The Ruuvitag devices now show up with their sensor entities, but now I see this error causing the integration to crash:
|
Remove the underscore _ in Thanks again for testing, it is a bit hard to do it without having the actual sensor. |
Works like a charm. Thank you so much for adding support for Ruuvitag sensors! |
3.4.4 has been released as final. |
In 3.3.1-beta, support for Ruuvitag sensors is added. I'm looking for someone who can test that it is all working
Especially the motion sensor might not work as expected. To explain, the sensor is now reporting motion detected when the motion counter has changed, if it doesn't change it will report motion clear. I'm not sure how fast the sensor will report motion clear, this all depends on the update interval.
So, if you have this sensor and want to do some testing, it would be appreciated.
The text was updated successfully, but these errors were encountered: