[HDRP] Fix spot shadow sampling when using custom spot angle by alelievr · Pull Request #5439 · Unity-Technologies/Graphics · GitHub
Skip to content

[HDRP] Fix spot shadow sampling when using custom spot angle#5439

Merged
sebastienlagarde merged 3 commits into
hd/bugfixfrom
hd/fix/spot-shadow-sampling
Aug 31, 2021
Merged

[HDRP] Fix spot shadow sampling when using custom spot angle#5439
sebastienlagarde merged 3 commits into
hd/bugfixfrom
hd/fix/spot-shadow-sampling

Conversation

@alelievr

@alelievr alelievr commented Aug 25, 2021

Copy link
Copy Markdown
Contributor

Purpose of this PR

Fix an issue regarding the shadow map sampling of spotlights with the custom spot angle option enabled

https://forum.unity.com/threads/custom-shadow-angle-for-spot-light-having-wrong-wrap-mode.1139824/


Testing status

Checked that there is no more shadow leaking from the side of the shadow map:

Before:
image

After:
image

I also tested the pyramid spot light shape

https://yamato.cds.internal.unity3d.com/jobs/902-Graphics/tree/hd%252Ffix%252Fspot-shadow-sampling/.yamato%252Fall-hdrp.yml%2523PR_HDRP_trunk/8370632/job

@Unity-Technologies Unity-Technologies deleted a comment from github-actions Bot Aug 26, 2021
@sebastienlagarde sebastienlagarde changed the base branch from hd/bugfix to master August 29, 2021 09:29
@sebastienlagarde sebastienlagarde changed the base branch from master to hd/bugfix August 29, 2021 09:29
@sebastienlagarde sebastienlagarde requested a review from a team August 30, 2021 17:17

@RemyUnity RemyUnity left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM, thanks for the screenshots 🟢

@sebastienlagarde sebastienlagarde merged commit 46bf4f0 into hd/bugfix Aug 31, 2021
@sebastienlagarde sebastienlagarde deleted the hd/fix/spot-shadow-sampling branch August 31, 2021 12:20
@github-actions

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants