aharvey's Comments
| 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.
|
| 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
|
| 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
|