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

Carryall: render no shadow from sprite actor. #21378

Open
wants to merge 1 commit into
base: bleed
Choose a base branch
from

Conversation

dnqbob
Copy link
Contributor

@dnqbob dnqbob commented Mar 30, 2024

No description provided.

@pchote
Copy link
Member

pchote commented Mar 30, 2024

Why? We do want to render the shadow…?

@dnqbob
Copy link
Contributor Author

dnqbob commented Mar 30, 2024

We use WithShadow to render a shadow right on the ground, not just at the actor place. I will show you an example of extra shadow problem.

@dnqbob
Copy link
Contributor Author

dnqbob commented Mar 30, 2024

2shad

As you see the Carryall render shadow of passenger in the mid-air, and there is also a shadow of passenger rendered on the ground by WithShadows. It is no doubt that The one rendered by WithShadows makes sense.

@pchote

@dnqbob
Copy link
Contributor Author

dnqbob commented Mar 30, 2024

And this is what happens when we apply this PR:
1shad

@dnqbob
Copy link
Contributor Author

dnqbob commented Mar 30, 2024

Why? We do want to render the shadow…?

In any case, our Carryall must be Aircraft, their shadow together with passenger's shadow will all be done by WithShadows.

@pchote
Copy link
Member

pchote commented Mar 30, 2024

That looks like a bug with shattered paradise not declaring shadows properly. Can you reproduce the issue in TS?

@dnqbob
Copy link
Contributor Author

dnqbob commented Mar 30, 2024

@pchote
2shad

@pchote
Copy link
Member

pchote commented Mar 30, 2024

Ok, but i'm not convinced this is the correct solution - it used to work properly without this change, and its not the only regression (voxels no longer follow the rotation of the carryall) suggesting that the bug may be somewhere else.

@dnqbob
Copy link
Contributor Author

dnqbob commented Mar 30, 2024

it used to work properly without this change

For god's sake, you can test it at 5cc1405, I have seen this bug since 2021.
2shad-old

This is an old bug actually, I guess it is just no one notice it.

@dnqbob
Copy link
Contributor Author

dnqbob commented Mar 30, 2024

its not the only regression (voxels no longer follow the rotation of the carryall) suggesting that the bug may be somewhere else

Voxels shouldn't be related to this issue, and voxels' shadow work well. This bug only happens to sprites that use shadow.

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

Successfully merging this pull request may close these issues.

None yet

2 participants