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

Implement a rumble delay pool #253

Closed
wants to merge 2 commits into from
Closed

Conversation

kakra
Copy link
Collaborator

@kakra kakra commented Nov 8, 2020

Research showed that there are users who only experience the controller crash with longer bursts of rumble. This PR tries to implement a delay pool to work around bursts.

My personal tests show no positive effect, tho.

Affects: #243
Affects: #247
Maybe-related: #198
See-also: https://answers.microsoft.com/en-us/xbox/forum/all/xbox-elite-series-2-disconnecting-from-pc/7e5a964a-16cc-4e83-8dec-aa8c858b875c

hid-xpadneo/src/hid-xpadneo.c Outdated Show resolved Hide resolved
hid-xpadneo/src/hid-xpadneo.c Outdated Show resolved Hide resolved
@kakra kakra force-pushed the rumble-delay-pool branch 4 times, most recently from e8092ed to cc660d2 Compare November 9, 2020 00:21
@kakra kakra force-pushed the rumble-delay-pool branch 10 times, most recently from 4b67969 to f2bbb7b Compare November 17, 2020 21:10
@codeclimate
Copy link

codeclimate bot commented Nov 22, 2020

Code Climate has analyzed commit dd49347 and detected 0 issues on this pull request.

View more on Code Climate.

@kakra
Copy link
Collaborator Author

kakra commented Nov 22, 2020

This doesn't help stability and only increases complexity and rumble lagging. Closing.

@kakra kakra closed this Nov 22, 2020
@kakra kakra deleted the rumble-delay-pool branch November 28, 2020 12:57
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