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 default sound latency #58

Closed
wojpawlik opened this issue Sep 20, 2020 · 4 comments
Closed

Lower default sound latency #58

wojpawlik opened this issue Sep 20, 2020 · 4 comments

Comments

@wojpawlik
Copy link

bsnes-hd defaults audio latency to 512, which delays jump sound effect in SMW until Mario is already at full height.

Upstream bsnes_v115-windows also internally defaults to 512, but ships with a config file lowering it to 128.

Unscientific tests on my machine (Windows 2004, bsnes-hd 10.3, Super Mario World, default settings: audio sync, no dynamic rate, no exclusive mode):

Driver No sound artifacts in game No sound artifacts when toggling fullscreen
WASAPI ≥ 0 ≥ 100
XAudio 2.1 ≥ 20 ≥ 80
Direct Sound 7.0 ≥ 40 unachievable
waveOut (default) ≥ 40 ≥ 160

That said, for actually playing on my system, I'd probably use the Dynamic Rate Control preset.

@DerKoun
Copy link
Owner

DerKoun commented Oct 5, 2020

I try to avoid shipping a default config file, especially one differing from the defaults in the code.
I'll look into it, but considering it's not HD functionality, I'm not sure if I want to make changes that might be more dangerous than I expected.

@wojpawlik
Copy link
Author

I'm only advocating for changing the default waveOut latency in the code to 160 or 128 (same as bsnes), to improve first impression. I'm not advocating here for shipping default config file, nor changing the default driver.

@wojpawlik
Copy link
Author

Even though on my system WASAPI allows the lowest latency with audio sync, and somehow produces the least artifacts without.

@DerKoun
Copy link
Owner

DerKoun commented Feb 20, 2021

Changed in new beta 10.4

@DerKoun DerKoun closed this as completed Feb 20, 2021
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

No branches or pull requests

2 participants