Person detected doesn't trigger motion sensor

TimE90 commented 3 years ago

I've enabled motion detection for persons. The Eufy camera sends a push notification with "Person detected" but the motion sensor of the camera doesn't get triggered. The sensor always shows "No Motion".

To Reproduce: Add a camera and enable "Person detection".

Expected behavior: Motion sensor gets triggered by "Person detected" notification.

Plugin Config:

        {
            "platform": "EufySecurity",
            "username": "mail@mail.de",
            "password": "password",
            "ignoreStations": "1234,1235",
            "ignoreDevices": "",
            "enableCamera": false,
            "pollingIntervalMinutes": 30,
            "hkHome": 1,
            "hkAway": 0,
            "hkNight": 3,
            "hkOff": 63,
            "enableDetailedLogging": 0
        },

Screenshots:

Environment:

Plugin Version: v1.0.0-rc.0
Homebridge Version: v1.3.4
Node.js Version: v14.16.1
NPM Version: 7.11.1
Operating System: Raspbian

TimE90 commented 3 years ago

As an addition I have just activated the motion detection for "All others". Now Eufy app sends two push messages "Motion detected" and "Person detected".

Nevertheless, the motion detector does not trigger in HomeKit.

lenoxys commented 3 years ago

What is the type of your camera ? If you select DEBUG and restart at very begining of logging you should see a list of device. Can you copy paste the list here ?

TimE90 commented 3 years ago

It's the Eufy Outdoor Cam Pro (C22).

lenoxys commented 3 years ago

Dep on Bropat Lib : https://github.com/bropat/eufy-security-client/pull/39

lenoxys commented 3 years ago

Ok got it. I've done the push request to @Bropat waiting his feedback

TimE90 commented 3 years ago

Perfect! That was fast. Am I able to update via HomeBridge with a beta version 0.3.x?

lenoxys commented 3 years ago

It's a dep from a lib. I need to wait PR merge before merge as well in our plugin. I keep you posted.

bropat commented 3 years ago

@lenoxys Fix released in eufy-security-client version 1.1.2 ;)

lenoxys commented 3 years ago

Can you test on 1.0.0-rc.2 ?

TimE90 commented 3 years ago

It works! Thank you!

One off topic question: do you recommend the camera livestream via this plugin or via RTSP and "Homebridge Camera FFmpeg"?

lenoxys commented 3 years ago

via plugin for now.

Thanks for your feedback and thanks to @bropat ! I'm closing this issue !

TimE90 commented 3 years ago

Hey @lenoxys ,

I've just updated to latest rc 13. But now my indoor and some outdoor cameras are not receiving the notification for person detection anymore. Would try to test with rc2 again but can't select rc2 in Homebridge ui.

TimE90 commented 3 years ago

Ok, here a example from log file:

[9/11/2021, 3:17:25 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Push Connected!
[9/11/2021, 3:17:55 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Garage Property Changes: personDetected { value: true, timestamp: 1631366273810 }
[9/11/2021, 3:17:55 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Garage Property Changes: personName { value: 'Unknown', timestamp: 1631366273810 }
[9/11/2021, 3:17:55 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Garage Property Changes: pictureUrl {
  value: '…'
  timestamp: 1631366273810
}
[9/11/2021, 3:17:55 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Garage ON DeviceMotionDetected: true
[9/11/2021, 3:18:06 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Garage Property Changes: personDetected { value: false, timestamp: 1631366286128 }
[9/11/2021, 3:18:06 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Garage Property Changes: personName { value: '', timestamp: 1631366286128 }
[9/11/2021, 3:18:06 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Garage ON DeviceMotionDetected: false
[9/11/2021, 3:18:28 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Innen Property Changes: personDetected { value: true, timestamp: 0 }
[9/11/2021, 3:18:28 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Innen Property Changes: personName { value: 'Unknown', timestamp: 0 }
[9/11/2021, 3:18:38 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Innen Property Changes: personDetected { value: false, timestamp: 1631366318299 }
[9/11/2021, 3:18:38 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Innen Property Changes: personName { value: '', timestamp: 1631366318299 }
[9/11/2021, 3:18:38 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Innen ON DeviceMotionDetected: false

The camera Garage is working, but the camera "Innen" isn't working.

Garage = INDOOR_OUTDOOR_CAMERA_2k Innen = INDOOR_PT_CAMERA

TimE90 commented 3 years ago

This one is the same. The log shows correct detection but inside the Home app there is no motion for the sensor.


[9/11/2021, 3:23:01 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Garten Property Changes: personName { value: 'Unknown', timestamp: 1631366580171 }
[9/11/2021, 3:23:01 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Garten Property Changes: pictureUrl {
  value: '…',
  timestamp: 1631366580171
}
[9/11/2021, 3:23:11 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Garten Property Changes: personDetected { value: false, timestamp: 1631366591318 }
[9/11/2021, 3:23:11 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Garten Property Changes: personName { value: '', timestamp: 1631366591318 }
[9/11/2021, 3:23:11 PM] [EufySecurity-1.0.0-rc.13] DEBUG: Garten ON DeviceMotionDetected: false```

lenoxys commented 3 years ago

Hello,

Did you enable the notification on your apple device ?

lenoxys commented 3 years ago

I've just done the test on rc13 and got the notification

You need to make sure to enable push notification when editing the arming mode.

github-actions[bot] commented 3 years ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.

github-actions[bot] commented 3 years ago

This issue was closed because it has been stalled for 5 days with no activity.

samemory / homebridge-eufy-security

Person detected doesn't trigger motion sensor #184