Skip to content

Introduce RenderGraphSystems#22958

Open
IceSentry wants to merge 1 commit intobevyengine:mainfrom
IceSentry:render_graph_system_set
Open

Introduce RenderGraphSystems#22958
IceSentry wants to merge 1 commit intobevyengine:mainfrom
IceSentry:render_graph_system_set

Conversation

@IceSentry
Copy link
Contributor

Objective

  • It's sometimes useful to order things relative to the RenderGraph

Solution

  • Introduce RenderGraphSytems that contains all the important phases of the RenderGraph execution

Testing

  • I tested the 3d_scene and it worked

Note

This is needed to fix diagnostic recording of gpu timestamps.

@IceSentry IceSentry added A-Rendering Drawing game state to the screen C-Code-Quality A section of code that is hard to understand or change C-Usability A targeted quality-of-life change that makes Bevy easier to use S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Feb 14, 2026
@github-project-automation github-project-automation bot moved this to Needs SME Triage in Rendering Feb 14, 2026
),
);

render_app.add_schedule(RenderGraph::base_schedule());
Copy link
Contributor

Choose a reason for hiding this comment

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

There's a small footgun here: if any of the plugins that are added above insert systems into this schedule they will be removed, because this overwrites the implicit schedule that they create. That's the reason Render::base_schedule() is currently added inside ExtractPlugin.

I made add_schedule print a warning though, and we're planning to move ExtractPlugin to bevy_extract, when that happens we'll have a better place to put this. So it might be fine for now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I wasn't totally sure where to put it. For now it's fine because they aren't used anywhere else and the only place I'm planning on adding it is also fine with this limitation.

@IceSentry IceSentry added S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Feb 15, 2026
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 C-Code-Quality A section of code that is hard to understand or change C-Usability A targeted quality-of-life change that makes Bevy easier to use S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

Status: Needs SME Triage

Development

Successfully merging this pull request may close these issues.

4 participants