j-b-n / domoticz_reolink

Domoticz plugin for use with Reolink cameras.
GNU General Public License v3.0
2 stars 0 forks source link

Handle more than one camera at the same time in the plugin #2

Open j-b-n opened 7 months ago

j-b-n commented 7 months ago

Suggestion: Be able to handle more than one camera at the same time in the plugin.

dyter69 commented 7 months ago

I use 4 cameras, but in my opinion it should be a priority to add options that work for everyone, because I needed to mod the plugin.py and it's perhaps difficults for over:

j-b-n commented 7 months ago

I am curious about your setup when it comes to the plugin folder. I changed to HomeFolder in 0.0.2 which is "Folder or directory where the plugin was run from.", does that not work for you?

Python or Python3 - camera.py requires python3 so I guess I can change to python3 as the default name for the python binary. I assume you have both python2 and python3 installed in your system.

The detection rules are interesting. I am using data from the camera to identify what sensors should be created. That would mean a camera that supports the detection of pets should have a sensor that is triggered for that. In your example you would disable some of those sensors. Why not just hide the sensor in the Devices tab?

dyter69 commented 7 months ago

Yes camera.py path folder work now, thank for mod. I use the official domoticz docker, and only python3 is installed in default, so I need to mod in plugin.py "python" to "python3"

I use 4 reolink cameras, and have too many devices created. For my alarm senario, I only use person detection, other is useless for me

j-b-n commented 7 months ago

Ok, I have changed to "python3" now.

I will think about how to solve the creation of devices. Easiest I guess is to allow for some sort of list of what devices is created.