aharvey's Comments
| Changeset | When | Comment |
|---|---|---|
| 157028467 | over 1 year ago | Generally OpenStreetMap practice is to "map what's on the ground" osm.wiki/Good_practice#Map_what's_on_the_ground In this case, from the imagery it does indeed look like there are footpaths existing here, seems doubtful the council removed them. See also the documentation for mapping footpaths footway=sidewalk |
| 139586332 | over 1 year ago | Thanks for the reply, I've tweaked it to align with what I can see. |
| 139586332 | over 1 year ago | hi you've extended the school zone much wider than what I can see on the ground, you've mentioned you've based this on a survey and local knowledge, but are you sure there was signage at all the entrance points where you start the school zone from? |
| 156800130 | over 1 year ago | hi, could you avoid hitting save in iD after every single object added? You can map out many objects, then just save at the end to create one changeset. |
| 154693578 | over 1 year ago | Hi, I just wanted to point out that SIX Maps still include some non CC BY terms https://maps.six.nsw.gov.au/js/sixmaps/app/coreTerms.html which we'd prefer to err on the side of caution and not use directly.
|
| 155917852 | over 1 year ago | I did not realise there has been a whole discussion on this point already at https://community.openstreetmap.org/t/rfc-feature-proposal-continuous-crossings/105478 |
| 155917852 | over 1 year ago | To add, eventually we could have every house in OSM with a driveway mapped, do you agree we should have every single one of those driveways crossing a footpath, tagged as highway=crossing? |
| 155917852 | over 1 year ago | I'm not saying driveway crossing is not unmarked, my thoughts are highway=crossing probably shouldn't even apply at all. What I just posted on the Talk page is I guess I see a distinction between crossing a road and crossing a driveway, with a crossing on highway=primary you're crossing the road because you're walking along the sidewalk and need to get to the sidewalk on the other side, but with the intersection of a service=driveway and footway=sidewalk, you're never crossing to the other side of the road, you're staying on the same sidewalk, just you reach a point where vehicles may cross your path, so I don't see this intersection node as the same kind of feature. crossing=unmarked includes driveways as a tricky case and simply acknowledges that sometimes highway=crossing is being used in OSM (it's just documenting usage "as is" in OSM) and also says "if such place would be considered as taggable with highway=crossing". It's true, as a data consumer, I can check the highway types of the intersection and drop these out if I don't want them, but I can also ask, if they are always unmarked and always should be highway=crossing, why even bother at all? The reason I noticed these is they started spamming my StreetComplete quests with all the other crossing tags, none of which should apply. |
| 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. |