Closed snozzley closed 1 year ago
This looks like it may be the problem: https://github.com/ikifar2012/amcrest2mqtt/blob/main/src/amcrest2mqtt.py#L372 That looks like it's stamping the image over the doorbell topic?
So what I did was change the fork of the addon which was supposed to solve some of the issues people have been encountering but it seems like the main branch has some bugs, I have been thinking of switching it to the patch-1 branch which only has a one line change https://github.com/dchesterton/amcrest2mqtt/pull/91. I do not have the doorbell to test so I need testers, I have also been thinking of using this fork as it looks somewhat more organized https://github.com/layertwo/amcrest2mqtt
Happy to test either for you - just not sure how to load a specific version into HA as the addons seem to just pull 'latest' ?
Otherwise if you're wanting to update/fix yours, I think it would just be a matter of putting the camera snapshot in a different topic, and publishing the doorbell topic. I could try make a PR for that if you wanted?
I think some other things are broken too, as for HA yes it only pulls whats in the config file in my addons repo https://github.com/ikifar2012/ha-addons I will switch the branch over to patch-1 in the next release and hopefully that will solve things
Was human detection ever broken for you because that's what patch-1 (https://github.com/dchesterton/amcrest2mqtt/pull/91) aims to solve
I have never used it (Or even sure if AD110 supports it?) sorry
Any idea when the next version is released?
I also did not use presence, only want the button trigger :)
Let me know if 2023.5.0 fixes anything
I just updated to 2023.5.0 and I still have the issue where doorbell press event (a binary sensor) is still unavailable. This stopped working a few weeks ago, any ideas/troubleshooting suggestions?
Same here after the 2023.5.0 update. Also removed it and added it again.
any errors in the logs or anything strange in mqtt?
After updating I am still seeing the same thing as @snozzley in the original post. There is an image being sent in the doorbell event amcrest2mqtt/serial#/doorbell. If there is any addition info needed please let me know. And thanks for maintaining this add-on. I appreciate it!
Doorbell is working well again with 2023.5.0, thanks! :D
FYI, if you are using go2rtc (such as via frigate), make sure you are using the correct back channel @mattstrong121 @karthikvenkat123 . If you are not, go2rtc will steal the doorbell press and you'll be in constant 'call' mode and you'll always be in an unknown state. To fix that, use the following similar like for go2rtc config:
rtsp://user:pass@hostOrIp:554/cam/realmonitor?channel=1&subtype=0#backchannel=0
So what I did was change the fork of the addon which was supposed to solve some of the issues people have been encountering but it seems like the main branch has some bugs, I have been thinking of switching it to the patch-1 branch which only has a one line change dchesterton/amcrest2mqtt#91. I do not have the doorbell to test so I need testers, I have also been thinking of using this fork as it looks somewhat more organized https://github.com/layertwo/amcrest2mqtt
I'd help testing. I'm encountering this issue I believe. I have the ad110.
I think the doorbell event for AD110 devices is no longer working.
I think this may be related to the recent release, but I'm not sure how to downgrade an addon in HA to test older versions.
In the logs I get the doorbell event like below:
However in MQTT I seem to be getting an image inserted where I previously had the doorbell event. i.e.
previously I would check for:
Has this maybe moved elsewhere? (I tried checking all MQTT events, but couldn't see a doorbell event getting set anywhere else)
Version info on my AD110: