Skip to content

spec_v2: Auto Exposure #20125

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

spec_v2: Auto Exposure #20125

wants to merge 13 commits into from

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.

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
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants