Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sensor pulse count #1007

Closed
michalkrejci1 opened this issue Apr 20, 2024 · 8 comments
Closed

Sensor pulse count #1007

michalkrejci1 opened this issue Apr 20, 2024 · 8 comments
Labels

Comments

@michalkrejci1
Copy link

michalkrejci1 commented Apr 20, 2024

Checklist

  • [ Sensor pulse count] I checked for similar existing requests (both open and closed) before posting.
  • [ yes, profi alarm systems have this option ] My request is generic, other users may benefit from it too.

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

Copy link

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

@github-actions github-actions bot added the Stale label May 31, 2024
@michalkrejci1
Copy link
Author

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

@github-actions github-actions bot removed the Stale label Jun 1, 2024
Copy link

github-actions bot commented Jul 1, 2024

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

@github-actions github-actions bot added the Stale label Jul 1, 2024
@michalkrejci1
Copy link
Author

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

@github-actions github-actions bot removed the Stale label Jul 5, 2024
@nielsfaber
Copy link
Owner

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.

@nielsfaber nielsfaber transferred this issue from nielsfaber/alarmo-card Jul 25, 2024
@marazmarci
Copy link

marazmarci commented Jul 29, 2024

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
Copy link
Author

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

Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants