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

CoreTiming: Allow forcing seek of accurate overall emulation time. #13392

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jordan-woyak
Copy link
Member

@jordan-woyak jordan-woyak commented Mar 4, 2025

Note this ties in with PR #13387 which should probably be merged first.

Previous Throttle code adjusted time points every call leading to potential rounding errors and accumulated drift over time.
Time points are now adjusted by exact seconds to avoid this.

MAIN_MAX_FALLBACK (default: 100ms) has already existed to reset the throttle if emulation strays too far from the target time, such as from a CPU bottleneck.
If stutters exceeds 100ms before Dolphin can catch up, it will adjust the reference time and effectively commit to running >100ms in the past.

That is not ideal for internet play and a user has reported it as a cause of issue with Wiimmfi.
https://bugs.dolphin-emu.org/issues/13609 (fixed)

The MAIN_MAX_FALLBACK value can now be adjusted in the UI and set to zero to always seek accurate overall emulation time.
image

@jordan-woyak jordan-woyak force-pushed the frame-pacing-accurate-time branch 3 times, most recently from 7a2158a to d5e08ec Compare March 4, 2025 04:59
@mbc07
Copy link
Member

mbc07 commented Mar 4, 2025

Ugh, not directly related to this PR but we really should move all those text descriptions from the advanced tab to tooltips or similar. It's already stupidly tall and will get even taller with the fallback timing option...

@jordan-woyak jordan-woyak force-pushed the frame-pacing-accurate-time branch from d5e08ec to 4e61ce5 Compare March 8, 2025 07:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants