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

NormalMaps are weird in 4.4-dev3 #97774

Open
caioraphael1 opened this issue Oct 3, 2024 · 6 comments · May be fixed by #97340
Open

NormalMaps are weird in 4.4-dev3 #97774

caioraphael1 opened this issue Oct 3, 2024 · 6 comments · May be fixed by #97340

Comments

@caioraphael1
Copy link

Tested versions

Ok in 4.4-dev2, weird in 4.4-dev3

System information

Windows 10, Ryzen 3 3300x, 16gb RAM 2666mhz, GTX 1660 6gb.

Issue description

I don't know what's happening. I just opened my project on in 4.4-dev3, and there were a lot of regions completely black. After tracking the source of the problem, it seems to be related to the normalmap from a CanvasTexture

2024-10-03.13-09-01.mp4

Steps to reproduce

I don't know :/, I just happened after opening the project in 4.4-dev3

Minimal reproduction project (MRP)

Sry

@caioraphael1 caioraphael1 changed the title NormalMaps area weird in 4.4-dev3 NormalMaps are weird in 4.4-dev3 Oct 3, 2024
@AThousandShips
Copy link
Member

Please upload an MRP to make this easier to test and fix

  • A small Godot project which reproduces the issue, with no unnecessary files included. Be sure to not include the .godot folder in the archive (but keep project.godot).
  • Having an MRP is very important for contributors to be able to reproduce the bug in the same way that you are experiencing it. When testing a potential fix for the issue, contributors will use the MRP to validate that the fix is working as intended.
  • Drag and drop a ZIP archive to upload it (max 10 MB). Do not select another field until the project is done uploading.
  • Note for C# users: If your issue is not C#-specific, please upload a minimal reproduction project written in GDScript. This will make it easier for contributors to reproduce the issue locally as not everyone has a .NET setup available.

@DarioSamo
Copy link
Contributor

DarioSamo commented Oct 3, 2024

CanvasRenderer got a lot of changes in dev3 to introduce batching. I suspect some state related to the normal map might be leaking to the rest of the draw calls. I'd probably try with a commit before #92797 to start bisecting from.

As ATS has said, it'll be really helpful to have an MRP.

@clayjohn
Copy link
Member

clayjohn commented Oct 3, 2024

This might be fixed by #97340 which fixes a couple of bugs introduced by batching

@Lielay9
Copy link
Contributor

Lielay9 commented Oct 3, 2024

MRP (Forward/Mobile), content in tree after a CanvasTexture with NormalMap are dimmed.
normaltexturebug.zip
Issue in Master [5ccbf6e]:
Screenshot 2024-10-03 214135

#97340 (Couple day old artifact), doesn't have this issue:
Screenshot 2024-10-03 214305

Didn't bisect, but I had builds directly before and after #92797 and it doesn't seem to be caused by it (unless I messed up which build was which) but the behavior does seem related to batching.

@clayjohn clayjohn added this to the 4.4 milestone Oct 3, 2024
@clayjohn
Copy link
Member

clayjohn commented Oct 3, 2024

This might have been caused by #97554 which was a hotfix for a different issue introduced by batching.

I checked and #97340 does fix this PR!

@Lielay9
Copy link
Contributor

Lielay9 commented Oct 3, 2024

This might have been caused by #97554 which was a hotfix for a different issue introduced by batching.

I checked and #97340 does fix this PR!

Out of curiosity, and since I had builds for that too, I can confirm 4d635b7 was indeed the cause of the issue.

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