Skip to content
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

Fix post processing example to only run effect on camera with settings component #10560

Merged

Conversation

IceSentry
Copy link
Contributor

Objective

  • The example says it will only run on a camera with the PostProcessingSettings component but the node never filters it.

Solution

  • Add the component to the ViewQuery

closes: #10541

Copy link

@EmbersArc EmbersArc left a comment

Choose a reason for hiding this comment

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

Works! Thanks a lot for the quick response.

@alice-i-cecile alice-i-cecile added A-Rendering Drawing game state to the screen C-Examples An addition or correction to our examples labels Nov 14, 2023
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Nov 14, 2023
@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Nov 14, 2023
Merged via the queue into bevyengine:main with commit b1fd9ed Nov 14, 2023
25 checks passed
@IceSentry IceSentry deleted the post-process-example-single-camera branch November 14, 2023 23:29
rdrpenguin04 pushed a commit to rdrpenguin04/bevy that referenced this pull request Jan 9, 2024
…s component (bevyengine#10560)

# Objective

- The example says it will only run on a camera with the
`PostProcessingSettings` component but the node never filters it.

## Solution

- Add the component to the `ViewQuery`

closes: bevyengine#10541
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-Examples An addition or correction to our 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
None yet
Development

Successfully merging this pull request may close these issues.

Post processing effect is applied to all cameras
3 participants