Skip to content

Conversation

@Alb-O
Copy link

@Alb-O Alb-O commented Feb 6, 2026

Objective

The F_AB() polynomial approximation produces F_ab.x values approaching zero or slightly negative on highly reflective surfaces viewed at grazing angles, causing fireflies and black pixelated aliasing.

Solution

Clamp F_ab.x to a minimum of 0.1.

Testing

  • cargo run --example atmosphere (with free camera to get grazing angle with water)

Showcase

Before:

image

After:

image

# Objective

The F_AB() polynomial approximation produces F_ab.x values approaching
zero or slightly negative on highly reflective surfaces viewed at
grazing angles, causing fireflies and black pixelated aliasing.

## Solution

Clamp F_ab.x to a minimum of 0.1.

## Testing

- `cargo run --example atmosphere` (with free camera to get grazing
angle with water)

---

## Showcase

Before:

<img width="1542" height="561" alt="image"
src="https://github.com/user-attachments/assets/b9649267-4dbc-4623-8938-9012d5059e3f"
/>

After:

<img width="1768" height="688" alt="image"
src="https://github.com/user-attachments/assets/709c070a-4366-49fe-882d-ad9e1c2f1edb"
/>
@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2026

Welcome, new contributor!

Please make sure you've read our contributing guide and we look forward to reviewing your pull request shortly ✨

@kfc35 kfc35 added D-Trivial Nice and easy! A great choice to get started with Bevy A-Rendering Drawing game state to the screen S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels Feb 7, 2026
@github-project-automation github-project-automation bot moved this to Needs SME Triage in Rendering (2026 Proposal) Feb 7, 2026
@kfc35 kfc35 added the C-Bug An unexpected or incorrect behavior label Feb 7, 2026
@alice-i-cecile alice-i-cecile added X-Uncontroversial This work is generally agreed upon D-Straightforward Simple bug fixes and API improvements, docs, test and examples and removed D-Trivial Nice and easy! A great choice to get started with Bevy labels Feb 8, 2026
@alice-i-cecile alice-i-cecile added this to the 0.18.1 milestone Feb 8, 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 C-Bug An unexpected or incorrect behavior D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward X-Uncontroversial This work is generally agreed upon

Projects

Status: No status
Status: Needs SME Triage

Development

Successfully merging this pull request may close these issues.

3 participants