aharvey's Comments
| Changeset | When | Comment |
|---|---|---|
| 90531971 | hi there, when mapping things like tunnels if you could try to avoid having these share nodes with above ground features unless they should be linked, eg. you can see at https://osmcha.org/changesets/90531971 that you're change as a number of shared nodes now. So eg if someone dragged a building above ground that would cause the underground rail to move because the nodes are snapped. If using the JOSM editor you can hold Ctrl while placing nodes to avoid snapping, there may be a way to do similar in iD, I'm not sure. |
|
| 90403765 | ping @Maradona11 not sure if you're interested in these changes, you can inspect in osmcha, eg https://osmcha.org/changesets/90403765 plus another change today at https://osmcha.org/changesets/90411239 |
|
| 90397056 | note=* is for other mappers, description=* is for end users of the map, What you've placed in note seems better fit for map users to be aware of hence better to use description.
|
|
| 90385274 | FYI I just created the proposal to try and formalise the existing tag for a rock overhang shelter see osm.wiki/Proposed_features/Tag:shelter_type%3Drock_shelter |
|
| 90297533 | also if you're tagging different parts of the buildings with different hights, but still part of the same building, then for the smaller inside parts it's standard to use building:part=*, eg. building:part=yes and then leave the building tag for the outside way which is for the footprint. |
|
| 90297533 | 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 | 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 | 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 | 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 | Thank you. |
|
| 90209436 | sure I see now this predated your changes. Thanks. I'll add a note here for someone to check it. |
|
| 90223764 | 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 | I've updated the tags. |
|
| 90223764 | 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 | 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 | 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 | 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 | way/842043479/ is better mapped as highway=raceway + sport=karting see
|
|
| 90067862 | 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 | 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 |