Skip to content

gate bloom on hdr check#23543

Merged
alice-i-cecile merged 1 commit intobevyengine:mainfrom
atlv24:ad/fix-bloom
Mar 28, 2026
Merged

gate bloom on hdr check#23543
alice-i-cecile merged 1 commit intobevyengine:mainfrom
atlv24:ad/fix-bloom

Conversation

@atlv24
Copy link
Copy Markdown
Contributor

@atlv24 atlv24 commented Mar 27, 2026

Objective

Solution

  • Don't run bloom if the view is not HDR

Testing

  • transmission example

@alice-i-cecile alice-i-cecile added C-Bug An unexpected or incorrect behavior A-Rendering Drawing game state to the screen S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Mar 27, 2026
@alice-i-cecile alice-i-cecile added this to the 0.19 milestone Mar 27, 2026
@github-project-automation github-project-automation bot moved this to Needs SME Triage in Rendering Mar 27, 2026
@alice-i-cecile alice-i-cecile added the D-Straightforward Simple bug fixes and API improvements, docs, test and examples label Mar 27, 2026
@IceSentry
Copy link
Copy Markdown
Contributor

I tested the bloom_3d example and it still works as expected

@atlv24
Copy link
Copy Markdown
Contributor Author

atlv24 commented Mar 28, 2026

Fixes #23142

@alice-i-cecile alice-i-cecile added this pull request to the merge queue Mar 28, 2026
@alice-i-cecile alice-i-cecile 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 28, 2026
Merged via the queue into bevyengine:main with commit 9b279b3 Mar 28, 2026
49 checks passed
@github-project-automation github-project-automation bot moved this from Needs SME Triage to Done in Rendering Mar 28, 2026
splo pushed a commit to splo/bevy that referenced this pull request Mar 31, 2026
# Objective

- Fixes bevyengine#23142

## Solution

- Don't run bloom if the view is not HDR

## Testing

- transmission example
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-Bug An unexpected or incorrect behavior 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.

Bloom + transmission bug

3 participants