aharvey's Comments
| Changeset | When | Comment |
|---|---|---|
| 172774907 | Bing imagery isn't aligned here, please check the GPS traces layer for alignment. |
|
| 172773153 | Bing imagery here has an offset, which needs to be accounted for if mapping against it, this can be checked via the GPS traces layer. Furthermore I believe service=alley was correct per service=alley due to this I've reverted this changeset. |
|
| 172778203 | ...also chiming in. It looks like you're trying to split roads into dual carriageways (two oneways) where they are physically split but in OSM they were previously mapped as a single two way, but in doing so many of the existing tags were lost and in some instances the original ways were deleted in place of new ways which looses the history. Is best to retain the existing ways for the history. Furthermore enabling the GPS traces here it seems that Bing imagery has an offset and ESRI imagery is better aligned, but the ways have been mapped to match Bing. Because of the lost way history I'll revert the problematic changes, we can continue to work on improvements from there. |
|
| 174258562 | how do you know the building height and building levels from aerial imagery? |
|
| 174187954 | In the iD editor there is a "Driveway" preset which should be used for driveways, the tags should be highway=service + service=driveway. |
|
| 174214161 | hi and welcome to OSM. The building outline should follow the building footprint instead of the property boundary. You can take a look at how other buildings nearby have been mapped to check. |
|
| 173930028 | I'm not sure the reason to remove historic=memorial here? I've reverted the changeset to restore it for now. |
|
| 173930156 | not sure what was wrong with this, but I've reverted this changeset in changeset/174187322 based on the reasons given in the changeset description |
|
| 173956929 | I've reverted this changeset since it appears to be related to a temporary event and per osm.wiki/Good_practice#Don't_map_temporary_events_and_temporary_features we don't map these kinds of temporary events in OSM. |
|
| 174003821 | the boundary=national_park tag is discouraged in Australia per the guidelines at osm.wiki/Australian_Tagging_Guidelines/Land_and_boundaries We generally use boundary=protected_area in combination with protection_title=National Park to indicate the designation along with the protect_class tag. |
|
| 174019497 | it's good practice to keep the history as you edit features osm.wiki/Good_practice#Keep_the_history this means instead of deleting something just to re-add it, it's best to adjust what's already there. this will depend on the situation, but in this case it would be better practice to try and fix the existing buildings. |
|
| 174056939 | It's also possible in JOSM to find the affected dragged nodes, look at their coordinates history and right click over the Latitude/Longitude text and choose "Restore". Sometimes this can be easier than a selective revert where you do want to keep most of the other changes. |
|
| 174180775 | that's not pyramidal, it's hipped roof:shape=* |
|
| 174174851 | building=yes should only apply to the building, if you're drawing the area to cover the grounds/site there shouldn't be a building tag. I think the iD editor adds this by default as part of the preset, so may need to be manually removed. |
|
| 174175779 | I've restored the landuse=commercial tag, there is no other landuse tagged here and I think commercial is the best fit for a childcare based on landuse=* |
|
| 173777381 | the original tree nodes seem to be well mapped and had further details confirmed via a survey so I've reverted this change. |
|
| 173777443 | While in this case I think it's worth retaining tree nodes, they were originally added from imagery and likely weren't accurate anyway (2 nodes added where there was one tree, or vice versa) therefore I've left this change |
|
| 173777486 | I think in this setting the trees can be mapped as nodes and to restore the deleted nodes I've reverted this changeset. |
|
| 173777634 | I've reverted this to retain the original tree nodes. |
|
| 173777574 | I've reverted this change. |