Skip to content

Add PrepareResourcesBatchPhases RenderSystem set#22952

Merged
alice-i-cecile merged 2 commits intobevyengine:mainfrom
atlv24:ad/render-ambiguities-4
Feb 17, 2026
Merged

Add PrepareResourcesBatchPhases RenderSystem set#22952
alice-i-cecile merged 2 commits intobevyengine:mainfrom
atlv24:ad/render-ambiguities-4

Conversation

@atlv24
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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

@alice-i-cecile alice-i-cecile added C-Code-Quality A section of code that is hard to understand or change 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 16, 2026
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Feb 17, 2026
Merged via the queue into bevyengine:main with commit 8edf483 Feb 17, 2026
46 checks passed
@github-project-automation github-project-automation Bot moved this from Needs SME Triage to Done in Rendering Feb 17, 2026
github-merge-queue Bot pushed a commit that referenced this pull request Feb 17, 2026
# Objective

- Fix #7386

## Solution

- #22951 #22952 #22949 #22945 and a handful more changes on this PR.
NOTE: Ci will not pass until all these other prs are merged.
- Fix a bug

## Testing

- examples look fine

---------

Co-authored-by: Alice Cecile <alice.i.cecile@gmail.com>
Co-authored-by: François Mockers <francois.mockers@vleue.com>
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 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: Done

Development

Successfully merging this pull request may close these issues.

3 participants