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

switch back to triangulating areas #229

Closed
ansis opened this issue Jan 8, 2014 · 2 comments
Closed

switch back to triangulating areas #229

ansis opened this issue Jan 8, 2014 · 2 comments

Comments

@ansis
Copy link
Contributor

ansis commented Jan 8, 2014

@kkaefer back when llmr used pre-triangulated tiles there were size issues, right?

If we store areas as convex polygons in the vector tiles (by rejoining the triangles with Hertel-Mehlhorn or something), it should be only marginally bigger than what we have now? The convex polygons would be easy to triangulate on the client.
@kkaefer
Copy link
Member

kkaefer commented Jan 9, 2014

Yeah, pretriangulated indices took up a lot of space. However, it's possible that the triangulation lib I used was not very good. Are you seeing an immediate need for switching back to tessellated polygons?

@ansis
Copy link
Contributor Author

ansis commented Jan 9, 2014

No immediate need. Did we ever get the stencil buffer figured out on iOS?

@ansis ansis mentioned this issue Feb 7, 2014
Merged
@mourner mourner added this to the future milestone Jun 24, 2014
@ansis ansis changed the title switch back to triangulating areas? switch back to triangulating areas Aug 11, 2014
@ansis ansis closed this as completed Aug 11, 2014
bensleveritt pushed a commit to bensleveritt/mapbox-gl-js that referenced this issue Oct 24, 2016
@mcwhittemore
Merge pull request mapbox#229 from conorhastings/master
ebc393e 
use Object.assign with defaultControls instead of several ternaries
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mourner @kkaefer @ansis