Skip to content

Use monitor's refresh rate#885

Draft
burgerindividual wants to merge 2 commits intotoolbox4minecraft:masterfrom
burgerindividual:use-monitor-refresh-rate
Draft

Use monitor's refresh rate#885
burgerindividual wants to merge 2 commits intotoolbox4minecraft:masterfrom
burgerindividual:use-monitor-refresh-rate

Conversation

@burgerindividual
Copy link
Contributor

This sets the refresh rate of the render and queue processing threads to the same as the refresh rate of the best monitor connected.

@burgerindividual burgerindividual marked this pull request as draft October 5, 2020 13:48
@burgerindividual
Copy link
Contributor Author

I marked this as a draft because some things like scrolling speed depend on refresh rate rather than time elapsed, so I'm going to have to fix those before this gets unmarked again.

@moulins
Copy link
Contributor

moulins commented Oct 6, 2020

This is a good idea :)
Maybe the framerate should be capped at a "reasonable" value (like 60Hz or 90Hz), to reduce CPU usage when the monitor has a high framerate, like 144Hz ?

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.

2 participants