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 soft shadows by increasing the bit count for specialization constants. #98614

Merged
merged 1 commit into from
Oct 28, 2024

Conversation

DarioSamo
Copy link
Contributor

Fixes #97839.

The previous amount of bits could not hold the possible 16 and 32 amount of samples. The amount has been expanded to the bare minimum required to hold this value, as space for these constants must be as small as possible for optimization reasons.

Copy link
Member

@Calinou Calinou left a comment

Choose a reason for hiding this comment

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

Tested locally, it works as expected in Forward+ and Mobile. Code looks good to me.

Copy link
Member

@clayjohn clayjohn left a comment

Choose a reason for hiding this comment

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

Makes sense! I think we just missed this when adding the ubershader

@clayjohn clayjohn merged commit 08f9cba into godotengine:master Oct 28, 2024
20 checks passed
@clayjohn
Copy link
Member

Thank you!

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.

Forward+/Mobile: Shadow filtering is broken with High or Ultra quality
4 participants