-
Notifications
You must be signed in to change notification settings - Fork 819
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
boundary=administrative, admin_level=4 labels obscures place=city labels #1391
Comments
What do you mean by admin_centre name? I do not find that text anywhere in the style. |
I may be wrong what really causes problem, because we have relations, places, labels - so here is the visible province relation: http://www.openstreetmap.org/relation/130935 and here is the obscured city relation: |
The problem is that boundary=administrative, admin_level=4 labels obscures place=city labels. I'm not sure if it's always a bad thing. What is more important to label, Northern Ireland or Belfast? Baden-Wurtenberg or Stuttgart? Massachussets or Boston? |
z3 - blank area, not even country name (?) z4 - only WP and MA voivodeships (quasi-states) are visible (clear USA-based influence, since nobody in Poland uses such abbreviations for voivodeships!), and no country name z5 - only "województwo wielkopolskie" (that was WP at z4) and country name in the middle z6 - now it's getting funny =} :
z7 - almost the same level of fun, now just the minority of capitals are visible, but country's capital is still not (!) For now I want to focus on z6-7 (at least in this ticket probably), because it's clear space for improvement here and it's local. z4-5 would be interesting for sure too, because we have many countries to check:
And I haven't even look at the rest of the world! There are probably much more things to consider, but I think admin_level area names shifting away from cities could also help here to some extent. |
Probably it is an already known issue but it is weird to view mapnik without finding immediately some obvious info. I expected to see New York City, Philadelphia and Washington DC. Instead I see Yonkers, Camden and Arlington. At zoom 8! In Italy at zoom 5 there are the labels for many of the larger cities but not for Rome (almost 3 million inhabitants) and Turin. But there are Compobasso (50000) and Catanzaro (91000). |
the Rome case is rather particular because there is a country label competing with it (vatican, not any more at the moment) as well as the region label (lazio, z6+7) |
I did not know that, how does it work? do those names come pre-selected in the dataset? Also: Berlin is weird, too. What I take from this exercise:
|
see #1069
Not all of them (lower divisions, but anyway currently such labels are displayed only for borders) |
as there is no thing such as an admin_center name and people are pointing out various separate issues, I'm looking closing this ticket. If people want to make more focused tickets about clearer issues, that's fine, but this ticket isn't clear. |
I think @math1985 phrased it better: |
Berlin is admin level 4, this is not "minor", it has the same significance as Bavaria or a state of the US, to make these new rules work we'd have to render (some?) cities before all admin names below the country level. Not bad IMHO, but will likely have quite some impact on certain levels, so this is a decision we should be well aware of and take it consciously. |
The ideal solution would be to let label for country/region move a bit to allow displaying of city icon but I see no example of anybody doing this in CartoCSS. I also have no obvious idea (beyond something inefficient and complicated like playing with ST_Buffer to find area close to country center but not next to cities..,) |
Isn't French style a fork of osm-carto? IIRC @cquest managed to do it somehow. |
They use |
I did the testing too with the same results. Yet it looks something works in FR, so I will try to look at it closer. |
Please reopen this issue, it was incidentally closed by #2314 ("Tries to partly fix" contains "fix" keyword). |
My last work on this uses text-placement to allow the state labels to move around if needed. Current dev style is visible on: http://u.osmfr.org/m/99740/ |
I've been already trying to debug your fork, because it works better than this style for many months, but it has diverged too much and I was not focused enough. I still plan to test it and find out the difference - thanks for the hint! |
I forked the style at its early days... and learned carto-css at the same time. I've also use some ugly tricks using the _rels / _ways tables to get some details not available in the geometry tables produces by osm2pgsql (for example to get the operator on bus stops when this tag in only available in the relation). |
This should be probably resolved by using |
The code is in the Mapnik repo, now I wait for v3.0.19 release to test it. |
Mapnik 3.0.19 has been released and is even available as a Debian package, so I was able to not wait for official Ubuntu package, npm Mapnik bindings and Kosmtik update, and forged it myself:
Mapnik 3.0.19 has new Poland, z6 Poland, z7 (click to see the full size images) The real change will be when we make a v5.0.0 code branch (since it means "upgrading software dependecies") and test the |
Looks like 3.0.20 (for Ubuntu Bionic 18.04 LTS) is finally deployed on OSMF servers, since Lake Victoria has label in the center now: also all the other label placing problems I was aware of are gone. Now that we have proper Mapnik version on OSMF servers, we can try using |
In Italy I see still problems in admin level priority labels. |
Change of internal algorithm does not solve such problems, but it may accidentally (just like any change, at the same time it can make some other things worse). It was introduced only to fix the problems like #1465. New algorithm is now ready to be tested instead and this should really help by moving labels to a free space. |
@nebulon42 said that
|
OK, it works now, an early test of this concept and new toolchain is here: #2962 (comment). |
So this currently needs wait for v5.0 since it requires Mapnik/CartoCSS update, right? |
Closing in favor of #3034. |
(This is more general problem already described in #1386 ):
Sometimes admin_centre name is obscured by admin_level name, which I consider a serious bug for a general map. For example Warsaw is not visible up to z=7 (I think there are some other bugs at low zoom levels, but 6 and 7 seem like obvious cases of above).
IMO admin_level should always try hard to allow admin_centre name to appear where it really is, because the area is probably huge and we have more flexibility in choosing the place to render its name. I would use the rule of thumb that admin_level name should always stay entirely in its boundaries - and only obscure the admin_centre name if it's not really possible.
The text was updated successfully, but these errors were encountered: