Hey mate, verifiability is an important concept for OSM as documented at osm.wiki/Verifiability. Part of this means that subjective opinions are best not mapped and instead we stick to objective truths we can verify on the ground. bicycle=stupid sound quite subjective and it's going to work out better to tag things like maxspeed, shoulder, lanes etc which bicycle routers can use to assess routes for different types of cyclists. I'

wow big changeset, but looks great!

Hi could you explain the reasoning for changing the maxweight tag? The one you've changed it to I couldn't find any documentation for and I think the standard maxweight=* tag is correct here. From my understanding this is a general weight restriction hence Key:maxweight.

I could still see the traffic lights at https://www.mapillary.com/app/?focus=photo&pKey=mF63TVvIODK5s81GCEBfBg&lat=-33.86163055555556&lng=151.2105472222222&z=17&x=0.6745173514626654&y=0.5276029931131762&zoom=0.024237851712115975 but that imagery is old and they could be in the process of removing them. I'll revert the change but leave in a fixme note for it to be checked (not urgent).

Great work finding out what these were!

I think you left in the traffic_signals tag by accident in https://osmlab.github.io/osm-deep-history/#/node/7728291989 and https://osmlab.github.io/osm-deep-history/#/node/7728291990 so I've fixed that up now.

Agreed, if you're happy to revert this that sounds good to me. If you could just post back here if it get's done?

hi, the fixme here is asking to check which kind of sport is played here sport=*

hi in this case the name applies to the whole aquatic center and not just the pool so it should go on way/172942441 the other tags are good!

hi the building for this already exists at way/335696957/history so I've moved your added building tags to the existing building

looks good

I've moved this tag to the way with tourism=hotel

I've rolled this one back in changeset/88198083 to retain the name on the main building footprint way to avoid duplication.

I rolled this back in changeset/88198008 to avoid the duplication in names

hi these buildings are already named on the footprint way eg way/423824984 I don't think this needs to be duplicate to all the parts.

looks good

There are two styles of doing traffic lights in OSM, 1. is place the tags on the intersection 2. is place the tags at the stopping position (for each road).

Looking at the change at https://osmcha.org/changesets/88190521 this now looses the pedestrian crossings which were mapped before, so probably the best way to model this one is a single highway=traffic_signals where the ways intersect and then where the 4 existing nodes were have highway=crossing + crossing=traffic_signals.

FYI I moved node/7727922877/history from on the bridge to underneath, pretty sure this is on the ground level not on the bridge https://www.mapillary.com/map/im/5-KSydn8Uy2RgITqge3BFA

Hi, the tourism=viewpoint tag is for scenic lookouts, it looks like the footway is already mapped here so I've reverted this change.

Hi was this mass edit discussed somewhere first?

Agreed with what ortho_is_hot said, the crossing would only be bicycle=designated if there is a bicycle light usually next to the walking/stop man light.