elad-bar / ha-shinobi

Shinobi Video custom component for HA
53 stars 5 forks source link

Motion sensor is being created although it's not enabled #10

Closed ovizii closed 3 years ago

ovizii commented 3 years ago

The description of this addon says:

Description Integration with Shinobi Video NVR. Creates the following components:

Camera - per-camera defined. Binary Sensors (MOTION, SOUND) - per-camera defined.

The result of adding 1 shinobi camera with your addon: image

elad-bar commented 3 years ago

Not sure I understand, what is the issue?

ovizii commented 3 years ago

Not sure if I am misunderstanding something but should your addon not create a binary sensor for Sound for this camera? As you can see it only created 1 binary sensor for Motion.

ovizii commented 3 years ago

Does this mean somehow the sound from the shinobi camera isn't picked up?

image

elad-bar commented 3 years ago

Binary sensor of audio is dependent on sound detector, please enable it to have that binary sensor

ovizii commented 3 years ago

yes, motion detection is indeed turned off in shinobi but so is motion detection and that binary sensor gets created, hence my confusion.

I'll share some relevant shinobi settings:

image

image

image

image

image

ovizii commented 3 years ago

Does this mean somehow the sound from the shinobi camera isn't picked up?

image

Does this really mean, HA is not picking up any sound via this camera? I tried changing the stream settings in shinobi from "Audio Codec: copy" to a few other codecs and HA is still not seeing any audio.

Any hints?

image

elad-bar commented 3 years ago

Will recheck it during the weekend, thanks for the details

ovizii commented 3 years ago

Will recheck it during the weekend, thanks for the details

Sure, may thanks for your help, especially since I don't think its related to your addon but more like a wrong setting on shinobi side.

elad-bar commented 3 years ago

Can you please export the camera setting (json) and share it (after cleanup of ip and credentials)?

Thanks

ovizii commented 3 years ago

Should be clean of any passwords. Just remove the .txt at the end. Shinobi_doggyCam_config.json.txt

elad-bar commented 3 years ago

thanks for the config file, regarding motion sensor, code was referring to the wrong parameter of detector for motion detector - it should be detector_pam, As for the audio detector, according to the config and the screenshot the audio detector is turned off, in order to enabled it, please enable it, reload the component / restart HA and let me know if it works for you

thanks

ovizii commented 3 years ago

I don't need the motion detector, I was just wondering why motion detector sensor was there but not audio detector sensor since both were turned off in shinobi.

so basically, once you update your integration, when I add the camera again, none of the 2 sensors should be created since both detection methods are off in shinobi, right?

elad-bar commented 3 years ago

That's the plan 😀, just wanted to make sure that I don't need to check the audio sensor, Thanks