-
Notifications
You must be signed in to change notification settings - Fork 84
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
Eufy Motion Sensor - Binary sensor states #22
Comments
can you confirm if changes are states happening in similar times? it can be an issue of mapping of eufy states into HA states. |
I will check it now and get back to you shortly. |
Maybe not. It didn't change from 'false'. I guess that would mean that it simply isn't getting any state from the sensor. Here is the info from the sensor state in developer tools:
compared with the state from the doorbell motion sensor:
|
please share your motion sensor with my eufy account so I can debug. |
How do I share it? |
apparently, only cameras are shareable between accounts, I will try to improve this blindly, hopefully we can make it |
Would the states from the MQTT entities be of any help? I did see that those are "on" or "off", but that may just be within that integration. Those do trigger, by the way, but I know nothing about how to set up MQTT automations etc. |
Please enable excessive logging following last section here: https://github.com/fuatakgun/eufy_security#troubleshooting |
This issue is about motion sensor device, not motion sensing functionality of cameras, please create a separate issue to discuss this.
|
Hey just chiming in here to mention my motion sensor doesn't work either and can help with any information you require as well. I am trying to have the motion camera turn on lights in a room when it detects motion but there seems to be no way to do this with a Eufy Motion sensor currently. I can't even get it to show up in Google Assistant (and it's officially supported), only the Homebase shows up there. |
I need you to enable excessive logging for the integration and trigger some events from motion sensor and share your logs from home assistant regarding. This will be first step. |
Here are what the logs look like when I trigger one motion sensor notification: Note: I swapped the station serial numbers for y's, and the motion sensor serial for x's.
|
That log corresponds to a single motion sensing trigger. I have no idea why it has multiple "false" trigger events and one "true", but that is one trigger that generated all those. |
I think the true values may indicate that "motionDetection" is enabled, which appears to be different from "motionDetected", which is always showing false. |
Ah yes I do see there is a difference there. Unclear though why it would be indicating that is a "property change" as the property was not changed by me. Also weird how it keeps throwing motion detection "false" despite it sending a notification to the contrary. Looking at my other logs of motion triggers it looks like it is the same pattern every time. So for some reason it seems like the |
thanks for investigation, so individual motion sensor should use motionDetection rather than motionDetected, now it is clear, but I believe this is a mistake. actions;
|
Yes that seems correct and it does seem to point to a naming error coming from the eufy-security-client |
I don't think it is a naming error. The phone app allows for motion detection to be turned off or on. There is also a timeout period after a detection event in which it will not register another motion event. I believe that it is simply indicating if motion detection is enabled, then indicating that the camera or sensor has entered the period in which detection is disabled, and subsequently, when it was re-enabled. |
@bropat , it would be nice to get your opinion on this. when motion sensor receives an motion event, This looks contradicting with camera but you know better internal dynamics. |
@boctok In my case I have 0 second delay set for the motion triggering cooldown so this may explain why the log messages are all happening at the same time then. |
Another interesting thing I noticed while looking through the logs is that sometimes, the URL for the snapshot of the last activity is https://security-app.eufylife.com/imageName and at other times, it is https://zhixin-security-pr.s3.us-west-2.amazonaws.com/ etc. |
I would rely on enabled/disabled attribute rather than |
please get latest version of codebase and retry and share your results. resolve if possible. (just published seconds ago) |
Looks like the "motionDetected" reported a 'true'... |
I checked the state within developer tools, since I have this one camera sitting on my desk waiting to be mounted and sure enough, I can trigger the state between off and on by moving my hand in front of the camera. |
@fuatakgun That is a typing error on my part. It will be fixed in the next release. If someone has such a device and could share it with me for a short time, I could check everything and implement missing features. |
I asked this earlier but user was not able to find sharing option and i thought it is not shareable |
As far as I know, I have the latest. I don't see any notification that there is a new one. |
so, you should have an entity named |
|
but I do not see any event here which arrives as True. This is contradicting with this: #22 (comment) |
That was from one of the cameras, which had also, up to that point, not given a result of true. |
From the camera Model# T8114
|
it is not clear to me why you are referring cameras here? we are only interested independent motion sensor, not motion sensing functionality of a camera. Motion or Person sensor for cameras should work without any issues, if you are facing some, let's keep it independently. I am interested below product. |
My apologies. I thought the issue was with motion detection in general, because until a few days ago, none of my cameras were providing any motion detection events either. Again, I apologize. |
They kind of worked before, but until a few days ago, I couldn't use them as triggers for automations. I will stick to the motion detector from now on. |
This was a BUG that was fixed in version 1.1.0 of the driver (eufy-security-client). In any case, you should use eufy-security-ws version 0.4.1. |
As far as I can see, eufy-security-ws version 0.4.1 isn't available in HA yet. I'll keep an eye out for it. |
That is on me, i need to test it throughly and probably include newly added
versions, please follow it closely but do not expect very soon, on vacation
:-)
…On Mon, Aug 16, 2021, 18:33 boctok ***@***.***> wrote:
As far as I can see, eufy-security-ws version 0.4.1 isn't available in HA
yet. I'll keep an eye out for it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#22 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACUSN7RJUJK2UYZRSDOYS4LT5EVTXANCNFSM5BAOJDXQ>
.
|
Enjoy yourself! |
So if I wanted to use this motion sensor to do something like turning on a light in a room and keeping that light on while it detects motion say every 5 minutes, I'd have to have notifications on for this? Is there any workaround or way around having to have the devices notifications on? This works fine for cameras for me as they don't detect motion nearly as often as I will want this motion sensor detecting it. |
That's the way it seems to work for mine. I tried with "React to motion" unchecked and the state never changed. I tried it with "React to motion" checked, but "Push notification" unchecked and the state never changed. It will only change the state if "React to motion" and "Push notification" are both checked. I'm not sure if @fuatakgun can work some sort of magic and make it only update the state on HA, but I kind of doubt it, because it's a push notification. HA and your phone get the same notification. |
Maybe you can have one account to hold all cameras and motion sensors to use with HA and have a separate one for your mobile app without motion sensor. I have just built the bridge between eufy security ws and HA, magic comes from @bropat |
Yesterday I bought a motion sensor. As soon as it arrives I will take a closer look at it to understand if I can do some magic. ;) |
I found out that with the wireless doorbell the motion sensor entities do work however, it depends on how you setup your Eufy device in the App. If set to all motion, only the motion sensor works and the person detector sensor doesn't. If you set it to Persons only then the normal motion sensor doesn't work. I was hoping that both would work regardless of the setting in the app you so you could have a combination of motion or motion and person simultaniously. |
FYI: this thread is about |
@sjikkervisje , thanks for your findings, can we continue over here: #49 I believe they are directly related |
back to here, we know that our integration relies on push notifications for events, if we are not able to receive any push notifications, there is no way around this. @bropat , do you foresee a solution here other than adding the motion sensor into integration specific eufy account? |
After I received the motion sensor, I checked how it works and unfortunately I have to inform you that the motion notification can only be integrated via the push notifications. There is no other interface that is openly accessible, not even via the P2P interface to the station. |
Thanks for update, i will close the issue, @boctok , please go with a separate account if you are worried about number of push notifications. |
One thing I noticed while trying to set up an automation using the binary switch motion detector entity, checking the logs, it goes from "false" (no motion detected) to "unavailable". I don't see any records of "true" or any motion detected, even though it shows up in the Eufy app.
Originally posted by @boctok in #15 (comment)
The text was updated successfully, but these errors were encountered: