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

GLES2: Make Nvidia flicker workaround opt-in #24987

Merged

Conversation

akien-mga
Copy link
Member

It has a big impact on 2D and text rendering performance (cf. #24466)
so the solution seems worse than the bug it aims to work around.

It's now opt-in via "rendering/quality/2d/gles2_use_nvidia_rect_flicker_workaround"
for those who need it and have a simple enough game for the performance
drop not to be an issue.

Fixes #24466.

@akien-mga
Copy link
Member Author

Testing again the performance in the 2d/platformer demo with this PR (see #24466 (comment) for previous values and testing environment):

  • Before this PR: 372 FPS (366 to 377)
  • rendering/quality/2d/gles2_use_nvidia_rect_flicker_workaround off (default): 554 FPS (513 to 566)
  • rendering/quality/2d/gles2_use_nvidia_rect_flicker_workaround on: 370 FPS (360 to 375)

It has a big impact on 2D and text rendering performance (cf. godotengine#24466)
so the solution seems worse than the bug it aims to work around.

It's now opt-in via "rendering/quality/2d/gles2_use_nvidia_rect_flicker_workaround"
for those who need it and have a simple enough game for the performance
drop not to be an issue.

Fixes godotengine#24466.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant