Skip to content

spec_v2: Auto Exposure#20125

Closed
ecoskey wants to merge 13 commits intobevyengine:mainfrom
ecoskey:specv2_exposure
Closed

spec_v2: Auto Exposure#20125
ecoskey wants to merge 13 commits intobevyengine:mainfrom
ecoskey:specv2_exposure

Conversation

@ecoskey
Copy link
Contributor

@ecoskey ecoskey commented Jul 14, 2025

Objective

#17373
#19969

  • Migrate auto_exposure to spec_v2

Solution

  • Did the things

Testing

  • ran auto_exposure

@ecoskey ecoskey mentioned this pull request Jul 14, 2025
43 tasks
@alice-i-cecile alice-i-cecile added A-Rendering Drawing game state to the screen C-Code-Quality A section of code that is hard to understand or change X-Contentious There are nontrivial implications that should be thought through S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Jul 14, 2025
@andriyDev
Copy link
Contributor

@ecoskey, #20002 changed auto exposure to use systems instead of FromWorld. Can you rebase this PR on that?

P.S., if you're doing more of these migrations, could you make sure you're using RenderStartup instead of FromWorld + initializing in finish?

@andriyDev andriyDev 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 Jul 16, 2025
@ecoskey
Copy link
Contributor Author

ecoskey commented Jul 17, 2025

P.S., if you're doing more of these migrations, could you make sure you're using RenderStartup instead of FromWorld + initializing in finish?

Will do, sorry bout that! I'd probably rather do migrate RenderStartup -> migrate spec_v2 in separate PRs so the diffs are smaller, what's left on the list for RenderStartup?

@andriyDev
Copy link
Contributor

No problem at all haha, this is gonna be a battle keeping devs from using finish and instead forcing them to use RenderStartup...

There are 2 cases of finish left in bevy_core_pipeline, a bunch of cases in bevy_pbr, a bunch of cases in bevy_render, some in bevy_solari, and one in bevy_gltf. I don't know how many of these are pipelines we need to convert to spec v2, I still need to work through them. The main blocker is agreeing on a pattern for conditional systems: previously, we would conditionally add systems based on whether some features were supported. Since we are now doing stuff in RenderStartup we can't do this conditional addition of systems, so we need to conditionally run systems, or something like that (proposed in #19918).

Hopefully RenderStartup should be mostly out of your way for spec v2 migrations.

@ecoskey ecoskey closed this Jul 31, 2025
@ecoskey ecoskey deleted the specv2_exposure branch July 31, 2025 05:05
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-Waiting-on-Author The author needs to make changes or address concerns before this can be merged X-Contentious There are nontrivial implications that should be thought through

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants