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

quick multiple presses not registered #26

Open
csiess85 opened this issue Feb 24, 2023 · 2 comments
Open

quick multiple presses not registered #26

csiess85 opened this issue Feb 24, 2023 · 2 comments

Comments

@csiess85
Copy link

I just realised pressing the buttons very fast ignores the first button press. I have not debugged this, but guess its due to the debounce functionality. Having the possibility to reduce the debounce timeout via mqtt would be great.

@nplan
Copy link
Owner

nplan commented Feb 24, 2023

The issues is not the debounce timeout.

If ESP32 is in deep sleep, it's woken up on the first press. Then it has to boot up before the second press happens. Boot takes about 60 ms, so that's the limit on how fast in can register multiple presses from deep sleep state. If you put it in awake mode (plug in the USB and turn on Awake mode switch in Home Assistant) it will register shorter multiple presses.

This could be improved by using the wake up stub functionality of the ESP32 but it's not a priority at the moment.

@MatthiasLohr
Copy link

Since also stumbled across this (see #9), some improvement here would be nice.

@nplan nplan changed the title Make DEBOUNCE_TIMEOUT configurable quick multiple presses not registered Feb 24, 2023
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

3 participants