Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[NOT READY] GeoJSON support #170

Closed
wants to merge 35 commits into from
Closed

[NOT READY] GeoJSON support #170

wants to merge 35 commits into from

Conversation

incanus
Copy link
Contributor

@incanus incanus commented May 7, 2014

This has some issues, but is a good first cut. It follows very closely the design on the WebGL side, which pre-tiles GeoJSON features after fetch/parse using multiple Transform objects, one per zoom level supported, so that tiles never need to be recalculated.

Things that need work:

@incanus incanus added this to the WWDC milestone May 7, 2014
@incanus incanus mentioned this pull request May 7, 2014
incanus and others added 24 commits May 7, 2014 10:37
 * uses OSM Bright style
 * adds dot shader
 * adds layer translation
 * adds dashed line support
 * adds style array parsing
 * improves background layer parsing
 * improves style constant parsing
 * fixes sprite JSON pixel ratio parsing
`./setup_libraries.sh` --> `./setup-libraries.sh`
This changes the clip IDs from sequential to a huffman prefix tree, meaning that
parent tiles share the same prefix with child tiles (but are shorter). This allows
us to use the same checkerboard pattern to draw all tiles.

TODO:
- make sure we're not overflowing the 8 bit limit of the stencil buffer
- draw all checkerboard fields at the same time, with one draw call
- fix raster drawing (fading, clipping)

refs #132
This is needed for correctly compositing raster imagery into the map

fixes #122
fixes #157
@incanus
Copy link
Contributor Author

incanus commented May 13, 2014

Ok, putting this on hold for a while in light of other priorities. Just merged master and we have a couple build errors in GeoJSONSource::tile_GeoJSON around direct access to buckets for parsing (see comment about TileParser in original ticket description). Main thing we need here is a parallel parsing routine that can create buckets, but otherwise this is done for basic parsing and rendering of LineString features.

@incanus incanus removed this from the WWDC milestone May 28, 2014
@incanus incanus closed this Jun 4, 2014
@jfirebaugh jfirebaugh deleted the geojson-work branch August 20, 2014 20:56
@incanus incanus mentioned this pull request Oct 13, 2014
@incanus incanus mentioned this pull request Oct 27, 2014
5 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature GL JS parity For feature parity with Mapbox GL JS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants