-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Change loadModel() example #2968
Conversation
To resolve #2904 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add the new example instead of replacing the old (teapot) example
@almchung Done! |
hi @TanviKumar, I'm not seeing your updates—can you make sure you've pushed your changes? thanks! |
Hey @kjhollen, sorry about that, changes pushed! |
great! I made one little edit because the word 'teapot' was still hanging around in the alt text for the other example. |
I'm sorry that I'm ignorant of the process here, and don't quite have the skills to contribute, but there are still tiny teapots in the examples and no octahedrons... |
@PAEDevelopment , the example code has been changed to reflect the requirement, I believe these changes are yet to be pushed to the site itself and should be in the near future. |
Use smaller .obj file for faster loading
Model used is suitable for canvas size of example. (current example is too small)