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 option 'arm after closing' has no effect #901

Closed
erkr opened this issue Mar 3, 2024 · 9 comments
Closed

Sensor option 'arm after closing' has no effect #901

erkr opened this issue Mar 3, 2024 · 9 comments
Labels

Comments

@erkr
Copy link

erkr commented Mar 3, 2024

Alarmo Version

V1.9.14

HA Version

2024.2.4

Bug description

Thanks for this great integration!
I found this advanced sensor option:
image
when I select the option arm after closing it seems to have no effect. Alarm won’t trigger when it was closed and then opened after arming

Steps to reproduce

“Bypass automatically” is off (not tried while on)
“Arm after closing” is on
“Allow open initially” is off

1: In the card I check the option “bypass open sensors” ( 3dots menu)
2: Then I arm the alarm
3: close the bypassed door sensor
4: open the door —> no alarm triggered!!!

Relevant log output

No response

@erkr erkr added the bug Something isn't working label Mar 3, 2024
@nielsfaber
Copy link
Owner

I use this feature on a daily basis in my own house, it definitely works for me.
I doný really understand why you combine it with bypassing sensors, I suggest to try without that.

@erkr
Copy link
Author

erkr commented Mar 6, 2024

Hi @nielsfaber

I reported the issue, as it seems that this specific combination of not configuring allow_open_initially in the sensor settings and selection the bypass_open option in the card when arming, is what doesn't work.

My reasoning not to
enable allow_open_initially as I want to explicitly decide to bypass them when arming (not accidental because they are bypassed automatically).

Copy link

github-actions bot commented Apr 6, 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 Apr 6, 2024
@erkr
Copy link
Author

erkr commented Apr 6, 2024

I still think it's a bug

@github-actions github-actions bot removed the Stale label Apr 7, 2024
@nielsfaber
Copy link
Owner

The reported issue is not clear to me.
Your report contains a combination of functions 'arm after closing', 'bypass' and 'allow initial open', it is very confusing since these are different features and should not be mixed.

Claim is made (the title states) that the function 'arm after closing' does not work.

when I select the option arm after closing it seems to have no effect. Alarm won’t trigger when it was closed and then opened after arming

I disagree on this, see video of a working example:
https://github.com/nielsfaber/alarmo/assets/16861605/c456230c-de05-468c-9078-e4da8bb544dd

Regarding the steps to reproduce

1: In the card I check the option “bypass open sensors” ( 3dots menu)
2: Then I arm the alarm
3: close the bypassed door sensor
4: open the door —> no alarm triggered!!!

Sensor was bypassed in step 1+2 (assuming the door was open in step 2). Result of step 4 is to be expected. Bypassed sensors have no response.

These steps/observations are also different and unrelated to what is claimed initially. Is the bug about 'arm after closing' not working or about a bypassed sensor not triggering the alarm?

I would like to ask you to update your report, clearly stating the issue, preferably with a proof (for example by video) and reproduction steps.

@erkr
Copy link
Author

erkr commented Apr 12, 2024

Hi Niels,

You mentioned:

these are different features and should not be mixed

So I take that I probably don't fully grasp the concepts, and there is also no restriction in the configuration UI safeguarding me from selecting unintended combinations.

So maybe it is more clear if I try to explain my use case:

I don't want to enable allow_open_initially
for some sensors as I want to deliberately decide to bypass them with bypass_open when Arming.

But once they are bypassed, it would be nice to re-arm these sensors when closed. That is why I enabled the option arm after closing

I now understand that arm after closing only works when enabling allow_open_initially. Right??

If that is the case, it is indeed not a bug, and my use case would be a new feature.

But I believe the UI can guide the novice user a little better with these options. Just some extra text to explain or more fancy gray out arm after closing when not enabling allow_open_initially ☺️

@nielsfaber
Copy link
Owner

Allow_open_initially is intended for windows.
When it is set, you can leave a window open when arming the alarm, if you later close it, it will be watched (opening again would trigger the alarm).
The behavior is similar to bypassing, only bypassing is permanent until disarming, so sensor will not be watched at all.

The arm_on_close feature is intended for doors. It should not be combined with bypassing or the allow_open_initially function, since they are contradictive.
with the feature enabled, you can open+close the front door and the alarm will become armed (instead of waiting until the exit delay has completed).

I believe that if you set the sensor type correctly, the options that are not suitable will become hidden.

@erkr
Copy link
Author

erkr commented Apr 12, 2024

Now I finally understand the options. Do you want to keep this issue open to add some more guidance in the UI? I now understand and it is working as intended.

This explanation would have helped me a lot ☺️

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

2 participants