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

Vector Tiles pt 2 #5243

Merged
merged 20 commits into from
Aug 25, 2018
Merged

Vector Tiles pt 2 #5243

merged 20 commits into from
Aug 25, 2018

Conversation

bhousel
Copy link
Member

@bhousel bhousel commented Aug 21, 2018

Still a bit of cleanup to do, but this PR merges:

  • the layer formerly known as "gpx" (really gpx / kml / geojson) with
  • the layer formerly known as "mvt" (really pbf / mvt but without the ability to place spatially in a z/x/y tile coordinate).

These features now live on a single layer called "Custom Map Data" (yes we will support multiple custom layers soon) and it has a settings screen (like #5207 screen added for custom background imagery). Users can choose a local file, enter a url to a remote file or a url template for a vector tile source:

screenshot 2018-08-21 18 52 26

It currently converts the vector tile data to geojson on the fly and renders just like the old gpx layer did (better styling coming soon too)!

screenshot 2018-08-21 18 34 50

screenshot 2018-08-21 18 36 26

HUGE thanks to @vershwal for her Google Summer of Code work that made this possible and @geohacker for being an awesome mentor 🙌

It seems like the ids stored in the features are not reliable, so I'm trying
to generate ids
If we return the original feature, and then change the hash later as the
feature merges with another, d3 won't exit/enter because it is joining
on the new hash, not the originally hash.
@bhousel
Copy link
Member Author

bhousel commented Aug 25, 2018

This is ready to merge! I've added the ability to hover over data and select it. Internally the data is stored as GeoJSON, so we show whatever is in feature.properties in a readonly raw tag editor.

I'm still considering this feature somewhat "beta", but I'll release a new version of iD in the next day or so that allows users to play around with it, and will include a few example vector tile layers which can help mappers participating in the Detroit Mapping Challenge.

data inspector2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant