nielsfaber / alarmo

Easy to use alarm system integration for Home Assistant
1.39k stars 120 forks source link

Trigger time & Actions condition for environmental sensors #366

Closed Masterz69 closed 2 years ago

Masterz69 commented 2 years ago

Checklist

As see no way to reopen closed issue - creating new one with reference on existing.

Proposal

Add mode "Disarmed" (or with another name) in General->Modes with only "Trigger time" to be defined and used for environmental etc. sensors, active in all "Arm modes" (aka "(Always)").

F.e. Xiaomi implement so called "Basic Guard" - for environmental sensors, which active always, independently from Armed% modes.

As well as in Actions: would be nice to have conditions:

Originally posted by @Masterz69 in https://github.com/nielsfaber/alarmo/issues/326#issuecomment-1008281514

nielsfaber commented 2 years ago

I'm sorry about the closed requests. I do agree that there is a need being able to create dedicated actions for fire/water leak alarms. On the other hand, I am wondering whether such sensors really belong in Alarmo, or it's better to treat these with separate automations/blueprints. What is the added value for having them inside Alarmo, if their behaviour (triggers+responses)is completely different from the rest? Just thinking out loud..

Masterz69 commented 2 years ago

As far I see around: lot of alarm/security solutions include environmental sensors inside.

Fire/smoke sensors/alarms included in home security solutions approx. always. My old DSC panel have fire sensors, as well modern revisions allow to add water leak sensors.

Solutions like Xiaomi, Samsung & Hubitat Elevation as well include all threats in Home Security.

IMHO see no dramatic difference. Just sensors, which armed always - just missing possibility to react differently.

"added value" is more happy Alarmo users :)

nielsfaber commented 2 years ago

I understand your point. The main issue I see currently is with assigning different delay times. Alarmo is configured with exit/entry/trigger times per arming mode, so for away/home/night etc., I guess you are aware of that.

In case the system is in armed away an environmental sensor can be triggered. The question is whether it is appropriate to use:

Ofcourse also an environmental sensor can be triggered while the system is disarmed. Here I think the best (and only option is:

What do you think about this?

Masterz69 commented 2 years ago

As far I see in known to me implementations for "Armed Always" aka environmental sensors: Status = always Enabled; Exit & Entry delay non-applicable as always Enabled; Only Trigger Time is applicable.

Just triggered by water leak sensor. As I see - it stay triggered till manual (or by sctipts etc.) disarm. As for me - it is completely fine. Possible bonus - configurable Trigger Time (slider + infinite checkbox) for "Disarmed" mode.

Only missing right now - notifications in "Disarmed" mode. Tried to create notification & remove all Modes from Condition. Actually in that case notification working also in Disarmed mode. But when open notification to edit again - all modes appears.

Probably just add "Disarmed" in Actions Mode filter and fix notifications with no modes specified.

nielsfaber commented 2 years ago

Let's follow-up on this feature in the discussion I made for this: https://github.com/nielsfaber/alarmo/discussions/496