aharvey's Comments
| Changeset | When | Comment |
|---|---|---|
| 167896191 | 7 months ago | how can you tell from imagery that these are all water storage, could they possibly be other kinds of silo? eg. for grain? |
| 167900568 | 7 months ago | + it would be more accurate to use source:geometry since actually the source was the person who mapped it as a node prior to you. You can also leave off the source tag and instead rely on the changeset level source. |
| 167900568 | 7 months ago | Please ensure you retain any other tags on the node, in this case it had historic=yes. |
| 167900656 | 7 months ago | some information was lost here, before we had address 86B and then address 2/68 but you've replaced both with just 68B. If you're not sure which applies to what best to leave the two address nodes as they are and only move the building=yes tag along to the building. I've fixed this up. If you're just working remotely probably best to leave the addresses and just change the building=yes tag from the node to the new way. |
| 167554132 | 7 months ago | I've updated this in changeset/167911590 |
| 167700842 | 7 months ago | I've reverted this change. |
| 167769850 | 7 months ago | I've changed the network and operator based on my prior comment. |
| 167769850 | 7 months ago | I've restored the prior route names. |
| 167764812 | 7 months ago | I've restored the prior route names based on community feedback at https://community.openstreetmap.org/t/public-transport-route-names/131624 |
| 167764029 | 7 months ago | I've restored the prior names based on community feedback at https://community.openstreetmap.org/t/public-transport-route-names/131624 |
| 167774788 | 7 months ago | I've restored the prior name based on this being the better route name over the artificial route name from PTv2 per community discussion at https://community.openstreetmap.org/t/public-transport-route-names/131624 |
| 167848114 | 7 months ago | I've restored the proper name "Dubbo XPT" based on this being the better route name over the artificial route name from PTv2 per community discussion at https://community.openstreetmap.org/t/public-transport-route-names/131624 |
| 167858004 | 7 months ago | I suspect the lanes:conditional should also be increased to 3 as likely it only changes by one lane during the AM lane shift. I'm not sure though. |
| 167858133 | 7 months ago | if there's no arrows on the road or other road signs then we shouldn't be setting turn:lanes per turn=* it should only be used where there are road markings or sign markings indicating the lane guidance. |
| 167851551 | 7 months ago | way/1396730552 turn lanes are not modelled like that, we only draw a separate way where there is physical separation, if there's just paint on the ground it's best to use the turn:lanes tag turn=*#Indicated_turns_by_lane to specify the turn lanes. The existing way at way/40427397 already allows for routing from Spencer Street turning right into Gatton Showgrounds. |
| 167851551 | 7 months ago |
I'm not sure about that one the routing will already let you turn there like this osm.org/directions?engine=graphhopper_car&route=-27.558516%2C152.278678%3B-27.558694%2C152.278799 usually we wouldn't model the intersection like that unless there is a large traffic island in the middle of the intersection. |
| 167851551 | 7 months ago | The ways which join the road to the parking lots (eg. way/1396730532) should be "Driveway" (highway=service + service=driveway) not "Unclassified" (highway=unclassified). As a rule of thumb if you drive up and over the footpath it's a driveway. For example this is unclassified way/151689579 |
| 167848114 | 7 months ago | https://community.openstreetmap.org/t/public-transport-route-names/131624 if you had any thoughts on the matter if you could contribute to the thread please. |
| 167848114 | 7 months ago | hi could you please reply to my earlier changeset comments about changing the route names? I might ask for wider community feedback, if you could hold off further changes until we get community feedback? |
| 167812155 | 7 months ago | Thanks for confirming. |