Skip to content

Add RenderScheduleOrder resource to allow for RenderApp Schedule changes#23513

Merged
alice-i-cecile merged 6 commits intobevyengine:mainfrom
Zeophlite:pre-render-startup
Mar 28, 2026
Merged

Add RenderScheduleOrder resource to allow for RenderApp Schedule changes#23513
alice-i-cecile merged 6 commits intobevyengine:mainfrom
Zeophlite:pre-render-startup

Conversation

@Zeophlite
Copy link
Copy Markdown
Contributor

@Zeophlite Zeophlite commented Mar 25, 2026

Objective

  • Render schedules are static, compared to the main schedules in the main world that can be dynamically added with the MainScheduleOrder resource
  • This is necessary to allow BRP introspection of schedules in the Render world (see BRP Add schedule.graph endpoint #23452 )

Solution

  • Add RenderScheduleOrder resource that allows for ordering of Render schedules

Testing

@Zeophlite Zeophlite requested a review from atlv24 March 25, 2026 14:30
@Zeophlite Zeophlite added the A-Rendering Drawing game state to the screen label Mar 25, 2026
@github-project-automation github-project-automation bot moved this to Needs SME Triage in Rendering Mar 25, 2026
@Zeophlite Zeophlite added D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Mar 25, 2026
@Zeophlite Zeophlite force-pushed the pre-render-startup branch 2 times, most recently from 4530c9f to 5560bd6 Compare March 25, 2026 14:44
@Zeophlite Zeophlite force-pushed the pre-render-startup branch from 5560bd6 to fdf7e9d Compare March 25, 2026 15:24
@alice-i-cecile
Copy link
Copy Markdown
Member

I'm not sure I understand the rationale well enough here. Can you restate the problem we're trying to solve for me using more words?

@atlv24
Copy link
Copy Markdown
Contributor

atlv24 commented Mar 25, 2026

So, I don't think this is the right approach. You can do this in a renderstartup system: just check if your resource is already initialized, and only do so if it hasn't been. The render world is never wiped, so you can store that state downstream and we don't need a RenderStartup change to accomodate this. Am i missing something?

@alice-i-cecile alice-i-cecile added S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged and removed S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Mar 25, 2026
@Zeophlite
Copy link
Copy Markdown
Contributor Author

Hmm okay I can be convinced on that, but I still need the RenderMainScheduleOrder to schedule things outside the main Render schedule (that way things like #23452 work) - will remove PreRenderStartup

@Zeophlite Zeophlite changed the title Add PreRenderStartup schedule and RenderMainScheduleOrder resource Add RenderMainScheduleOrder resource to allow for RenderApp Schedule changes Mar 25, 2026
@Zeophlite
Copy link
Copy Markdown
Contributor Author

Okay so this is much simpler now - this follows the same as the main world Main schedule phases (see MainScheduleOrder )

@Zeophlite Zeophlite added S-Needs-Review Needs reviewer attention (from anyone!) to move forward and removed S-Waiting-on-Author The author needs to make changes or address concerns before this can be merged labels Mar 26, 2026
@kfc35 kfc35 self-requested a review March 26, 2026 21:31
@Zeophlite Zeophlite requested review from atlv24 and removed request for atlv24 March 27, 2026 00:04
Copy link
Copy Markdown
Contributor

@kfc35 kfc35 left a comment

Choose a reason for hiding this comment

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

I don’t have any blocking comments

@alice-i-cecile
Copy link
Copy Markdown
Member

I like the new direction much better here.

@Zeophlite Zeophlite changed the title Add RenderMainScheduleOrder resource to allow for RenderApp Schedule changes Add RenderScheduleOrder resource to allow for RenderApp Schedule changes Mar 27, 2026
@Zeophlite Zeophlite 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 Mar 27, 2026
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Mar 28, 2026
}

fn run_render_schedule(world: &mut World) {
world.run_schedule(Render);
Copy link
Copy Markdown
Contributor

@BenjaminBrienen BenjaminBrienen Mar 28, 2026

Choose a reason for hiding this comment

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

nit: The name of this function should probably be updated

Merged via the queue into bevyengine:main with commit 61a294d Mar 28, 2026
40 checks passed
@github-project-automation github-project-automation bot moved this from Needs SME Triage to Done in Rendering Mar 28, 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 D-Straightforward Simple bug fixes and API improvements, docs, test and examples 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.

5 participants