search

andrewphorn / ClassiCube-Client

The applet used for classicube.net
34 stars 19 forks source link

Limit framerate #293

Closed samwfowler closed 9 years ago

samwfowler commented 9 years ago

I like the fact you can choose the FPS limit, but I think 40FPS as a maximum "limit" is too low! before it would limit to 60 FPS- which is great because that's about the maximum smoothness my eyes can see- but now I have to disable the limit and run at 2500FPS, just so it isn't jerky, which is a bit overkill! Anyway, an additional "60FPS" option would be great, cheers!!

123DMWM commented 9 years ago

144 is the maximum limit I believe

mstefarov commented 9 years ago

Actually, the limiter is supposed to choose from several options -- 20, 30, 40, 60, 75, 85, 120, 144 -- maxing out at your screen's refresh rate. Unfortunately I did not realize that 59Hz is a common refresh rate, which is not quite enough to ClassiCube to choose 60. That will be fixed in the next update!

mstefarov commented 9 years ago

I'm also considering separating vsync and framerate-limit options, which would decouple framerate limits from refresh rate. Maybe.

samwfowler commented 9 years ago

Okay, yeah, when I cycled through it only went from 0 to 40, which would make sense if my screen has a refresh rate of 59 haha, anyway, thanks!

mstefarov commented 9 years ago

Refresh rate limit fixed in ec9cd81cbf18aeb5c824fe053f1fa9e2a703a6ee

It'll be pushed as an update within a few days, after we fix a couple other bugs.