mapbox / mapbox/mr-mvt

Just discussion

Open
#1 8 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
C++
Stars
4
Forks
4
PR merge metrics
No merged PRs in 30d

Description

@flippmoke this worked 💯 on my first try. Pretty slick. I just thought I'd open a ticket here where we can discuss what's happening next.

- Right now, is the output one line per feature per zoom level? Or one line per **tiled**-feature per zoom level? That is, are you doing simplification and clipping yet?

- I guess the reduce functions for actually building vector tiles are still to come? Do you think that the details of how that works can be kept distinct from decisions about the architecture for storing the outputs of the map step?

- From the perspective of figuring out how the storage architecture will work, I do think that we'll probably want to generate a property-store that is separate from the clipped, simplified geometry store. I guess what would be cool is if `m2f` were to set the properties aside in another file (?) so that they aren't duplicated in each zoom/tile of the `m2z` output.

- Having a distinct property store means the reduce step will need to be able to do lookups based on feature ids. Do you have any thoughts about how to handle geojson features that don't bring their own ids? Or do bring ids that aren't numbers?

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

No source file, test, or entry point is named. Review the discussion around m2f, m2z, vector-tile reduction, and property storage; this issue needs a decided implementation direction before a newcomer can start.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
data
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
15/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.