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.

Sign up for GitHub

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

Jump to bottom

Indexed vertices API #4

Closed
mourner opened this issue Jan 21, 2015 · 2 comments · Fixed by #11
Closed

Indexed vertices API #4

mourner opened this issue Jan 21, 2015 · 2 comments · Fixed by #11
Labels
enhancement

Comments

@mourner
Copy link
Member

mourner commented Jan 21, 2015

Instead of outputting triangles as 3 vertices each, we could output unique vertices + list of indexes for triangles instead, like e.g. https://github.com/claus/libtess2.js/. This can save a lot of memory and vertex processing time when uploading vertices data to GL.
@mourner mourner mentioned this issue Jan 22, 2015
Closed
@Maksims
Copy link

Maksims commented Feb 8, 2015

I would like to have such optional feature too.
But based on pull request, you've mentioned it has performance drawback, weird, as from quick ideas, it should be opposite as for performance as well as for memory footprint.

mourner added a commit that referenced this issue Feb 19, 2015
@mourner
accept an option to return indexed vertices, close #4
68b6593
@mourner mourner mentioned this issue Feb 19, 2015
Merged
mourner added a commit that referenced this issue Feb 20, 2015
@mourner
accept an option to return indexed vertices, close #4
754b24d
@mourner
Copy link
Member Author

mourner commented Feb 24, 2015

Merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Accept an option to return indexed vertices mapbox/earcut
Indexed vertices API mapbox/earcut
2 participants
@mourner @Maksims