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

Handling of tamper sensors #586

Closed
2 tasks done
segdy opened this issue Oct 1, 2022 · 2 comments
Closed
2 tasks done

Handling of tamper sensors #586

segdy opened this issue Oct 1, 2022 · 2 comments

Comments

@segdy
Copy link

segdy commented Oct 1, 2022

This is based on the discussion in https://community.home-assistant.io/t/custom-component-alarmo-browser-managed-alarm-system/238910/916?

Checklist

  • I checked for similar existing requests (both open and closed) before posting.
  • My request is generic, other users may benefit from it too.

Proposal

As proposed in the forum discussion, best would be to have a separate "zone" similar to "24h". When adding a tamper sensor (or smoke/gas etc) they should automatically be added to this zone.
This zone is always active (regardless of arming status) but can temporarily be disabled during maintenance (battery change etc)

@nielsfaber
Copy link
Owner

Thanks for making the request.
Alarmo uses areas instead of zones.
However, most users don't use this functionality (all their sensors are in a single area).
I think we should focus on a solution where users can easily add tamper sensors without the need for splitting their alarm in multiple areas.

The default behaviour for a tamper sensor is defined in this table. In summary, the sensor will be active whilst the alarm is armed, and trigger the alarm instantly (no delay).

The difference with your request is that you want the sensor to be always active, except when performing maintenance.
I think there a two options to accomplish this:

  1. In Alarmo there is an 'always on' setting which can be applied to a sensor. When set, the sensor can bring the alarm to triggered state even when the alarm is disarmed. There is no option for temporary disabling for maintenance, the only way would be to go into the sensors tab in Alarmo, and use the toggle switches here to disable each manually.
  2. A user can create an area for all tamper sensors. It will be a separate alarm entity which can be armed and disarmed independently (read here). The user can keep this area in armed state at all times except when performing maintenance. The user can make use of the alarm master to catch a trigger event of a sensor in the "24h" area or the regular area. combined

Both options are already possible to set up without changes in Alarmo, and I think choosing for either depends on the preference of the user. I have my doubts whether any of these should be a default setting for a newly added tamper sensor.
Personally I don't have a desire to have the alarm trigger on a tamper sensor while the alarm is disarmed.

Please share your thoughts on this.

@segdy
Copy link
Author

segdy commented Oct 1, 2022

Hi Niels, this makes sense to me and as indicated in the forum thread (https://community.home-assistant.io/t/custom-component-alarmo-browser-managed-alarm-system/238910/918 and https://community.home-assistant.io/t/custom-component-alarmo-browser-managed-alarm-system/238910/919) this is likely a bug.

In the table you quoted, tamper maps to device classes sound, opening, vibration. However there is a device class tamper itself (Zigbee contact sensors expose this device class for example). So maybe the solution is as easy as adding "tamper" to this list?

So now, this is likely not a feature request any more.
I created a bug report #587 and closing this one then.

Thanks!

@segdy segdy closed this as completed Oct 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants