OpenStreetMap logo OpenStreetMap

Changeset When Comment
90297533 over 5 years ago

Okay please don't tag them as highway=steps then if they are not steps there. You can just save your edits locally until you are ready to commit the change, or if you still want to upload it then leave the ways untagged and maybe add a note= tag so explain that you're still working on it.

90297533 over 5 years ago

hi what do you mean by temporary markings? Do you intend to delete them later? Are these existing on the ground, are they on the ground level?

90255459 over 5 years ago

actually looks like this was a duplicate of what was added in changeset/90249432 so I deleted the duplicate to keep just one instead of overlapping ways.

90255459 over 5 years ago

I updated trail_visibility from 0 to "no", "no" is a much more common and accepted way to say it's pathless https://taginfo.openstreetmap.org/keys/trail_visibility#values

89859138 over 5 years ago

Thank you.

90209436 over 5 years ago

sure I see now this predated your changes. Thanks. I'll add a note here for someone to check it.

90223764 over 5 years ago

I see now this was an existing turn restriction, because you deleted the existing one relation/11155539/history and added it as a new one relation/11566490/history. It's good practice to retain the history of objects osm.wiki/Good_practice#Keep_the_history which I know that's not always possible and I'm being pedantic here, but it would have helped me understand the change better if retained the history.
I think leaving the restriction in place is fair enough without local knowledge either way, so I've drop a note here for someone local to review it note/2331888.

90104041 over 5 years ago

I've updated the tags.

90223764 over 5 years ago

What was the source for the left turn only restriction in relation/11566491/history ? From Mapillary I can't see any indication of such (but also can't see any indication that there is none) https://www.mapillary.com/map/im/hUPzG2Q1Jxzvaq35UKk_FA

90209436 over 5 years ago

the source is listed as Maxar but the imagery here is not high enough resolution to make out changeset/90209436 what is your source for adding this? Is it a roundabout or not?

89859138 over 5 years ago

way/840189281/history should inherit all the applicable attributes like name surface etc from the road segment before you split it. I think there may be other issues like this if you could please fix those too.

89923645 over 5 years ago

way/840589945/history should have the road name that was existing before the split, there are other places this happens to if you could please check those too.

90104041 over 5 years ago

way/842043479/ is better mapped as highway=raceway + sport=karting see
highway=raceway

90067862 over 5 years ago

Hi Stephen, for https://osmlab.github.io/osm-deep-history/#/way/597954684 and https://osmlab.github.io/osm-deep-history/#/way/697517823 given https://www.mapillary.com/map/im/ERbSnkfk_IKtrH1acdXD6w shows that it is signposted as for both bicycles and pedestrians, then bicycle=designated is more correct as according to access=*#List_of_possible_values bicycle=yes just means you have legal access whereas bicycle=designated is a stronger form saying both you have access and it's explicitly signposted as for that particular mode of transport.

90019941 over 5 years ago

hi just a tip that it's good practice to reserve the name tag for proper names, so if the house or building has a signposted name that could go into the name field but otherwise street address etc goes into other tags like you have and not the name field. see also osm.wiki/Good_practice#Don.27t_use_name_tag_to_describe_things

90009628 over 5 years ago

thanks. I just made a few extra minor tweaks to try and place the start point more exactly per https://www.mapillary.com/map/im/F5QTIcNn8UEL3Xpf44_HYA

89999036 over 5 years ago

I just checked your recent changesets, looks good now. Thanks.

89999036 over 5 years ago

Hi Ase83, no problem, if you need a hand with reinstating just shout out. It's much easier to do with JOSM there is both an undelete single object option and revert whole changeset option. Then after it's reinstated you can just update the footway to a private driveway and even add a node where the gate is.

89999036 over 5 years ago

Agreed with nevw here, private residential driveways can be mapped in OSM, so it's not right to delete them if they exist on the ground. You also deleted the footway way/174520032/history could you explain why that was done too please?

89954767 over 5 years ago

hi Stephen,

Since you've requested a review, just a comment that "self" no u-turns where it's from one road back onto itself like you added in
relation/11546114/history
relation/11546115/history
relation/11546118/history
relation/11546116/history
generally are not needed (it's already assumed you can't u-turn like that) and unless it's signposted as such usually these shouldn't be mapped.