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

Increase or remove limit for "Reset after" #210

Open
Foxi352 opened this issue Mar 5, 2025 · 1 comment
Open

Increase or remove limit for "Reset after" #210

Foxi352 opened this issue Mar 5, 2025 · 1 comment

Comments

@Foxi352
Copy link

Foxi352 commented Mar 5, 2025

Hey,

Is there any specific reason to limit the "reset after" for binary sensors to a maximum of 10 seconds ?
I have "presence" sensors who are not very good at their job and are more like motion detectors. They just send "1" to the bus when a movement is detected, but never a "0". So in config.yaml i had a "reset after" of 300s which simulated quite reliably a true presence detector.

Sadly, in UI the maximum is 10 seconds which does not work for me.

Any chance to upper that limit to something like 600 seconds ? Or remove the limit entirely ?

Thanks for your work. I really appreciate the KNX UI and i slowly convert everything from config.yaml to UI as soon as new device types are added ...

@farmio
Copy link
Member

farmio commented Mar 6, 2025

Hi 👋!
The maximum of 10 was set in order to be able to show a usable slider selector.
This function was not really meant to be any sort of logic / timer / presence detector thing, but was initially intended to be able to support momentary switches or sensors (those that only ever send 1 - never 0). Meanwhile there are better ways to handle those things - so this is now mostly for backwards compatibility.

I think it would be ok to change it to a bigger upper bound, but then the slider would have to be replaced by a box selector.
If you like and if you are able to do some testing, go ahead and open a PR changing it. It would probably be somewhere here:

selector: { number: { min: 0, max: 10, step: 0.1, unit_of_measurement: "s" } },

Not sure how the box-selector fits into current layout though - especially on mobile.

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

No branches or pull requests

2 participants