Closed efirestone closed 4 years ago
Hi, thanks for the work, I will take the readme parts relevant but setting the type as hard-coded instead based on &TYPE
is worng,
will explain why:
Please let me know if you would like to:
thanks!
Ah! Good to know. The root issue for me is that I have the motion detection set up on the camera hardware itself, which then shows up as an ONVIF event in Blue Iris. Whether those events get grouped into the existing motion sensor entity or are discrete doesn't totally matter to me. If they're discrete I could see it being slightly annoying to have all of those sensor show up if you're not using ONVIF or DIO, so maybe there's a way to only create them after those events are first seen in MQTT? Or offer a config setting to opt into them in the integration.
The other alternative I could see is to have a single "motion" sensor entity with an attribute of something like "trigger_type". That setup feels simpler to me for what I suspect is the common use case where people care about any sort of motion/audio/presence, but don't actually care what the detection method is. For those that do care about separating out audio, that should be a pretty simple template sensor in HA (but would require slightly more work for that case). This would also be a breaking change of course since the audio entity would disappear.
I'll update this in a bit to go back to using &TYPE
(which is great; it's simpler), and I'm curious about your thoughts on how to best show the other sensor types.
Released new version with external & dio support, now you can choose which sensors to create motion or external or having both of them (to use with different functionality)
Implemented enhancements:
Note: All camera will have 2 additional sensors, to disable use integration's options
Amazing! That seems like the ideal usage. Will try it out tonight.
I noticed some of these settings have changed since BI4, and also that the
&TYPE
macro that was previously mentioned doesn't actually work since it inserts things likeONVIF
orDIO
as the type value, rather thanMotion
.Alternatively we could probably simplify the setup procedure by having the code look for all of the different source types (like
ONVIF
andDIO
) rather thanMotion
and leave the&TYPE
in the instructions.