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

Lower debounce time #714

Merged
merged 1 commit into from
Sep 8, 2024
Merged

Lower debounce time #714

merged 1 commit into from
Sep 8, 2024

Conversation

probonopd
Copy link
Owner

Copy link

github-actions bot commented Sep 1, 2024

Build for testing:
MiniDexed_2024-09-01-c92185d
Use at your own risk.

@miotislucifugis
Copy link

miotislucifugis commented Sep 1, 2024

Wow, Thanks! This seems to be working great. Even the fastest, lightest taps get registered. However, Im finding that if i do a couple of fast successive taps, like 3 quick ones in a row to go back 3 steps, only only one press gets identified....but I guess that this is a limit of the double tap timeout.

@probonopd
Copy link
Owner Author

@diyelectromusic agree to merge this? See @miotislucifugis last comment in #713.

At this time, I'd rather not touch BUTTONS_UPDATE_NUM_TICKS because it might impact overall performance.

@diyelectromusic
Copy link
Contributor

It's a little hard to say really, but yes, that last comment is encouraging.

I do think we should wait until someone tests this with other buttons though. We can't be sure we haven't introduced a debouncing issue with slower/noisier buttons, so I think others ought to give it a bit of a test first.

If I get a chance, I'll load it up onto my various builds and see what it does, but that probably won't be for a few days and all my builds include capacitors for some hardware debouncing so might not be representative anyway.

It would be nice to have some proper profiling of the timers/responses involved in buttons and their scanning to put some sense to those numbers, but that would need an instrumented (e.g. with oscilloscope/GPIO outputs) build to be tested which would take a fair bit of time, so I guess we just go with the "is anyone shouting this is broken" approach for now :)

Kevin

@miotislucifugis
Copy link

miotislucifugis commented Sep 3, 2024

I can test that tomorrow. My key switches are on a daughter board and I have an other board that I can swap in that has the standard little pcb push buttons. For what its worth, I currently see no issue w/ the encoder switch.

@miotislucifugis
Copy link

Regular switches appear to be working fine in this build... at least for me

@probonopd
Copy link
Owner Author

Merging. Please comment here if you experience any button related issues.
Thanks!

@probonopd probonopd merged commit 6c56960 into main Sep 8, 2024
1 check passed
@probonopd probonopd deleted the lower-debounce-time branch September 8, 2024 11:58
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

Successfully merging this pull request may close these issues.

3 participants