Welcome to OpenStreetMap and thanks for updating the map!

That looks fine to me.

The only thing I might suggest changing is that it should be building=semidetached_house (or building=house + house=semi-detached) rather than putting the value in building:part. It's very minor and it's very unlikely to cause anyone any problems.

The building:part tag is mostly used as part of the 3D representation of a building.
https://wiki.openstreetmap.org/wiki/Key:building:part