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 setting a CanvasTexture for TileMap, there will be lighting errors #83874

Closed
Az-qianchen opened this issue Oct 24, 2023 · 4 comments · Fixed by #83887
Closed

When setting a CanvasTexture for TileMap, there will be lighting errors #83874

Az-qianchen opened this issue Oct 24, 2023 · 4 comments · Fixed by #83887

Comments

@Az-qianchen
Copy link

Godot version

4.2.beta2

System information

WIn 11

Issue description

The problem does not exist in Beta1
Using ImageTexture can get normal results

2023-10-24.135250.mp4

The lights will only show half
The problem does not exist on Sprite2D

Steps to reproduce

pass

Minimal reproduction project

bug.zip

@AThousandShips
Copy link
Member

@AThousandShips AThousandShips added this to the 4.2 milestone Oct 24, 2023
@groud groud removed the regression label Oct 24, 2023
@groud
Copy link
Member

groud commented Oct 24, 2023

Seems to come from: #83489

It was not possible to use CanvasTexture in TileSets before at all, so it's not really this change that caused the issue. So it's not a regression either.

@Az-qianchen have you tried disabling "Use texture padding" ? Do you see any difference ?

@AThousandShips AThousandShips changed the title Tilemap sets a CanvasTexture, will be lighting errors When setting a CanvasTexture for TileMap, there will be lighting errors Oct 24, 2023
@AThousandShips
Copy link
Member

My bad, the issue title made it sound like it made it a CanvasTexture causing bugs, instead of that it happens when someone assigned one, updated the title

@Az-qianchen
Copy link
Author

have you tried disabling "Use texture padding" ? Do you see any difference ?

After the change, I got the right result

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.

3 participants