amcrest2mqtt icon indicating copy to clipboard operation
amcrest2mqtt copied to clipboard

Update amcrest2mqtt.py

Open carefulcomputer opened this issue 3 years ago • 6 comments

JSON structure seems to have changed in latest firmware. Updated the row 426 to use right json path for human detection.

Credits to jweston2112 for finding the issue.

carefulcomputer avatar Oct 30 '22 17:10 carefulcomputer

This fixed my issues with the Amcrest 410. HA is now ingesting events.

unixorn avatar Nov 04 '22 04:11 unixorn

I built a multiarch image with this change - unixorn/amcrest410-2-mqtt you can use until this gets merged.

unixorn avatar Nov 07 '22 04:11 unixorn

I need a stable fork to base my addon on, if anyone can confirm this fix still works I would be happy to switch to @carefulcomputer's fork till the main repo is updated

ikifar2012 avatar Mar 08 '23 23:03 ikifar2012

@ikifar I just took a look at my history and see some human detection events in the log.

unixorn avatar Mar 14 '23 00:03 unixorn

And motion detection too.

unixorn avatar Mar 14 '23 00:03 unixorn

FWIW the motion detection is reliable, the human detection fires events sometimes but not always.

unixorn avatar Mar 14 '23 00:03 unixorn