-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Add support for Oil-SonicSmart #2279
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
Conversation
400bb8e
to
accad40
Compare
I'll run the new code and monitor. |
Thanks! Over all very much like the "Standard" model/protocol then. |
I think the sensor is pretty much the same as the classic Apollo sensor, it might just be the receiver that's different. The Apollo kit is very badly documented so it's difficult to know. Should I build this branch by source and run it that way or is there a branch release? Eventually I want to run this within the Home Assistant RTL add-on but can run within an Ubuntu VM for the time being for testing. |
In terms of features very much the same, but there is more "room" in the data fields, so e.g. flags might map differently. If you running docker is more convenient then use the branch |
Yup, I've built from the feat-oilsmart branch and I've left it running in an Ubuntu VM (rather than docker). Fingers crossed no hardware crashes overnight. I think the oil monitor only sends updates very infrequently so they're difficult to catch! What were the specific flags in the classic monitor? I'll see if I can figure out from functions in the smart display what any additional flags might be and how I could trigger/test them. |
That would be great! The knowledge we have is summarized here: rtl_433/src/devices/oil_standard.c Lines 60 to 84 in 46491d3
|
Ok, so I left it collecting data overnight and while it has now identified the data and rtl_433 recognises it the data contained within must be being decoded incorrectly. rtl_433 shows: model: oil-ultrasonic The alarm value appears to change variously to: Readings appear to be every half and hour or so. |
Thanks. I've updated the branch with those hints and disabled the binding/depth switch (same value anyway) |
I'm now getting depth readings. model: oil-ultrasonic I've no idea if this depth reading is correct at the moment, I'll have to pull it out of the tank again and measure against a known distance. |
We've used oil since yesterdays comment (smart display shows less in litres) but the depth_cm is still reading 51 so I don't think it's decoding the correct part of the data. I'll attempt to get the sensor out today to see if that makes a difference. |
Can you simply compare the reading you got before and after the change? We only need/can compare these three The first two bytes will have a bit (MSB) for >255 cm, and may also have bit(s) for sub-cm reading. 2 bits for 1/4 cm readings maybe. |
Sorry, I'm not sure I understand what you mean. Which readings do you think I should compare? |
Yesterday it was |
This is the last ~24hrs |
depth_cm: 52 flags are 17 and alarm 30 or 32. Anything else I should be doing/testing at the moment? |
We can copy your table to this BitBench. The bits just before the "cm" field seem to roughly increase, e.g. |
This is looking good, I've got data since my last post and everything seems pretty stable. The flags data seems to be very stable, the alarm data seems to regularly change but I'm not sure what that's related to. |
OK, thanks. I guess "flags" then includes alarm and "alarm" is really a counter or temperature or such. |
I guess if it merges and more people come across it we may come across other events. We've not had a refill yet for example, I guess those things can always be added as time goes on. |
aa9894a
to
7b1e769
Compare
Merging this now. If you ever get the chance to observe a distance going above 255 cm then please post the output (e.g at 250, 255, 256, 260). Also if you detect a pattern in the "unknown" fields :) |
See #2244
Open questions: