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

when enable force software skinning, the shadow will become static #53018

Closed
snougo opened this issue Sep 24, 2021 · 6 comments · Fixed by #53144
Closed

when enable force software skinning, the shadow will become static #53018

snougo opened this issue Sep 24, 2021 · 6 comments · Fixed by #53144

Comments

@snougo
Copy link

snougo commented Sep 24, 2021

Godot version

3.4 beta5

System information

macOS 10.14.6 intelGPU GLES3 and GLES2

Issue description

when enable force software skinning, the shadow will not update and become like static shadow, both GLES3 and GLES2, not sure is it only happen on macOS or not

in Editor

Screen.Recording.2021-09-24.at.11.48.30.PM.mov

in Runtime

Screen.Recording.2021-09-24.at.11.47.55.PM.mov

also see this

Screen.Recording.2021-09-25.at.12.18.33.AM.mov

Steps to reproduce

see Minimal reproduction project

Minimal reproduction project

TPS.zip

@snougo
Copy link
Author

snougo commented Sep 24, 2021

both in Editor and Runtime

@Calinou
Copy link
Member

Calinou commented Sep 24, 2021

@snougo Can you reproduce this in 3.3.3?

@snougo
Copy link
Author

snougo commented Sep 24, 2021

@snougo Can you reproduce this in 3.3.3?

yes

Screen.Recording.2021-09-25.at.12.46.56.AM.mov

@lawnjelly
Copy link
Member

At a guess it could be trying to use hardware skinning for the shadow (along with software for the main pass) and the bones aren't set. I wonder why we haven't seen this before... @pouleyKetchoupp

@pouleyKetchoupp
Copy link
Contributor

It looks like shadows are just not updated because the mesh needs to be marked for update on the render side. It's done in the standard skeleton update but not in software skinning.

I'll open a PR soon.

@akien-mga
Copy link
Member

Fixed by #53144.

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