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

Mention buffer size and Windows 11 mouse polling in Fixing jitter, stutter and input lag #7829

Merged

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Aug 21, 2023

This also documents command line arguments that may be of interest when running Godot projects.

@Calinou Calinou added enhancement area:manual Issues and PRs related to the Manual/Tutorials section of the documentation labels Aug 21, 2023
@mhilbrunner mhilbrunner marked this pull request as draft October 6, 2023 11:07
@mhilbrunner
Copy link
Member

mhilbrunner commented Oct 6, 2023

Put a safety on the big green button for now, seeing the discussion in the PR and that it seems dicey whether it'll be merged for 4.2 for now. Once its merged, this can be too, the docs PR LGTM.

@skyace65 skyace65 added the waiting on PR merge PR's that can't be merged until a engine PR is merged first label Nov 27, 2023
…utter and input lag

This also documents command line arguments that may be of interest when
running Godot projects.
@Calinou Calinou force-pushed the fixing-jitter-stutter-buffer-size branch from 8200af4 to 16c34b2 Compare March 5, 2024 18:04
@Calinou
Copy link
Member Author

Calinou commented Mar 5, 2024

Put a safety on the big green button for now, seeing the discussion in the PR and that it seems dicey whether it'll be merged for 4.2 for now. Once its merged, this can be too, the docs PR LGTM.

The original PR was closed, but godotengine/godot#87340 integrates most of its functionality (except it doesn't allow decreasing frame queue size to 1 like godotengine/godot#80566 did).

I've updated the PR accordingly to only mention swapchain image count, since the default frame queue size value is already the lowest you can use.

@Calinou Calinou marked this pull request as ready for review March 5, 2024 18:37
@mhilbrunner mhilbrunner removed the waiting on PR merge PR's that can't be merged until a engine PR is merged first label Mar 27, 2024
Copy link
Member

@mhilbrunner mhilbrunner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Calinou I guess this can then be merged now? :)

@mhilbrunner mhilbrunner merged commit 36630bc into godotengine:master Aug 14, 2024
1 check passed
@mhilbrunner
Copy link
Member

...merged. Thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:manual Issues and PRs related to the Manual/Tutorials section of the documentation enhancement topic:input
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants