-
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
Add rendering for emergency=phone #318
Comments
The biggest problem with this is that we do not store the emergency tag for osm carto. It is a fairly major database change to add new tags. I imagine this will not be fixed any time soon despite the usefulness/need for this. |
If people are doing mechanical edits to change tags on the basis of "depreciated tags", that's wrong. OSM doesn't have a process for depreciating tags. |
Adding something to list of deprecated tags in validator (especially with an automatic fix) is exactly this. |
@pnorman, |
2014/1/20 Rovastar [email protected]
not remotely as quick as with a bunch of mappers doing large scale |
+1 on a fix for this: both in carto and via a mass retagging to centralize on one emergency phone tagging. I mapped a phone just yesterday and noted it did not show up on mapnik. |
mass retagging is never a good idea because it affects each and every user of our data (think about iPhone or android apps, web-searches, all the different map-styles, all database imports on all servers). All of them need to change their data-processing or their database structure and may require a complete planet re-import. We should not honor such mass retagging by adopting the style early. After all we're also suffering from the same problem. |
Passive-aggressive "no honoring of mass retagging" is useless. It is either overall change for good and change of rendering rule is also a good idea, or it is a bad idea - in this case complaining people should start from https://josm.openstreetmap.de/newticket IMHO this was a correct idea to migrate from absurdly bloated amenity tag as soon as possible, before it will be nearly impossible but it is probably topic for other discussion. |
For what it's worth (whatever your take on the retagging may be), emergency=phone currently outnumbers amenity=emergency_phone by 3 to 1. Supporting the emergency tag in osm-carto would also enable rendering the 186000 emergency=fire_hydrant (althoug I'm not sure we want to render something that the general public is not allowed to use), 1700 emergency=aed/defibrilator (that one seems very worthy of rendering), and a few other values. |
fire_hydrant would make sense after introducing additional zoom levels, aed/defibrilator makes sense but there is ongoing tagging debate on this topis. |
Surely, as described, this isn't a bug with the map style at all? If JOSM is suggesting mass retagging (based on a wiki page?) then that's a bug with JOSM, and if JOSM users are making automated edits without going through the documented mechanical edit procedures (or it seems sometimes engaging their brains at all) then that's a "bug" with those mappers. |
@Bulwersator wrote 'mass retagging is never a good idea because it affects each and every user of our data'. The advantage of mass retagging is data consumers will quickly figure out that something changed. The downside of course is disruption. Avoiding mass tagging results in a slow rot of the old data, leaving (for example) maps showing half the emergency phones. Clearly neither extreme is ideal. But I rather prefer the retagging, as (post disruption) it puts the map in a better state. |
Changing tagging standards is a general problem with OSM. I think the underlying cause of this is that we cannot guarantee (and do not require) that people upgrade their rendering/routing rules whenever they update their database. Therefore, it must in theory be possible to render every version of the database with every version of rendering/routing rules. Perhaps a way to work around this problem:
So basically the idea is to propose new tagging schemes that only start being valid 6 months in the future. |
2014/1/23 brycenesbitt [email protected]
well, they could evaluate both tags (with the disadvantage that both tags There is also an example that this might work out on the long run: |
The lag in the rendering support is also a factor. There's pressure to use obsolete rendering so things show up. amenity=ranger_station is a tiny example, bigger examples abound with shop= where there's pressure to pick a rendered shop tag even if it's not the most descriptive. (See #116 ). A moderated one-way announce list for data consumers might help. Or an RSS feed of tag changes. But that said mass retagging is a form of notice to data consumers, in a way that gradual re-tagging is not. Anyone doing fuzzy matches on tiger:tlid for example is seeing the data slowly melt away, rather than facing the issue up front. |
I hoped that both milestones may be set. Anyway, I think that this one is more suitable. It is not really a new feature, also "3.x - Needs upgrade to mapnik or openstreetmap-carto.style" is more restrictive. |
The old, deprecated "amenity:emergency_phone" are shown on the map, What's the big problem, why does that take all year ? Also, when zooming out, the icon for emergency-phone is no longer shown at zoomlevel 16, |
Please open a separate issue if you think that emergency phone should be shown at lower zoom levels with some example and explanation.
The big problem is that the tag |
Is this going to be done anytime soon? |
As indicated above, it requires a database reload before it can be considered. |
Maybe we should start double tagging them, like we have to do with public_transport for the coming 20 years. |
|
@maxxer This is offtopic in this issue. Please comment in existing issue about emergency=defibrillator or maybe create new if there is no issue on this topic. |
To be fair to @maxxer , they were sent here by a rather terse answer at https://help.openstreetmap.org/questions/41537/why-aed-defibrillators-are-not-displayed-on-maps and the root cause for the problem is explained above at #318 (comment) . |
JOSM validator complains about amenity=emergency_phone being deprecated and auto-corrects it to emergency=phone when using the "Fix" button. However, the latter is not rendered by the carto style, thus all emergency phones disappeared from the map in some areas [1] recently.
I'd suggest to also include emergency=phone with the same render rule as amenity=emergency_phone in the carto style.
[1] Example for a recent mass update turning amenity=emergency_phone into emergency=phone: http://www.openstreetmap.org/changeset/19082658
The text was updated successfully, but these errors were encountered: