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

Object loader and rendering #26

Open
aromanro opened this issue Dec 4, 2022 · 0 comments
Open

Object loader and rendering #26

aromanro opened this issue Dec 4, 2022 · 0 comments
Assignees

Comments

@aromanro
Copy link
Owner

aromanro commented Dec 4, 2022

There is some benefit of having the vertices indexed. As it is now, it duplicates the vertices, adding unnecessary overhead (and it's more than simply a vertex, it's also the normal, the texture coordinates, whatever).
OpenGL can deal with indexed stuff.

It ended it as it is now because I ported it from the ray tracing project, where it made sense to avoid the indexing complexity at the time when I implemented it.

As it's only a toy project, I'm not sure I'll do the change, but maybe later the code could be reused in some other project so it might be worth changing it.

@aromanro aromanro self-assigned this Dec 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant