Keep in mind that highway=cycleway is for a physically separated path, if just separated from the road by paint, it's a cycleway=lane tag on the road way, not a seperate way as highway=cycleway. I know it can be hard to model this one though since it's quite a complex junction and changes between physical separation, but from the imagery, way/738071033 shouldn't be a separate way as there is no physical separation.

Did you want to fix this?

PS. The placement tag placement=* is used to say where the geometry is, so that when rendered with lanes they align eg. from JOSM with the Lane Attributes style https://imgur.com/ilAgBQx

Hi, could you point out where this path is signposted for bicycles? I couldn't see a sign from my Mapillary photos and bicycle=designated should only be used where signposted. It's a stronger indication of you're allowed to use this way over just bicycle=yes. It certainly looks on the ground like a footpath or generic path that you can cycle on rather than a cycleway or shared path. I'm not saying I'm right here, just looking for more information about your change.

Hi thanks for trying to improve the data here, unfortuantly the correct changes were mingled with some not so correct changes, so I've reverted the changeset and then subsequently re-instated the good changes.

My reasoning is:
1. https://osmlab.github.io/osm-deep-history/#/way/503996837 was unnecessarily deleted, per osm.wiki/Good_practice#Keep_the_history the history should be retained by editing existing features where possible.
2. https://osmlab.github.io/osm-deep-history/#/way/738069132 highway=cycleway is for a physically separated cycleway see highway=cycleway#When_not_to_use, in this case it's a cycle lane and is tagged as such.

I've used operator:abn in the past, but maybe that should be operator:ref:abn

I moved the remaining tags across from the node to the way now in changeset/76033258 per osm.wiki/Good_practice#One_feature.2C_one_OSM_element

No worries. As I said I'm not too fused either way. Just adding the religion tag is good, that wasn't the issue, it's about where the amenity=place_of_worship tag should be, should it be on the church building only or on the whole grounds. Either way I think is fine, it's just that the wiki kind of implies it shouldn't be on the grounds.

It looks like the amenity=place_of_worship tag is now lacking entirely, so I've added that back in now.

When I walked past here today I didn't see a comma in between "...Church Roseville" on the sign, but I'm just nit picking here.

Please where possible retain the history, I don't see why relation/10162010/history needed to be deleted? You could have just updated the existing relation.

See osm.wiki/Good_practice#Keep_the_history

According to landuse=religious the way I had it before was considered more accepted, but I'm not too fussed either way...

Are you sure this is not more like a stream than a river, downstream is marked as a stream. (the edit in AU)

Overall this is okay, but you should separate out the address number from the street, add the +61 prefix to the phone number.

I've fixed this now in changeset/75746596

I don't think this is right, https://www.mapillary.com/map/im/m4NHRN4FjJz-fPJ4zA8EFw.

This sign is for cars turning into way/24248082 but your from/to is just Henry Lawson Drive to itself.

Hi you asked for a review, so I'll provide some feedback.

Overall it's good, just keep in mind that a name is only something as signposted or otherwise generally known as, not a description see osm.wiki/Names#Name_is_the_name_only. So "Walking Path" is probably not the name, so best to leave it blank.

You've added this tag to every node of the way, it looks like you've used select all in JOSM. Could you please fix this?

I didn't hear back on this so I've updated this again so that name:en is only the English name so that you can distinguish the English name vs. the indigenous name.

I noticed you've added "Old Bells Line of Road" to https://osmlab.github.io/osm-deep-history/#/way/37627194 on a recent survey I found this one named "Newnes Forest Road" https://www.mapillary.com/map/im/k1mhrLMDQy3dwnWHk-fDkg

Do you have any other local knowledge about this road name, I'm trying to decide if I should update it to the signage I saw...

You've added noname=yes to a bunch of notes along the ways eg. node/4073250116/history I assume this was a mistaken select all in JOSM, just a hint to be extra careful when making mass edits like that.

Hey I've fixed the helipad you accidentally moved in this changeset changeset/29445911

I'm planning on cleaning up this intersection to remove the parallel ways which have no separation apart from paint on the ground, to use lanes and turn:lanes. Turn lanes shouldn't be a separate way they should use turn:lanes instead.