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

Display: Allow unthrottle to skip only flipping #12970

Closed
wants to merge 2 commits into from

Conversation

unknownbrackets
Copy link
Collaborator

Before, it either flipped continuously, or forced frameskip on. This makes it so you can still draw frames, but skip actual flips.

This is useful when games draw things only in a single frame and reuse later. It's also useful when measuring speed improvements if you already get 100% speed on a device.

This resets the previous FrameskipUnthrottle setting. That said, this may be a good thing because some users changed it when vsync/postshaders were not interacting as well with it, and I think those problems are fixed.

-[Unknown]

@hrydgard
Copy link
Owner

hrydgard commented May 25, 2020

Yay, I've wanted this for a long time. On the fence whether to merge it for 1.10...

@hrydgard
Copy link
Owner

Gonna need a little rebase here, sorry :)

Before, it either flipped continuously, or forced frameskip on.  This
makes it so you can still draw frames, but skip actual flips.

This is useful when games draw things only in a single frame and reuse
later.  It's also useful when measuring speed improvements if you already
get 100% speed on a device.
Three spaces is weird, but better to be consistent and have the config
specify what's being used.
@hrydgard
Copy link
Owner

There was another conflict (caused by my latest changes) so I merged locally. Closing.

@hrydgard hrydgard closed this Jul 19, 2020
@unknownbrackets unknownbrackets deleted the unthrottle branch July 20, 2020 00:10
@unknownbrackets
Copy link
Collaborator Author

We may even want to consider exposing this in the UI or making it default to SKIP_FLIP when frameskip is otherwise off.

-[Unknown]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GE emulation Backend-independent GPU issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants