Skip to content

Upstream bevy_framepace - #26392

Open
IceSentry wants to merge 5 commits into
bevyengine:mainfrom
IceSentry:upstream_bevy_framepace
Open

IceSentry wants to merge 5 commits into
bevyengine:mainfrom
IceSentry:upstream_bevy_framepace

Conversation

@IceSentry

@IceSentry IceSentry commented Oct 31, 2026 •

Copy link
Copy Markdown
Contributor

Objective

  • bevy_framepace has been used in production by many bevy users for a few years. It's a useful feature that could benefit from a more complete integration in the engine.

Solution

  • Upstream the crate almost as-is with some very minor changes.
    • The monitor refresh rate is now read from bevy_window instead of bevy_winit so this will work with any window backend.
  • The crate now lives as a module inside bevy_render

Testing

  • I tested the new example and it behaves like the one from the original crate

Notes

There are various improvements that could be done to this crate, but I believe this should be done as follow up PRs.

@IceSentry IceSentry added C-Feature A new feature, making something new possible A-Rendering Drawing game state to the screen A-Windowing Platform-agnostic interface layer to run your app in S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Oct 31, 2026
@github-project-automation github-project-automation Bot moved this to Needs SME Triage in Rendering Oct 31, 2026
@Zeophlite Zeophlite added the M-Release-Note Work that should be called out in the blog due to impact label Oct 31, 2026
Comment on lines +21 to +25
//! same same
//! /-- latency --\ /-- latency --\
//! sleep -> input -> render -> sleeeeeeeeeeeeeeeeeeeeeeeep -> input -> render
//! \----- event loop -----/ \---------------- event loop ----------------/
//! 60 fps limited to 10 fps

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This is fun (especially the sleeeep part, love it!) :D , I think a table would be a better way to format this information.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That was copied from the original crate. I'd prefer keeping changes to a minimum in this PR.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Rendering Drawing game state to the screen A-Windowing Platform-agnostic interface layer to run your app in C-Feature A new feature, making something new possible M-Release-Note Work that should be called out in the blog due to impact S-Needs-Review Needs reviewer attention (from anyone!) to move forward

Projects

Status: Needs SME Triage

Development

Successfully merging this pull request may close these issues.

3 participants