aharvey's Comments
| Changeset | When | Comment |
|---|---|---|
| 90009628 | 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 | I just checked your recent changesets, looks good now. Thanks. |
|
| 89999036 | 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 | 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 | 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
|
|
| 89955037 | hi I updated the bounds per the LPI Basemap it looks like you included an apartment building as part of the school, if you have local knowledge if you could confirm that? I also updated the tags of the buildings. |
|
| 89940835 | Hi Stephen, this is good, but a few tips since you asked for a review.
|
|
| 89812377 | golf=tee is the common tag for tee's so I've changed the tag. |
|
| 89804209 | hi you deleted the water areas here, could you please elaborate on why? |
|
| 89825701 | If it's a declared Wildlife Protection Area by the local council you can consider adding boundary=protected area eg way/174172877 |
|
| 89778493 | If you don't use the preset, and just manually enter the tags one by one then it won't auto-add the building tag, otherwise you'll just need to manually remove the building tag if using the preset in iD. |
|
| 89717919 | way/839191190 looks like a building not a tower, so I've changed it and then mapped the satellite dishes as nodes, did you want to take a look and see if that's okay? |
|
| 89717919 | way/677568458/history you added layer=-1 but the building is marker as a roof and it's the highest object, I don't see why layer is needed at all here unless you have footways on top of the roof or multiple roofs. |
|
| 89779719 | I've reverted the change to Bradleys Head Lighthouse, the other changes seem okay. |
|
| 89775375 | No problem making changes, I know accidents like this can happen, so take this as a pointer for next time to watch out for it, and certainly not a discouragement to map. Mapping things from your local knowledge or from compatible sources is fine, if you're unsure about something feel free to ask on one of the community sites (click Australia at https://openstreetmap.community/ to see all the places OSM community is at) |
|
| 89779719 | https://osmlab.github.io/osm-deep-history/#/node/1903566739 there are other tags indicating this is a cardinal beacon, by changing it to light_major it looses information that it's cardinal. Are you sure about this change? |
|
| 89779719 | the lighthouse you added already existed so I've removed the duplicate you added in changeset/89784100 |
|
| 89778493 | The iD editor preset automatically adds building=yes, so where you're marking the grounds you need to make sure there is no building=yes tag, you can then add the individual buildings inside the grounds separately. I've fixed this up for the ones in this changeset in changeset/89783972 |
|
| 89770128 | looks like you accidentally dragged the bus stop into a building https://osmlab.github.io/osm-deep-history/#/node/2626808800 safe to assume that was an accident? I'll move it back. |
|
| 89774346 | what's your source for https://osmlab.github.io/osm-deep-history/#/node/247040994 being asphalt, is it from a survey? from the imagery it looks like it could be grass |