Skip to content

Add PrepareResourcesBatchPhases RenderSystem set#22952

Open
atlv24 wants to merge 2 commits intobevyengine:mainfrom
atlv24:ad/render-ambiguities-4
Open

Add PrepareResourcesBatchPhases RenderSystem set#22952
atlv24 wants to merge 2 commits intobevyengine:mainfrom
atlv24:ad/render-ambiguities-4

Conversation

@atlv24
Copy link
Contributor

@atlv24 atlv24 commented Feb 14, 2026

Objective

  • reduce RenderApp ambiguities

Solution

  • give batch phases their own system set to order them after skin mesh preparation (which lives in PrepareResources, and matches the GetBatchData param query)
  • also make atmosphere ordering a bit more specific

Testing

  • ran some examples. atmosphere looks good

@atlv24 atlv24 added A-Rendering Drawing game state to the screen 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
/// Collect phase buffers after
/// [`PrepareResources`](RenderSystems::PrepareResources) has run.
/// A sub-set within [`Prepare`](RenderSystems::Prepare) that creates batches for render phases.
PrepareResourcesBatchPhases,
Copy link
Contributor

Choose a reason for hiding this comment

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

Doesn't that reduce parallelism since now other systems that don't care about batch phases can't run in parallel? I guess both system sets already have plenty of systems in them so it might not matter at all.

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 im not sure itll be a measurable regression, lets get to ambiguity zero and then see about refining ordering to get parallelism opportunities back imo

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 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.

2 participants