Skip to content

Add a FPS limiter - #4091

Open
RobCubed wants to merge 3 commits into
keldaanCommunity:masterfrom
RobCubed:fps-limit
Open

Add a FPS limiter#4091
RobCubed wants to merge 3 commits into
keldaanCommunity:masterfrom
RobCubed:fps-limit

Conversation

@RobCubed

Copy link
Copy Markdown

My laptop is a bit old and tries to light itself on fire over the course of a single PAC round. Did some profiling and found that simply limiting the FPS brings it from being on fire to merely smoldering, which is a significant improvement.

Tried to keep the UI to established elements since I'm new to the codebase.

@ql1111

ql1111 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Wouldn't limiting to 60hz be better for older devices and tablet/chromebooks? Those are usually C a pped at 60hz.

Very good idea though, I'll do some measurements.

@RobCubed

Copy link
Copy Markdown
Author

Wouldn't limiting to 60hz be better for older devices and tablet/chromebooks? Those are usually C a pped at 60hz.

Very good idea though, I'll do some measurements.

It's adjustable in the settings - I just set 144 as the default to largely mimic the existing behavior. I was actually using 10-15 FPS on my laptop while testing to keep the CPU not at 100%

@sylvainpolletvillard

sylvainpolletvillard commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

@RobCubed Thanks for your contribution. Can you add a no limit option, just like it is currently ? The "0" value can be used for that, see https://docs.phaser.io/api-documentation/typedef/types-core#fpsconfig

@RobCubed

RobCubed commented Sep 6, 2026

Copy link
Copy Markdown
Author

@RobCubed Thanks for your contribution. Can you add a no limit option, just like it is currently ? The "0" value can be used for that, see https://docs.phaser.io/api-documentation/typedef/types-core#fpsconfig

Done

@sylvainpolletvillard

Copy link
Copy Markdown
Collaborator

Good to merge for me 👍

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

Successfully merging this pull request may close these issues.

3 participants