-
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
Moving smaller airports to higher zoom levels #2674
Conversation
It's a low priority request, since this is not v3.x compatible code, so it probably has no chance to be merged before at least v4.2, but I'd like this PR to be reviewed (or we should reconsider our new review policy). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's fine from a technical perspective. I agree we definitely need somethin like this.
However I'd much prefer if we fix things from the tagging side of things first. Your solution is probably the best we can do now, but it it's lengthy and won't handle all cases well. Can we not propose a commercial=yes/no tag snd rely on that one?
Such an ill defined tag directly used for a major rendering decision just cries for abuse. I am unsure about if this change is a good idea but it is certainly not the worst possible approach. Making some effort to add missing ICAO/IATA codes together with such a change would definitely be advisable - there are many remote airports where this is missing. Maybe the wikipedia/wikidata guys could use this opportunity to do something more useful than adding arbitrary language name tags 😉. |
In my understanding, commercial aviation is a formal term (as opposed to general aviation, which might also operate commercially). But I might be mistaken. Would be best to have an aviation expert helping us. Anyway that's something gor tagging@. |
Thanks for your responses! I was thinking for a long time what features could make some airports "major" and found no such thing. I was proposing "range" (like international/regional), but IIRC that was rejected. The closest thing to determine how important it could be is aerodrome:type. It has no wiki definitions, but here we have "private", which probably is close to "commercial" (just guessing). At least we already treat private parkings as less visible, so this would follow the same rule. BTW: could anyone of you tell me how to add aerodrome:type to our SQL queries? It didn't work the same as iata and icao tags, probably because of the colon. I think access=private still makes sense, but the type is more airport related and quite popular, so I'd like to use it. |
My long standing suggestion has been to use the runway length. This is a fairly good and objective measure of importance - even if military airports with long runways are not significant for travelers they are usually still important for the map in general. The real charm of this would of course be when using relative importance based display (see http://www.imagico.de/map/osm_populated_en.php) which would only be feasible with preprocessing.
|
I remember this proposition and also that it was rejected too with some examples. What's more important, I don't think we have such tag currently in use, let alone being documented - or am I wrong? |
Runway length would not be tagged, it would be determined from the separately mapped runway via spatial query. There were opinions against using that but i don't think there ever was a real decision. |
There was no definite decisions, so we have to choose anyway. Do we want to merge this PR or reject it? Do you plan to provide the code with runway length? If yes, I would like to test it how it works. If no, I prefer to use this code. |
Note i did not suggest to reject this, i just said i am not sure. No, i don't plan to try an on-the-fly runway length based implementation - if i look into this matter in the future i will probably concentrate on a preprocessing solution which i find more interesting cartographically. |
Sure - I just try to make well informed decision. With lack of "commercial" tagging and with runway length not being the ultimate solution for you (and with no code, which would be non-trivial), I'm now inclined to use this code when the merging window opens. However if there will be some new arguments and ideas regarding airports rendering, I will take them into account. |
sent from a phone
On 10. Jul 2017, at 11:45, kocio-pl ***@***.***> wrote:
I was thinking for a long time what features could make some airports "major" and found no such thing.
some relevant properties could be the number and size of runways, the number (and "rank") of different destinations and carriers and number of flights/passengers per year. ;-) The runways are often available in osm.
|
👎 for any overcomplex solution such as runway length calculations. |
Could you provide a code to test it? At this point I prefer something we could use in practice in the near future, general ideas discussion can continue at #1143. |
Nothing beats Indiana, but in Toronto area Metro extract this code reduces number of airport from about 24 to just 3 (and all of them are located in the city), so this simple approach to prioritize airports proved to be surprisingly effective. |
Generating list of missing ICAO/IATA codes that are listed in Wikidata should be fairly simple (no idea how many new ones will be found). Tricky part is that someone would need to manually verify whatever produced matches are OK and add them. If someone would be interested in using such data I may generate them (I will not do it blindly, we already have many well done unused QA sites). |
Thanks for picking up on this. I don't think additional specialized tools are really needed here. As you said this should be manually verified anyway (both for data accuracy and to avoid possible license troubles if data in wikipedia/wikidata comes from questionable sources). I don't know what tools exactly those who add name translations in OSM based on wikidata use but i would assume these are also suited for adding ICAO/IATA codes. |
I am also not sure, and googling revealed nothing relevant (I know about some people who make this kind of edits manually). I already have basically equivalent. Iinitial reason for writing code was completely different, but in its current state it requires the same amount of additional configuration as mentioned name adders (what I will do if someone will use produced data). |
Thanks for this, it does make the map nicer! |
It looks like highly possible, but it's easy to test it - just try rendering this area with removed station rules. |
Resolves #1143.
Simply moves aerodromes with access=private or lacking any of ICAO/IATA codes from z10-z14 to z11-z14.
I was not able to properly add
aerodrome:type
tag toproject.mml
queries, but it would be good to filter also private types here. Still the effect is good enough for me:Full London area from Mapzen extract at z10 (click to see unscaled images):
Before
After
Full Indiana area from Geofabrik extract at z10 (click to see unscaled images):
Before
After