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

Broken lighting on Collada mesh with normal map [Intel HD 3000] #8711

Closed
khairul169 opened this issue May 10, 2017 · 8 comments
Closed

Broken lighting on Collada mesh with normal map [Intel HD 3000] #8711

khairul169 opened this issue May 10, 2017 · 8 comments

Comments

@khairul169
Copy link
Contributor

khairul169 commented May 10, 2017

Operating system or device - Godot version:
Ubuntu 16.04 - Intel HD 3000
Godot 3.0 custom from commit 6d0568d

Issue description:
Mesh imported from collada have a weird lighting.. But testcube are fine..
screenshot from 2017-05-10 19-38-00

Steps to reproduce:
Create a simple mesh, uv unwrap, and export it with better collada exporter, with tangent arrays exported.. Give it material with normal map, and place some point light or directional light..

Link to minimal example project:
Normal Map Test.zip
normal map by Calinou btw

@bojidar-bg
Copy link
Contributor

Are you sure that the normals of the mesh point outwards?

@bojidar-bg
Copy link
Contributor

Related to #8673, #8004, #6440, and potentially a few others...

@khairul169
Copy link
Contributor Author

yeah i'm sure about that
screenshot from 2017-05-11 17-08-22

@akien-mga akien-mga changed the title Weird Normal Map Issue Broken lighting on Collada mesh with normal map [Intel HD 3000] May 18, 2017
@ippan
Copy link
Contributor

ippan commented May 19, 2017

commit a75623f
Mac Nvidia 755M
s

@HummusSamurai
Copy link
Contributor

Seems like the problem is solved?

@bojidar-bg
Copy link
Contributor

Yes, but it would be nice if @khairul169 confirms so as well.

@khairul169
Copy link
Contributor Author

Yeah with the latest commit, test project are works fine.. 👍
Did i have to close this issue myself?

@akien-mga
Copy link
Member

Great news :)

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

No branches or pull requests

5 participants