Opened 3 years ago
Last modified 3 years ago
#21318 new defect
MVT style drops data when two points have same coordinates
Reported by: | Owned by: | team | |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Core imagery | Version: | |
Keywords: | mvt mapbox vector tile | Cc: | taylor.smock |
Description
I have set up a postcode layer and am using this in JOSM via the MVT option. A user has pointed out that when two postcodes have the exact same location, JOSM is only rendering one of them. This could lead to improper mapping. Expected behaviour: show both as is the case in the PNG tile layer I am also providing.
Example location:
51.494328 -0.212871; W14 0QR; W14 0QL; Llewellyn Mansions, Hammersmith Road and 125 Hammersmith Road.
MVT style JSON:
See https://www.openstreetmap.org/user/RobJN/diary/397365
MVT tile the above example appears in shows that it is not an issue in the source data as both postcodes appear within the tile data:
https://api.mapbox.com/v4/robjn.3uur23nz/13/4091/2724.mvt?access_token=pk.eyJ1Ijoicm9iam4iLCJhIjoid0dYNkY1QSJ9.A-0lzQOawGYICYPfURsjDA
PNG raster tile layer (JOSM WMTS):
https://api.mapbox.com/styles/v1/robjn/ckt38gru202nd18labo5bzds4/wmts?access_token=pk.eyJ1Ijoicm9iam4iLCJhIjoid0dYNkY1QSJ9.A-0lzQOawGYICYPfURsjDA&request=GetCapabilities
P.S In the raster tile layer the postcodes are just rendered on top of each other. Whilst this makes readability hard, it does at least show the OSM mapper than there is a conflict here to review. In QGIS the postcode labels are placed one above the other when I pass it the MVT data converted to geojson.
Attachments (0)
Change History (3)
comment:1 by , 3 years ago
comment:2 by , 3 years ago
Cc: | added |
---|---|
Component: | Core → Core imagery |
Keywords: | mvt mapbox vector tile added |
comment:3 by , 3 years ago
@rob.j.nickerson: Thanks for the feedback.
When I was originally implementing MVT support, I think I made the decision to reuse points whenever possible for memory reasons. I'll look into options for this -- it is probably going to be technically easy to change, but I'll want to check out the side effects.
I am the original user who reported this issue to Rob Nickerson. Below is my status report.
The ticket does not offer me the possibility of adding a Cc to taylor.smock but I think he should have a Cc.