You can also use branch=Milperra to tag the branch.
Also per start_date=* the format yyyy-mm-dd would make this more machine readable.

I couldn't find these toilets, are you sure the location is correct?

This has been discussed before, but I can't find a link currently. I disagree with this change since you loose valuable information about the size and extent of the station, you loose information about when you can considered yourself inside central station or not. So I think this should be reverted.

Thanks!

Whats LPO?

The suburb of Sydneham is already mapped at relation/5698938 as a relation and node/3862413054 as a label node.

What's the reasoning for adding this?

A number of hiking maps do support sac_scale, trail_visibility and route=hiking see osm.wiki/Hiking_Maps.

I agree with tekim that we should refrain from mapping for the render, the name should be the name only.

I use foot=designated to say this is a signposted path for foot traffic (including signposted hiking routes, and "official" routes), and foot=yes for paths that you can walk on, but not signposted or otherwise as specifically for walking ("unofficial" route).

The level tag level=* is something most commonly used for indoor mapping to say which floor of a building something is on.

Per the wiki "For typical street and freeway crossings with bridges, layer=* should be used instead."

Hi it looks like you've set admin_level=1, but it looks like that's for super-national level like the European Union. Country capitials are admin_level=2 and subnational capitals are admin levels 3 and higher.

No worries, based on that photo it looks like the service road is no longer there so all good.

Thanks. Just a tip that while lanes=3 is still okay, it is ambiguous with respect to how many lanes in each direction. When it's an even number you can default to an even number in each direction, but not when it's odd. So in this case it can be helpful to add lanes:forward=X and lanes:backward=X depending on which direction the way is.

also I noticed you've deleted https://osmlab.github.io/osm-deep-history/#/way/702219833 it might have changed recently, so maybe you can confirm but there did used to be a service road here for one of the driveways to exit out onto Bathurst street you can see the arrow on the road at https://www.mapillary.com/app/?lat=-33.87412222222224&lng=151.20692222222215&z=19.58287877887588&pKey=kZzz6HFF5SkfutV5PDjYHw&focus=photo&x=0.28491036517820095&y=0.5103073083519546&zoom=0.870639500132132

highway=pedestrian highway=pedestrian " is a road or an area mainly or exclusively for pedestrians in which some vehicle traffic may be authorized (e.g. emergency, taxi, delivery, ...)". Pitt Street Mall is a good example since it's mostly for pedestrians but during certain hours of the data delivery drivers can use it as a road. Are you sure these small segments of footway which connect the sidewalk to the platform are best mapped as roads or should they simply be highway=footway?

In my opinion saying the destination:ref here is T2 is not quite correct. T2 means it's a transit lane, ie. requiring two occupants in the vehicle to use it, ie. what other jurisdictions and what OSM calls a hov lane. It's not a destination ref like the highway or road reference. On the other hand I guess you are keeping left and following that transit lane to the left.

If this is your company office, you should also add the tag office=company. office=company

I've added back into the route now.

Hi, welcome to OSM. Thanks for contributing.

In your edit here, keep in mind that tagging something as a building means the geometry should match the building footprint only, not the backyard as well.

If you only want to tag the whole property and not the building footprint then you can just add the address as either a point or a way covering the whole property without the building tag.

Hi could you please explain the reasoning for removing the service tag from the route relation?

According to osm.wiki/Public_transport#Railways service=commuter is the correct tag for urban mass transit systems with short headways (which is what the Northwest Metro mostly is).

PS. I did already had platforms roughly mapped as lines (not as accurate as your data though), it's better practice to reuse those existing ways when improving them rather than deleting them and starting afresh. I know it's not always possible though, but if possible it's better.

For https://osmlab.github.io/osm-deep-history/#/way/173170861 where you've added footway=no what was the intention there?

To say that pedestrians aren't allowed here you can use foot=no (but it's implied already because of access=no), or to say there is not footpath parallel to this road you'd use sidewalk=no or sidewalk=none.

ps to tag a service road as in construction you either need to do construction:highway=service OR highway=construction + construction=service. You've missed the highway=construction here, and in a few other places. I've fixed it now.