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

Feature request: Add LambertAnimationMaterial #9

Closed
supermoos opened this issue Oct 3, 2017 · 2 comments
Closed

Feature request: Add LambertAnimationMaterial #9

supermoos opened this issue Oct 3, 2017 · 2 comments

Comments

@supermoos
Copy link
Contributor

First of all, thanks for this amazing library!
It's my understanding that LambertMaterial is slightly more performant than PhongMaterial. Would be nice to add it as default material to the library :-)

@zadvorsky
Copy link
Owner

Thanks for the suggestion.

I added BAS.LambertAnimationMaterial in the new release (2.2.0).

Do note however that Lambert does not support flat shading, so you will have to compute vertex normals every time.

@supermoos
Copy link
Contributor Author

Wow, thanks! That was fast :-)

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

No branches or pull requests

2 participants