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

audio latency fix #42

Merged
merged 5 commits into from Feb 15, 2014
Merged

audio latency fix #42

merged 5 commits into from Feb 15, 2014

Conversation

degasus
Copy link
Member

@degasus degasus commented Feb 6, 2014

This branch will control the interpolation to reduce the latency as much as possible. The current setting are 50ms, but the need to be tested on different plattforms.

@delroth
Copy link
Member

delroth commented Feb 6, 2014

Could you get some user testing for this? Maybe on forums.

@JMC47
Copy link
Contributor

JMC47 commented Feb 10, 2014

I have seriously tested this in a ton of games and find it absolutely necessary for the future of Dolphin. This makes it mostly comparable to using Zelda u-code HLE in most games latency wise. I did notice Midway Arcade Treasures 2 was oddly unaffected, but, perhaps that game just has shit audio even on console.

I know this does need testing from more people, but I can confirm pulse audio (with the pulse audio rewrite stuff) and xaudio 2 on Windows both have extremely good audio latency (compared to previous builds) with these changes with no known drawbacks.

I tested Mario Superstar Baseball, F-Zero GX, Super Smash Bros. Melee/Brawl, Super Monkey Ball 1/2, Midway Arcade Treasures 2, Wiiplay, Wiisports, Wii Sports Resort among others. I mainly use the fork for playing because the audio latency difference makes a lot of games more enjoyable to me. Once you notice how bad the audio latency is in dolphin, it really bugs you. This alleviates it.

OpenAL itself stretch the time on slowdowns, so the Mixer isn't allowed also to change the rate.
delroth added a commit that referenced this pull request Feb 15, 2014
@delroth delroth merged commit cbe7656 into dolphin-emu:master Feb 15, 2014
@degasus degasus deleted the latencyFix branch February 15, 2014 17:29
rapito pushed a commit to project-lylat/dolphin that referenced this pull request Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants