Hey I know this is a bit of a late comment for this changeset, but going forward the height should not be zero for streetlights/utility poles, as this is for the actual height of the object. You may have meant that these are level 0 (ie ground level) however that is a different attribute tag.

I had based it on the aerial imagery / the right-in / right-out configuration of the north leg of the intersection. It can probably be undone if there isnt any local signage.

This one was for certain right across the street at Dundas Square. Whether it is the one down south in the right position or an additional one I cannot dallsay for certain as I didn't notice the existence / non-existence of the one to the south

oh I meant off of the main sidewalk way

I think there may be an issue with this changeset with some of the commercial areas being converted to relations. It seems the area boundaries were converted to lines in a relation?

Thank you for catching and fixing your address copying error. One thing I would suggest as you continue is to make use of the Q shortcut to square off your building corners.

There seems to be some duplications here with some of the buildings you added

oh also added a sidewalk and stop signs

I think its a matter of ongoing debate online about what is the "correct" way to code curbs tags on these smaller crossings. From what I've gathered the most important thing is to never have the curb tag on the main sidewalk way itself (as that would suggest that you have to step over a curb to continue down the sidewalk).

Typically in cases where the sidewalk is wide enough to support a 90 intersection between the two sidewalks, I separate the curb tags (similar to the graphic shown in crossing=* or like node/5765177201 if there wasn't any grass between the two curb tags), even if the curb is in reality one large curb let down. I usually only do the nubbin for those crossings where the curb let down is directly on the apex of the sidewalk curve and right after the stop sign (as the crossing shouldnt be before the stop sign). That way its off the main sidewalk way without having the sidewalk way mapped overtop of the asphalt areas.

One general comment I have seen online against the nubbin is that it can create additional routing instructions, which is a fair point, although I personally support the maps being more realistic than abstract.

With regards to your comment about bypassing the curb I agree that could be possible. However, you could make the same argument for those cases where the curbs are physically separated in real life. If one of them is a high curb, someone could theoretically follow the curb along the asphalt roadway to reach the other crossing. As we don't account for these edge cases for larger intersections where it would require another separate link between the two, I am somewhat less worried about it in the cases of smaller intersections (if that makes sense). If we wanted a truly 100% accurate representation for a freemoving pedestrian, all the sidewalks and crosswalks would be pedestrian areas haha.

Really I think that at the very least having the presence of a potential curb on OSM is worthwhile as that can allow additional information to be added through apps like StreetComplete.

Is there any reason why some of the nodes on the Kettle Crescent Park polygon have a Elevation 0.0 tag?

Based on what I recall there were no indications that you couldn't cross 13th SB (as all guiding islands have been removed)

Oh thank you. Forgot to check the turning restrictions.

Ah I am definitely in error here. When placing the street name Vanness Ave didn't show up as a named road in the app, so I was assuming it was an alleyway. Good catch.

Mostly I'm adding them because they are there and can be added, if that makes sense. I never add poles if I cannot actually see a pole (or in most cases its shadow) in aerial imagery, but that is a good point about connections between lines.

In this case there looks to be a bunch of semi detached houses in the area. I am going to be going there at some point to tag addresses on Streetcomplete and will do more detailed building types then.