ps opening_date=* is only for future dates, now that it's open it should be start_date=* per the wiki.

no worries, trust you with the local knowledge then. It was only back in my uni days that I did the occasional lap around here.

okay if you know what's on the ground, I thought that they closed off all these roads to cars because of all the extra cyclists out during covid.

but if these are no longer open to motor vehicle traffic, we should at least add motor_vehicle=private.

Hi Russell, that makes sense. Certainly I'm not saying you're wrong, I guess this was just different to how I'd seen it mapped in other places and how I'd interpreted the wiki. But after reading that thread I can understand where you're coming from. So happy to leave it like this then, but interested in hearing what others think and seeing more examples of how other places map.

For example see https://osmcha.org/changesets/87368583 which put a building=* as the footprint and then the towers inside it with different heights as building:part=*

Though same applies to way/335699134 it would be the footprint so building=* and then the parts way/618221458 and way/618221457 would be building:part.

^ talking about way/16748116/history

hi could you say a bit more on why this should be building:part? According to osm.wiki/Simple_3D_buildings#Building_parts typically the building footprint would be building=* and then if you need to add parts which have different physical characteristics then you'd add extra building:part=* ways inside. In this case which building=* is this inside? The way itself already has building=* so I don't see the need for building:part.

+1 to the ground survey to verify. But please don't use Google Street View, otherwise your edits may need to be reverted, see https://help.openstreetmap.org/questions/710/can-i-use-google-streetview-to-help-create-maps on why we don't use Google Street View for mapping in OSM.

hey @speedmeup, we can't just assume that the TfNSW Speed Zones data is 100% accurate, so it should only really be used as a guide to address areas which OSM may be wrong, or to add in missing speed zones. So it would be wrong to just change everything over blindly to match their dataset.

Hi did you really mean to move https://osmlab.github.io/osm-deep-history/#/node/3571292104 so far or was that an accidental drag?

Was there a reason for removing ford=stepping_stones from https://osmlab.github.io/osm-deep-history/#/node/2491748968 ?

Hi, welcome to OSM. Just a tip generally the name tag is used for proper names, not to describe things osm.wiki/Good_practice#Don.27t_use_name_tag_to_describe_things.

So usually XXX Fire Trail, is there a signposted name? If not you can just omit the name tag.

Simply motor_vehicle=private is a much more common practice where only authorised vehicles like emergency services and maintenance workers can drive access=*#List_of_possible_values

access=permit is used commonly but not documented, so I can only guess it's more applicable where you can anyone can apply for a permit?

Hi welcome to OSM. Could you please provide your reasoning for this change? leisure=nature_reserve says "A nature reserve is a protected area of importance for wildlife" which this aquatic reserve seems to match. This can be tagged together with the protected_area tagging, which covers more detail than nature_reserve. What's area=yes needed for? The fact that this is a closed way indicates it's an area already.

Hi per railway=station#A_Simple_Railway_Station and public_transport=stop_position the stop_position can still have the station name, so I'll add that back in.

Yep I just noticed and fixed the building tag.

I've just now restored the retaining walls you added, so the pedestrian area change you made was reverted + the area to node was reverted, but retaining walls added was retained, was that everything?

I've reverted this change until it can be discussed. Personally I think the area is better as it covers the whole station area. It's more consistent with all the other metro stations and eventually we'll get around to converting all train station points into areas.
railway=station#A_Simple_Railway_Station indicates it's reasonable to map as an area.

Hi just a few tips, junction=roundabout implies oneway=yes so there is no need to add the oneway tag junction=roundabout.

Also you deleted the cycleway:left tag and added cycleway:right both =no. Is there ever a case where you have a cycleway on the right side (inside) of a roundabout? Do you need to say =no here? https://osmlab.github.io/osm-deep-history/#/way/171015278