{"action":"created","issue":{"url":"https://api.github.com/repos/gravitystorm/openstreetmap-carto/issues/1461","labels_url":"https://api.github.com/repos/gravitystorm/openstreetmap-carto/issues/1461/labels{/name}","comments_url":"https://api.github.com/repos/gravitystorm/openstreetmap-carto/issues/1461/comments","events_url":"https://api.github.com/repos/gravitystorm/openstreetmap-carto/issues/1461/events","html_url":"https://github.com/gravitystorm/openstreetmap-carto/pull/1461","id":63816954,"number":1461,"title":"Order placenames by population and substitute NE usage on low zoom level...","user":{"login":"sommerluk","id":6830724,"avatar_url":"https://avatars.githubusercontent.com/u/6830724?v=3","gravatar_id":"","url":"https://api.github.com/users/sommerluk","html_url":"https://github.com/sommerluk","followers_url":"https://api.github.com/users/sommerluk/followers","following_url":"https://api.github.com/users/sommerluk/following{/other_user}","gists_url":"https://api.github.com/users/sommerluk/gists{/gist_id}","starred_url":"https://api.github.com/users/sommerluk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sommerluk/subscriptions","organizations_url":"https://api.github.com/users/sommerluk/orgs","repos_url":"https://api.github.com/users/sommerluk/repos","events_url":"https://api.github.com/users/sommerluk/events{/privacy}","received_events_url":"https://api.github.com/users/sommerluk/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":74,"created_at":"2015-03-23T19:51:36Z","updated_at":"2015-09-14T09:07:39Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/gravitystorm/openstreetmap-carto/pulls/1461","html_url":"https://github.com/gravitystorm/openstreetmap-carto/pull/1461","diff_url":"https://github.com/gravitystorm/openstreetmap-carto/pull/1461.diff","patch_url":"https://github.com/gravitystorm/openstreetmap-carto/pull/1461.patch"},"body":"Order placenames by population and substitute NE usage on low zoom levels.\r\n\r\nResolves  #50 \r\nResolves #255 \r\nResolves #482\r\nResolves #1082 \r\nResolves #1505 \r\n\r\nIt tries to order cities and towns by population. This would improve the rendering regarding which label has to be supressed in case of collision.\r\n\r\nIt substitutes Natural Earth populated places labels on low zoom levels (3–4). It uses OSM data instead, making a choise based on population=*. It uses the same font color at 3–4 like on the higher zoom levels.\r\n\r\nIt tries to guarantie that once a label has been rendered at a specific zoom level, it is not supressed at higher zoom levels by other labels of start rendering only at higher zoom levels (as far as possible).\r\n\r\nIt tries to preserve more or less the current placenames density."},"comment":{"url":"https://api.github.com/repos/gravitystorm/openstreetmap-carto/issues/comments/140008670","html_url":"https://github.com/gravitystorm/openstreetmap-carto/pull/1461#issuecomment-140008670","issue_url":"https://api.github.com/repos/gravitystorm/openstreetmap-carto/issues/1461","id":140008670,"user":{"login":"imagico","id":5408119,"avatar_url":"https://avatars.githubusercontent.com/u/5408119?v=3","gravatar_id":"","url":"https://api.github.com/users/imagico","html_url":"https://github.com/imagico","followers_url":"https://api.github.com/users/imagico/followers","following_url":"https://api.github.com/users/imagico/following{/other_user}","gists_url":"https://api.github.com/users/imagico/gists{/gist_id}","starred_url":"https://api.github.com/users/imagico/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/imagico/subscriptions","organizations_url":"https://api.github.com/users/imagico/orgs","repos_url":"https://api.github.com/users/imagico/repos","events_url":"https://api.github.com/users/imagico/events{/privacy}","received_events_url":"https://api.github.com/users/imagico/received_events","type":"User","site_admin":false},"created_at":"2015-09-14T09:07:39Z","updated_at":"2015-09-14T09:07:39Z","body":"Selecting a low fallback population number will probably not be much encouragement for tagging correct populations but likely significantly push tagging for the renderer with the place classes (if a town does not show up it is much more straight forward to tag it place=city instead of finding out the population number).\r\n\r\nThe most encouraging approach for the mapper for correct tagging overall is quite surely interpreting the tags exactly as they are supposed to be used.\r\n\r\nThe  most important problem of the approach here is that is does not display smaller places even if they are far away from other places.  Like:\r\n\r\nhttp://www.openstreetmap.org/node/244081999\r\nhttp://www.openstreetmap.org/node/31219396\r\nhttp://www.openstreetmap.org/node/29341070\r\n\r\nThe first two are correctly tagged as city, the latter is tagged for the renderer.  They could all well show up at z5 although they do not.  In the first case tagging the correct population will probably help, in the other two probably not.\r\n\r\nApart from that i still think major cities should have priority over subnational admin entities.\r\n"}}