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

Vulkan Mobile: Decal and light Cull Mask is not working #65848

Closed
Tracked by #55871
jcarlosrc opened this issue Sep 15, 2022 · 5 comments · Fixed by #72810
Closed
Tracked by #55871

Vulkan Mobile: Decal and light Cull Mask is not working #65848

jcarlosrc opened this issue Sep 15, 2022 · 5 comments · Fixed by #72810

Comments

@jcarlosrc
Copy link

Godot version

4.0 Beta 1

System information

Windows 10

Issue description

Setting a cull layer does not have any effect when using vulkan mobile. In the minimal project, a decal and a light have cull layer as none. However, both are shown on a box that is at visual layer = 2.

Steps to reproduce

Set a cull layer to a decal or light.

Minimal reproduction project

DecalCull.zip

@Calinou Calinou added this to the 4.0 milestone Sep 15, 2022
@Calinou Calinou changed the title Cull layers are not working when using vulkan mobile backend Vulkan Mobile: Decal and light Cull Mask is not working Sep 15, 2022
@Calinou
Copy link
Member

Calinou commented Sep 15, 2022

Can you reproduce this on previous 4.0 alphas?

@jcarlosrc
Copy link
Author

Seems like Alpha 16 and 17 had the same problem.

@alcomposer
Copy link
Contributor

alcomposer commented Oct 21, 2022

DirectionalLight3D Looks like it honours cull layer on mobile vulkan, while Spot & Omni doens't.

I have tested on Godot4 Beta3

test_cull_light.zip

Screencast.from.22-10-22.02.06.29.MP4

@jcarlosrc
Copy link
Author

jcarlosrc commented Nov 29, 2022

Still a problem in Beta 6.

@clayjohn
Copy link
Member

clayjohn commented Jan 4, 2023

This may have been fixed for Omni and Spot lights in #70638 which will be available for testing in Beta 11

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

Successfully merging a pull request may close this issue.

5 participants