OpenStreetMap logo OpenStreetMap

Changeset When Comment
76223837 about 6 years ago

Thanks. Agreed with @SK53, I've used disused for now, once the chains have been removed can be updated to demolished or removed.

76213250 about 6 years ago

Okay I've marked this as a common pending a more information and updated the etymology tags in changeset/76279583

57608131 about 6 years ago

>robe data isn't enough to justify this, I checked on the ground and there is no signage or physical barrier indicating this restriction.

56476313 about 6 years ago

In this changeset it looks like you changed Garigal Road from a single way to two parallel ways (dual carriage), however lacking a physical barrier for most of the road it's better mapped as a single way, I've changed it back.

76213250 about 6 years ago

Hi, It looks like the wikipedia/wikidata links are for the person this is named after not the actual square. In that case see name:etymology=* for the correct tags to use.

Also because you're lacking a physical tag, can you say anything more about what this is? Is it a pedestrian area, a showground, a theatre or just an open grass space?

76176791 about 6 years ago

Ha I've seen tagging for the renderer, tagging for the router, but never tagging for the editor UI in this way before. I think it's better addressed by the editor UI being smart and generating a name like this based on the relation type.

I mean it's not harmful how you have it, since the name tag shouldn't really mean anything in this context.

76174909 about 6 years ago

Thanks, I updated one of these restrictions to simply except=bus instead of except=psv as per the sign https://www.mapillary.com/map/im/9mbQx7OPz81_3dFNfvv2fQ this only excepts busses not other psv vehicles like taxis. Is that right?

76176481 about 6 years ago

Oh yeah please feel welcome to keep up with your contributions.

In this case, even though bicycles are allowed here, it's still a footpath on the ground from a pedestrian perspective. We have this tag footway=sidewalk which says this footway is a sidewalk since it runs alongside the road, it's useful for some downstream data users.

So just because bicycles can also ride here, doesn't change the fact that it's a "sidewalk" footway. I see now you've added that footway=sidewalk tag back in thanks.

However at https://osmlab.github.io/osm-deep-history/#/way/530746983 you've added cycleway=sidepath, was that something you found documented somewhere? footway=sidewalk should be enough.

76176481 about 6 years ago

Yes but for a pedstrian it's also still a footpath, in OSM footway=sidewalk footway=sidewalk=* is used to map footways which are the sidewalk.

76177014 about 6 years ago

Do you know why way/246596679/history was deleted? It's best practice to try and retain history where possible? So generally it's always best to reuse existing ways rather than deleting them as we loose the history of the object when it's deleted.

76176791 about 6 years ago

osm.wiki/Relation:destination_sign doesn't indicate what a name tag on the destination_sign relation means, but the name you've used here seems more like a description than a proper name. Are you sure they need a name at all?

76176481 about 6 years ago

https://osmlab.github.io/osm-deep-history/#/way/530746983 is still a pedestrian "sidewalk" so can you re-add that tag that you deleted back in?

76176736 about 6 years ago

What were you trying to change here? The tunnel was already mapped and it seemed your change moved the surface tags to the wrong place, so I've reverted this.

https://osmcha.mapbox.com/changesets/76176736

76172622 about 6 years ago

+1 from me. You could also use opening_date=*

76128662 about 6 years ago

Ok I've added cycleway=crossing back into https://osmlab.github.io/osm-deep-history/#/way/724653636

76127937 about 6 years ago

Okay I've changed this back and made a few more updates from the latest Mapillary images.

76132261 about 6 years ago

Oh yeah the crossings you added were good, thanks for that, it was more the turn restrictions which I didn't understand the intent behind.

76132261 about 6 years ago

I've fixed these in changeset/76133836. there were some conflicts when I made the upload, looks like you already started deleting some.

Would be good to understand what you were trying to do here?

76132303 about 6 years ago

I've fixed a bunch of these in changeset/76133836

76132261 about 6 years ago

also not sure what relation/10207313 is