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

Add CPUCull to Pokemon Colosseum and Pokemon XD inis. #11818

Conversation

CasualPokePlayer
Copy link
Contributor

@CasualPokePlayer CasualPokePlayer commented May 7, 2023

These games seem to constantly have unused vertices, and this is worst shown in the Shadow Pokemon purification cutscene. The Shadow Pokemon purification cutscene is even worse on XD with forced single core mode, as instead of having FPS dropping with VPS staying 60ish, it will drop both, resulting in audio stuttering. Turning on CPUCall seems to have a 7/8 reduction of draw calls for that cutscene (~800 -> ~100), doubling performance (~70% max to ~150% max on my setup). Many other areas of the game seem to benefit from this setting too, having some kind of performance boost (although this is best seen in 60 FPS areas as the 30 FPS areas generally don't struggle)

These games seem to constantly have unused vertices, and this is worst shown in the Shadow Pokemon purification cutscene. The Shadow Pokemon purification cutscene is even worse on XD with forced single core mode, as instead of having FPS dropping with VPS staying 60ish, it will drop both, resulting in audio stuttering. Turning on CPUCall seems to have a 7/8 reduction of draw calls for that cutscene (~800 -> ~100), doubling performance. Many other areas of the game seem to benefit from this setting too, having some kind of performance boost.
@AdmiralCurtiss
Copy link
Contributor

I seriously have to wonder what the game is doing to have that many drawn vertices offscreen. I guess since it runs fine on hardware it doesn't really matter, but still.

@AdmiralCurtiss
Copy link
Contributor

Oh lmao that's indeed a massive difference, I get twice the speed with CPU cull in the Agate clearing cutscene.

I guess it's because it's rendering this entire area without culling anything.

Clipboard012

@AdmiralCurtiss AdmiralCurtiss merged commit 99d7fe3 into dolphin-emu:master May 7, 2023
14 checks passed
@CasualPokePlayer CasualPokePlayer deleted the pokemon_cull_vertices_cpu branch May 8, 2023 02:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants