Skip to content

Solari: Fix self-intersection artifacts#23813

Merged
alice-i-cecile merged 2 commits intobevyengine:mainfrom
JMS55:soalri6-self-intersect
Apr 15, 2026
Merged

Solari: Fix self-intersection artifacts#23813
alice-i-cecile merged 2 commits intobevyengine:mainfrom
JMS55:soalri6-self-intersect

Conversation

@JMS55
Copy link
Copy Markdown
Contributor

@JMS55 JMS55 commented Apr 15, 2026

Fix self intersection artifacts by offsetting the ray origin along the geometric normal (or shading normal when geo normal not available, e.g. from gbuffer).

Fixes flickering in Solari's many_lights test scene.

@alice-i-cecile alice-i-cecile added C-Bug An unexpected or incorrect behavior A-Rendering Drawing game state to the screen S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it D-Modest A "normal" level of difficulty; suitable for simple features or challenging fixes labels Apr 15, 2026
@alice-i-cecile alice-i-cecile added this pull request to the merge queue Apr 15, 2026
@github-project-automation github-project-automation Bot moved this to Needs SME Triage in Rendering Apr 15, 2026
Merged via the queue into bevyengine:main with commit c0e8e29 Apr 15, 2026
46 checks passed
@github-project-automation github-project-automation Bot moved this from Needs SME Triage to Done in Rendering Apr 15, 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-Modest A "normal" level of difficulty; suitable for simple features or challenging fixes 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.

3 participants