nielsfaber / alarmo

Easy to use alarm system integration for Home Assistant
1.35k stars 122 forks source link

Sensor pulse count #1007

Closed michalkrejci1 closed 2 weeks ago

michalkrejci1 commented 6 months ago

Checklist

Proposal

Please consider whether it would be possible to add a "pulse count" option somewhere, preferably in the advanced settings for an individual sensor.

I have used this feature on some "HW" alarms and some sensors can do it too (hw jumper - two pulse=alarm)

If I'm typically using a PIR that can't do some sort of false alarm filter, it would be absolutely perfect if this system added this functionality.

That is, the number of pulses a sensor has before its signal is considered valid.

Many thanks for considering this functionality. Michal

github-actions[bot] commented 4 months ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days

michalkrejci1 commented 4 months ago

Hello, I am still alone ? Isn't this functionality worth considering ?

github-actions[bot] commented 3 months ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days

michalkrejci1 commented 3 months ago

So no one wants to join the plea for a pulse counter to prevent false alarms if insects fly past the sensor, etc...

nielsfaber commented 2 months ago

Why did you post this under alarmo-card? Alarmo-card is only used to arm/disarm the alarm via the HA dashboard. This request seems to be in the wrong place.

As for your request: if I understand you correctly you are requesting a feature to have alarmo ignore the first state change of a sensor but trigger on the next one instead? In alarmo a different approach is used, here you can set up a sensor group comprising of multiple sensors. The triggering of the alarm only occurs when multiple sensors are triggered within a set time window. I have experienced issues with PIR sensors becoming unreliable either when sunlight is directly facing them or when their battery is almost empty. In these cases I have seen flickering, so even with a pulse count a false alarm would occur. Therefor I think the sensor grouping is a much more reliable approach.

marazmarci commented 2 months ago

Some preliminary detection delay* could be more useful for motion sensors to filter out false positives, similar to the entry delay, but with the difference that if it stops detecting motion before the delay expires, it won't trigger the alarm.

(*: alternatively, it could be described as hold time or debounce time)

update: #1014 is about just this

michalkrejci1 commented 2 months ago

Hi, yes. That sounds reasonable, basically a spider crawls over the PIS and creates a single pulse, should the pulse occur before the resetable timer expires it will trigger an alarm, if not within x seconds the timer will restart.

That would be about right.

Thank you, Michal

github-actions[bot] commented 3 weeks ago

This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 7 days