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

Shaders: Added new shader chunks for tangent support #22269

Merged
merged 2 commits into from
Aug 4, 2021

Conversation

WestLangley
Copy link
Collaborator

This PR adds three new shader chunks to consolidate some frequently-used code patterns.

These shader chunks are required for universal tangent support.

The new shader chunks have been applied to MeshPhongMaterial and MeshToonMaterial as a start.

@WestLangley WestLangley added this to the r132 milestone Aug 4, 2021
@Mugen87
Copy link
Collaborator

Mugen87 commented Aug 4, 2021

Seems this PR breaks some node examples.

@WestLangley
Copy link
Collaborator Author

/ping @sunag I am trying to make these PRs as concise and clear as possible so it is easy for you to see how it impacts Node. I am grateful for your help and/or feedback.

@WestLangley
Copy link
Collaborator Author

I updated Node and the examples impacted by this PR.

@Mugen87
Copy link
Collaborator

Mugen87 commented Aug 4, 2021

LGTM!

@sunag
Copy link
Collaborator

sunag commented Aug 4, 2021

For me it's great too. I was going to check it out now to fix... thanks!

@mrdoob mrdoob merged commit df12db0 into mrdoob:dev Aug 4, 2021
@mrdoob
Copy link
Owner

mrdoob commented Aug 4, 2021

Thanks!

@WestLangley WestLangley deleted the dev_normal_shader_chunks branch August 4, 2021 20:02
0b5vr added a commit to pixiv/three-vrm that referenced this pull request Oct 21, 2021
MasatoMakino added a commit to MasatoMakino/threejs-shader-materials that referenced this pull request Jun 18, 2022
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.

4 participants