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

Move ShaderNode internals to a Node class #24961

Merged
merged 5 commits into from
Nov 18, 2022
Merged

Move ShaderNode internals to a Node class #24961

merged 5 commits into from
Nov 18, 2022

Conversation

LeviPesin
Copy link
Contributor

Related issue: -

Description

Currently something like material.colorNode = new ShaderNode( ... ) does not work in WebGL nodes (tested that), because new ShaderNode( ... ) does not have an isNode property (and also does not have a cache key implementation).
This PR fixes that and implements the @sunag's suggestion in the comment // @TODO: Move this to Node extended class.

@sunag sunag added this to the r147 milestone Nov 16, 2022
@mrdoob
Copy link
Owner

mrdoob commented Nov 16, 2022

@sunag Feel free to merge this if it looks good to you 👍

@sunag
Copy link
Collaborator

sunag commented Nov 16, 2022

@LeviPesin this broke webgpu_compute and webgpu_audio_processing examples.

@LeviPesin
Copy link
Contributor Author

Should be fixed now.

@sunag
Copy link
Collaborator

sunag commented Nov 17, 2022

@LeviPesin I added some changes.

@sunag sunag merged commit 044d7b2 into mrdoob:dev Nov 18, 2022
@sunag
Copy link
Collaborator

sunag commented Nov 18, 2022

Thanks!

@LeviPesin LeviPesin deleted the patch-1 branch November 19, 2022 06:26
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.

3 participants