relation/10207312 again isn't needed since there is only a signposted only left turn which is already mapped.

relation/8515478 isn't needed as you already have the no right turn restriction https://www.mapillary.com/map/im/I1AgsYQT-AX6atqvTO0D3g

way/738103276
I think you meant "public_transport" not "public_tra" ;-)
same for the second one you've added.

relation/10207320
relation/10207319
^ what are these two for?

Yes anything signposted as dismount should be tagged as bicycle=dismount see osm.wiki/Bicycle#Bicycle_Restrictions and access=*#List_of_possible_values

I was once riding in the opposite direction so never saw the sign and got told off by an elderly man that I should dismount.

Most bicycle routers should still route you through bicycle=dismount, but it will probably apply a speed penalty. Either way we need to map this accurately to reflect what's on the ground.

Access tags like bicycle= and foot= are completely separate things than what the feature is highway=footway, highway=cycleway.

2. Also for https://osmlab.github.io/osm-deep-history/#/way/724653636 you removed footway=crossing but according to footway=crossing that tag is used to mark segments of path which are crossings, which this segment is, so it should have that tag.

I know I use it all the time too, even though it's not signposted for bicycles.

With respect to https://osmlab.github.io/osm-deep-history/#/way/629142495 last time I was here there was a bicycle dismount sign -> https://www.mapillary.com/map/im/UnMwUempYTkOa1uAti79vg

Is that sign no longer there? As you changed this from bicycle=dismount to bicycle=designated.

Oh yeah I missed that. Yes that implies that it's some kind of bicycle infrastructure and bicycles are allow. Thanks for confirming that.

In your change you've made Arthur Street continue further along past the intersection but according to the signage on the ground https://www.mapillary.com/map/im/Sd8o0YzS4Az6az1JTMaE5Q the road name changes at the intersection, so I've reverted this change.

Also on the footbridge I couldn't see any signage that says it's a cycleway.

Yeah I can see this was only you're second edit. I certainly don't want to discourage you from more editing, please keep it up.

It's best practice to map what's on the ground osm.wiki/Good_practice#Map_what.27s_on_the_ground so generally what other maps say doesn't matter much compared to what information can be determined from on the ground.

According to https://www.mapillary.com/map/im/c0t-T4YQyaBPZt5Hugnj6g the pedestrian and bicycle lanes are segregated, but not physically separated, segregated=*. Has that changed recently or is the on the ground the same as that older Mapillary image? If it hasn't change recently we'll need to revert this back to a single way with segregated=yes.

This looks good, according to https://www.mapillary.com/map/im/eH7jFyeMbYDtnxlBUu6fgw it is physically separated so it's good that you've mapped it with a separate way. You could also update the road https://osmlab.github.io/osm-deep-history/#/way/34282299 to remove the bicycle tags which aren't really correct as it's not a counter flow cyclelane, it's a separate path.

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