Skip to content

Controller Lab 5/7: configurable button debounce - #14

Open
slkass wants to merge 7 commits into
Brunovncs:mainfrom
slkass:pr/lab-05-button-debounce
Open

slkass wants to merge 7 commits into
Brunovncs:mainfrom
slkass:pr/lab-05-button-debounce

Conversation

@slkass

@slkass slkass commented Nov 13, 2026 •

Copy link
Copy Markdown
Contributor

Add optional per-profile 0-100 ms button debounce for mapping mode. Both press and release must remain stable for the configured interval before virtual gamepad or keyboard output changes. Zero retains the original input latency.

Pending transitions settle on the engine timer without requiring another HID report. Bounces cancel/restart only the affected button's timer; native input remains unchanged. The Lab page exposes the interval and explains its latency tradeoff.

Dependencies and focused review

Stacked PR 5/7; depends on #13 and the earlier layers, starting with #9. All targets are upstream main, so GitHub's Files changed view currently includes unmerged dependencies. Review only this feature's incremental diff (commit eb40a98).

Merge in the order below. Preserve the shared commit ancestry when merging, or rebase the remaining branches after squash/rebase merges so already-merged layers disappear from their diffs.

  1. Controller Lab 1/7: DualSense trigger effects and timed trials #10 — DualSense trigger effects and timed trials
  2. Controller Lab 2/7: touchpad mouse and trackball #11 — touchpad mouse and trackball
  3. Controller Lab 3/7: Flick Stick and gyro mouse aiming #12 — Flick Stick and gyro mouse aiming
  4. Controller Lab 4/7: Hair Trigger, two-stage pulls and Hip Fire #13 — Hair Trigger, two-stage pulls and Hip Fire
  5. Controller Lab 5/7: configurable button debounce #14 — configurable button debounce
  6. Controller Lab 6/7: local DSU motion forwarding #15 — local DSU motion forwarding
  7. Controller Lab 7/7: dynamic lighting and matching touch feedback #16 — dynamic lighting and matching touch feedback

Validation

  • cargo fmt --all --check
  • cargo clippy --workspace --all-targets --release --locked -- -D warnings
  • cargo test --workspace --locked: 270 passed, 5 driver/manual tests ignored, zero failures (Windows x64).
  • Unit tests cover bouncing presses, stable release and timer settlement. The real SDL -> ViGEm -> XInput loopback validates an 80 ms interval and eventual button release.

All new tools are disabled by default. No additional dependencies are added. Cross-platform CI results are reported by GitHub checks.

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.

1 participant