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

Added shader material #2

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DanielAbalde
Copy link
Collaborator

Here is a basic implementation for shader material.

image

It can be extended once you pass this implementation, mainly to add new uniform types.

I think you have to implement your Script object first in order to be able to update uniform values since Render().

Sample file (is more a dev testing file that a sample file):
ThreePlus-Materials-11_Shader.zip


It has been a headache to synchronize my project with yours because I downloaded the project as a ZIP and your VS solution is inside a folder. I was able to fix it by re-downloading the project, this time with "Open with Visual Studio" and the cloning was successful. I've probably used a bad habit but it has worked for me so far. However when I copied and pasted my changes into the new properly cloned project, there was a formatting adjustment adding unnecessary spaces. Keep this in mind when reviewing the changes.

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.

1 participant