pai icon indicating copy to clipboard operation
pai copied to clipboard

Pushover notifications for "restore" events

Open lpaolini opened this issue 1 year ago • 9 comments

Alarm system

EVO192 v.7.70.1

Environment

RPi/docker

Git branch

master

Questions

  1. Why do I receive Pushover notifications for "trouble restore" events, with the following configuration?
  2. Is there a way for lowering the priority (e.g. 0 instead of 2) for "restore" notifications?
PUSHOVER_EVENT_FILTERS = [
    'live,alarm,-restore',
    'trouble,-clock,-restore',
    'live,tamper,-restore',
    'live,arm',
    'live,disarm'
]

lpaolini avatar Sep 27 '24 08:09 lpaolini

  1. Strange, should be filtered out.
  2. Not possible with the current implementation.

yozik04 avatar Sep 27 '24 09:09 yozik04

Give me an example of an event that comes through.

yozik04 avatar Sep 27 '24 09:09 yozik04

Or check yourself. If you have EVO then this is correct file: https://github.com/ParadoxAlarmInterface/pai/blob/dedc352b9babcac198821003bcbc6c79a99f57e0/paradox/hardware/evo/event.py#L509

Maybe tags are missing for your event.

yozik04 avatar Sep 27 '24 09:09 yozik04

Also if you want to study the subject in depth here are tests that describe possibilities pretty well: https://github.com/ParadoxAlarmInterface/pai/blob/dev/tests/lib/test_event_filter.py

yozik04 avatar Sep 27 '24 09:09 yozik04

Give me an example of an event that comes through.

There you go.

Screenshot_20240927-122334

lpaolini avatar Sep 27 '24 10:09 lpaolini

Add "-zone_tamper_trouble=False". Should fix. And "-trouble=False", to discard "No trouble for ..."

'trouble,-clock,-restore,-zone_tamper_trouble=False,-trouble=False',

yozik04 avatar Sep 27 '24 10:09 yozik04

Thanks! I wish I could understand this syntax fully :-) I'll try digging into the code...

In any case, I think treating restore events as emergency notifications with mandatory acknowledgment is a bit too much. Perhaps priority 0 or even -1 would be more appropriate for this kind of events. Wouldn't it?

lpaolini avatar Sep 27 '24 12:09 lpaolini

Not that simple. We have two things that emit events that end up sending notifications.

  • Live events sent by the panel.
  • Property changes that were detected during RAM memory periodic reads by PAI.

For Live events it is easy to filter out what you want as most of them have 'restore' tags set. Property changes are mostly booleans and there are no restore tags assigned to the property changes.

BTW: you probably can use: 'live,trouble,-clock'

This will send events only for Live events sent by the panel.

But I recommend you to fix the root cause of the trouble =) It is not a normal situation.

yozik04 avatar Sep 27 '24 12:09 yozik04

Thanks for the explanation!

Sure, I'll try to fix the root cause of the trouble. Not trivial though, as it seems to be caused by sun rays hitting one of the outdoor IR sensors at some time in the morning, and that seems to be interpreted as a masking attempt.

lpaolini avatar Sep 27 '24 12:09 lpaolini