mapbox / mapbox/tilelive-vector
Format type JSON should return GeoJSON?
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 72
- Forks
- 42
- PR merge metrics
- No merged PRs in 30d
Description
This is a question that could lead to an actual issue.
Was it intentional to return a JSON representation of the Mapbox vector tile when requesting for format type "json"? [See this line](https://github.com/mapbox/tilelive-vector/blob/master/index.js#L193).
Or should that `toJSON` be replaced with `toGeoJSONSync`?
If the current implementation is correct then is there a design or requirements reason for it? My amateur understanding is that `.mvt` is provided as a protobuf and made for rendering. I'd imagine viz libraries expects it in pbf so what good is a JSON representation?
Thanks
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading index.js at line 193 and comparing the current toJSON behavior with the proposed toGeoJSONSync call. Establish the intended JSON response and its design or requirements rationale; the issue is done when that behavior is documented or an agreed implementation path is defined.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- api
- Issue type
- Bug
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100