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

WiimoteReal: Only duplicate data reports when speaker data is enabled. #9098

Merged
merged 1 commit into from Oct 23, 2020

Conversation

jordan-woyak
Copy link
Member

Currently, data reports of real remotes are repeated to trigger sending of speaker data.

This causes some games to fail at detecting motion gestures.
https://bugs.dolphin-emu.org/issues/10111
https://bugs.dolphin-emu.org/issues/12030

With this change if speaker data is disabled so is repeating of data reports.
This fixes gestures in these games so long as speaker data is disabled.

Another potential fix might be to interpolate accelerometer data in repeated reports but that is getting pretty hacky.
I think this is cleaner.

@JMC47
Copy link
Contributor

JMC47 commented Sep 24, 2020

Everything in this makes sense to me. I quickly tested Sonic and the Secret Rings and it seemed to be fine with Speaker Data off.

Question? Does Bluetooth Passthrough get into sniff mode though? If it does, why is audio data so iffy still?

@jordan-woyak
Copy link
Member Author

Question? Does Bluetooth Passthrough get into sniff mode though? If it does, why is audio data so iffy still?

Yes, "sniff" mode is set in Bluetooth Passthrough where Wii remote audio works flawlessly here.

@leoetlino leoetlino merged commit e89db89 into dolphin-emu:master Oct 23, 2020
10 checks passed
@cobalt2727
Copy link

I've got a report of this causing severe input delay for the pointer and nothing else, oddly enough.
https://discord.com/channels/521709831132807179/521711075721478145/787156878944894986
(you have to join the unofficial Dolphin Discord or the link won't work)

@jordan-woyak jordan-woyak deleted the wm-report-duplication branch December 30, 2020 02:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants