Skip to content
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

HA wont detect person movement #91298

Closed
MrGlad8 opened this issue Apr 12, 2023 · 8 comments
Closed

HA wont detect person movement #91298

MrGlad8 opened this issue Apr 12, 2023 · 8 comments

Comments

@MrGlad8
Copy link

MrGlad8 commented Apr 12, 2023

The problem

I have been using Reolink now for 1½ years using the fwestenberg HACS-integration without problem. The camera notice person movement without problem.
Tried today uninstall the HACS version to try this official support... but it wont work! Home Assistant will simply not register movement infront of my camera everytime. Sometimes it do register the _motion-sensor. But so far the _person-sensor has never been anything else than "off".

One diffrence here between HA version and fwestberg is that he had 2 motions sensors with "clear and detected". One for Motion and one of Person.

This new HA version only have "clear and detected" with the motion sensor.
The person-sensor seems to have "on and off". There might be a problem there maybe?

I'm using RLC 510A and it should be supported.
The HACS version had more settings, but what I can see I used protocol RTMP there and I selected that as well in the HA version of the integration.
Is this a known bugg?

What version of Home Assistant Core has the issue?

2023.3.6

What was the last working version of Home Assistant Core?

none

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Reolink IP NVR/camera

Link to integration documentation on our website

https://www.home-assistant.io/integrations/reolink

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

@home-assistant
Copy link

Hey there @starkillerOG, mind taking a look at this issue as it has been labeled with an integration (reolink) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of reolink can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign reolink Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


reolink documentation
reolink source
(message by IssueLinks)

@MrGlad8
Copy link
Author

MrGlad8 commented Apr 12, 2023

Actually, I tried to restore an old backup from yesterday when I was using fwestenberg HACS integration. And that "person detected" sensor was "Unavalible". Until I showed my self infront of camera it turned "detected". So that integration still works.
But that on/off person detected sensor wont work with the new HA integration.
Maybe ask the guy behind the HACS integration?

@starkillerOG
Copy link
Contributor

@MrGlad8 the HACS integration could potentially miss some events, I made a fix for that particular issue by shielding the ONVIF push callback.
Unfortunetly that introduced a new problem that actually made HA miss even more events.....
A fix for that new issue has already been made in this PR: #91070
It is under review and will hopefully make it into HA 2023.4.4, but can't promise anything.

@starkillerOG
Copy link
Contributor

If you appreciate the reolink integration and want to support its development, please consider sponsering the upstream library.

@MrGlad8
Copy link
Author

MrGlad8 commented Apr 14, 2023

@MrGlad8 the HACS integration could potentially miss some events, I made a fix for that particular issue by shielding the ONVIF push callback. Unfortunetly that introduced a new problem that actually made HA miss even more events..... A fix for that new issue has already been made in this PR: #91070 It is under review and will hopefully make it into HA 2023.4.4, but can't promise anything.

Thanks for the answer!

@starkillerOG
Copy link
Contributor

HomeAssistant 2023.4.5 is now available and includes the ONVIF push fix.
Please let me know if it is now working for you on 2023.4.5.
I will close this issue, if it still does not work for you, please create a new issue.

@starkillerOG
Copy link
Contributor

If you appreciate the reolink integration and want to support its development, please consider sponsering the upstream library.

@starkillerOG
Copy link
Contributor

@home-assistant close

@github-actions github-actions bot locked and limited conversation to collaborators May 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants