aharvey's Comments
| Changeset | When | Comment |
|---|---|---|
| 155917852 | over 1 year ago | some discussion osm.wiki/Talk:Tag:highway%3Dcrossing#When_should_a_sidewalk/service-road_crossing_be_tagged_with_highway=crossing? |
| 155917852 | over 1 year ago | My understanding is that highway=crossing is for street crossings per highway=crossing and should not apply if a footpath simply crosses a driveway. In this changeset I see a few instances of highway=crossing be applied on the junction of a highway=service and footway=sidewalk where that the service road is actually a driveway as opposed to a road like a laneway. I'm planning to fix these up, but maybe something to consider for your workflow? Are you applying an particular logic for identifying these missing crossings? |
| 67379973 | over 1 year ago | Given these roads just service small block residential houses, and don't connect to anywhere else I feel they should be classified as highway=residential. Do you think this is okay? |
| 124139461 | over 1 year ago | are you sure the school zone was removed at way/755189316/history#map=19/-33.878906/151.236409 ? From all the imagery sources I can see it's still there. |
| 137478235 | over 1 year ago | Thanks for your edits here, though I noticed you've used a lot of multipolygon relations eg. the parking areas, which I feel make editing more complicated. Is there a reason these couldn't just be ways which share nodes so the boundaries are snapped together? |
| 155537275 | over 1 year ago | Just looking on their website
|
| 155537275 | over 1 year ago | hmm admittedly I made this change based on the towers being complete, which I was thinking shouldn't be marked as construction if complete. Based on your photo I think we should at least convert way/1068760239 back to construction landuse. Do you think that works? |
| 152483789 | over 1 year ago | hi northchun, you'll find that in Australia, common practice has been to map out roundabouts as a circular way and tag them as junction=roundabout. I realise this does conflict with highway=mini%20roundabout?uselang=en but due to this longstanding practice you should consult the community before changing things around. Regardless the style of mapping you've used here adding a single node not connected to the highway=* way is incorrect and would cause validation issues. |
| 155731884 | over 1 year ago | Are you sure this is not signposted for use by bicycles? Because this was previously tagged bicycle=designated (explicitly signed for bicycle use) and this changed it to bicycle=yes (bicycles okay to use but not designated as for bicycles).
|
| 152107545 | over 1 year ago | You also removed the ferry route from the way to Palm Beach https://osmlab.github.io/osm-deep-history/#/way/247939475 since then TfNSW have re-added it, but now the two tags conflict, so I'll remove the intermittent:route=ferry tag you added based. I agree that I believe Brooklyn to Patonga is water taxi only. |
| 152107545 | over 1 year ago | Is this not serviced by https://www.boathouseferryco.com.au/timetable? |
| 153875079 | over 1 year ago | I feel this is a bit unnecessary to split the way, it complicates the data model when a simple tag on the way to say there is a median barrier would have worked. But regardless the lane tagging was not correctly updated, which I've now fixed. |
| 153643977 | over 1 year ago | Please take care moving features when editing with iD as it will snap unexpectedly. node/3018096089/history#map=19/-33.81230/151.17152 was incorrectly snapped to the tunnel in this change. I've fixed it now. |
| 91168597 | over 1 year ago | Given these are not current military installations, only historical I believe we should use one of the lifecycle prefixes osm.wiki/Lifecycle_prefix |
| 154875295 | over 1 year ago | I've restored node/2921282680/history which was deleted here, and merged in your changes, it's important to try and retain the history of existing objects when you are making improvements to the data. |
| 154727924 | over 1 year ago | That syntax does conform to the proposal at osm.wiki/Proposal:Defaults so I'll change it. |
| 154727924 | over 1 year ago | What about if we did something like this? |
| 154727924 | over 1 year ago | It doesn't. It tries to make some assumptions that mostly residential roads are controlled by the LGA and will mostly be 40km/hr. However we know that might not always be true so best to always explicitly tag maxspeed on each way. As I understand it, these tags for setting the default aren't implying that there's never exceptions. Do you think we shouldn't tag a default residential limit for City of Sydney in this way? |
| 143889108 | over 1 year ago | "data I've held for a long time originally obtained from the DMR/RTA in the 1980s" is likely not a compatiable source for use in OSM due to copyright. We have a license to use the open data from https://data.nsw.gov.au/data/dataset/1-52821edb8608470abf117897b6ef7385 which may be used. That said if planning to bulk import these, it's best to discuss your plans with the community first for feedback. |
| 153776297 | over 1 year ago | This changeset has been reverted by changeset/154001143 |