-
Notifications
You must be signed in to change notification settings - Fork 120
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
Add OSM alternate viewpoints for country boundaries #1810
Comments
I've been playing around with the data for this for a while, and it doesn't look like the data is even in a suitable state today to build sensible-looking lines, let alone polygons for left/right labels. For example, this is the "on the ground" view from OpenStreetMap: And this is what the data indicates India's view should be: And what the data says Pakistan's view should be: And what the data says China's view should be: What's being shown?This is looking at the
What's going on with the extra random lines?Most of the line-of-control between India and Pakistan is tagged as Some other problems appear to stem from typos, for example this alleged dispute between India and Switzerland (fix: How widespread is this data?This particular schema seems to only be usefully well implemented around the three-way IN / PK / CN dispute. There's some related tagging on the IL / SY dispute, but the SY claims aren't collected into a claim relation, nor are SY's claims tagged as Other disputes, such as Doklam, the Ilemi Triangle, Hala'ib Triangle are mapped with a simple Next stepsOn the basis of the flakiness of this data, I'm not sure how to reliably recover view-dependent polygons to ensure the correct sidedness of the labels. At the moment, all borders |
See active discussion in OSM wiki: |
I've prototyped almost all of this data into OpenStreetMap. Remaining work:
For the tile generations:
|
We're missing data / modeling around Cyprus, followup here: #1898. |
We should also add the Followup in: #1900 |
@zerebubuth Do you see why Taiwan country label comes in at zoom 6? It's min_label in NE is 3. |
Ah, that's interesting! The place point from OSM has a Wikidata ID of Q22502, which is Taiwan Island. However, Taiwan the "state with limited recognition in East Asia" is Q856, which is the value in the Natural Earth data. This means that the data doesn't join up, we don't assign an The country boundary relation in OSM has the correct value, so I'm tempted to call this a data issue - the place point is a label for the relation, not just the island. If we want to work around this, it might be possible to link back up to the relation from the place point... but might be complicated. |
Agree it's a data problem, I've edited OSM to have Q865 instead (from your link, not the link text): |
Followup for disputed modeling in OSM (works in NE now) in #1901. |
@zerebubuth how could we solve this one? We don't want Western Sahara to label on the Morocco side of the boundary on the south side of this image. But doesn't look like there's enough data in tiles to turn it off – to know that Morocco disputes the "left" side of the line to turn that label off. |
NOTE: I think the stub of boundary above is https://www.openstreetmap.org/way/526798853 that had a |
You're right - we don't build the disputed / claimed shapes of the countries, so we can't automatically ensure that the label point is within the boundary always. However, we can:
|
Ooops, sorry - I misunderstood what you meant. I was thinking of the country label, but I think you probably meant the boundary label. The way to turn it off is somewhat complex and annoying, but analogous to the (claimed) border between India and Afghanistan. What we can do is:
Then the claim will appear for Morocco with the label in the |
I'll do some more editing as advised in #1810 (comment) and then we can do a new build once that shows up in a planet file. |
In #1809 we added schema support for alternate points of view in the boundaries layer, and added data from Natural Earth.
This issue tackles adding data parsing from OpenStreetMap, where the data for this is sparse today. And we'll need to figure out how to build polygons out of them so we can still do the left/right boundary labels.
The text was updated successfully, but these errors were encountered: