Change default custom refresh rate value from 200% to 100% #691
Loading…
x
Reference in New Issue
Block a user
No description provided.
Delete Branch "custom-refresh-rate"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
This changes the default custom refresh rate value from 200% to 100%. Config will migrate from v67 to v68.
Why? This is identical to just using normal VSync, it makes much more sense to swap to VSync on then to make the custom refresh rate pointless
I think 100% should be the base value for the custom refresh rate. You can always increase it from there. With many games, if you enable it and leave it at 200%, the game will run at 2x speed. Depends on the engine, of course, but setting it to the default VSync value will minimize potential issues.
Well yes but this is the entire point. It's in the hack section after all. If people enable it and get issues that's kinda on them for enabling something in the "hacks" section that has a note that it can cause issues
I just feel like this change doesn't make much sense for 2 reasons:
Download the artifacts for this pull request:
Only for Developers
Understandable, just thought it would be cleaner. Do you think a value between 100% and 200% would make more sense?
I honestly think 200 makes perfect sense. Most of the games are 30 base and 60 is desirable.
This option is very niche as is; and only really grants a benefit in a specific case:
The game you are playing is not FPS-sensitive. Either it was made that way, or has a mod to make it that way.
When it comes to games like that, 200% base makes sense, since you can use VSync for 30, custom refresh rate for 60-120, and VSync off to just let it rip.
Leaving custom refresh rate at 100% default seems odd since the entire point of the setting is that it's custom and is a way to achieve a framerate target you couldn't previously without directly modding the game to request a specific refresh rate target.
I see what you mean. I also made this change in my fork, but you made some really good points. Closing the PR.
Pull request closed