Hi Rosey,
this is incorrect. everything I'm about to say is mostly UK specific. addr:unit should be used with a named building. addr:place is used when multiple buildings belong to a "place" that is not a street.

I was going to use this as an example but just discovered I forgot to map it :D
way/1298656985#map=19/51.774907/-3.300622
but here, there are multiple named houses on this street but the street has no name, so i'll be adding "addr:place=Butetown" when I map it addr:place is used as a replacement for addr:street for these cases.

Addresses shouldn't be split between objects. so in this case, all the addr tags should be deleted from the building and added to each of the entrances so each entrance has a complete address. or to put it another way the building shouldn't have an address since no one would ever write a letter to the building as a whole so it shouldn't have any address tags.
Hope this helps.
Kind regards,
Kits

forgive my bad grammar on the last message, what I meant to say is this, the simplest way to fix this would be to add a point where the waterway and path meet, then tag that point as a ford.

Hi there Ken,
the simplest way to fix this would be to add a point where the path where the waterway and path meet, then tag that point as a ford.
Kind regards,
Kits

Hello Anarchy,
Can you please add a comment when make a changeset so others are able to tell what you've changed at a glance.

westfield rd also appears to be part of this private road area. I would consider doing access=permissive instead of private as there is nothing really stopping or discouraging people walking or using these lanes
---

Published using OSMCha: https://osmcha.org/changesets/155523964

Hi there,
`managed` is not the correct tag here. I think the one you want is `operator`
---

Published using OSMCha: https://osmcha.org/changesets/155529197

Hi rosey, it appears to may have mistakenly added maxspeed to this node node/417210328

that's looks good. I've been following the naming convention from the wiki of "Bus <ref>: <place A> => <place B>" for the bus route and "Bus <ref>: <place A> <=> <place B>" for the master relation but i know this standard is very controversial so its up to you if you want to follow it or not

No, route_master is a superrelation, that is a relation that only contains other relations. I had to work out how you do this in ID :D.

you select the bus route relation, at the bottom below the members section is the relations section. here you can add or create a route_master relation for this bus relation.
Hope this helps :)

also on a side note, ID is really not suited to working with PTv2 routes. If you plan to do much with them I would recommend learning something like Josm

i've just added the Merthyr stop position. it appears you mistakenly added the stop position as a platform but i've fix that now. hope this helps.

No worries, PTv2 is not exactly user friendly. you are correct stop positions are optional, I tend to only use them at start and end routes to give a nice place for the path to begin and end. with Merthyr bus stations the stop positions are already there so you might as well add this one.

With the route master, i'm not sure if its required with only one variant, the wiki isn't clear. I've always just added it since its extremely simple to do compared to the route relation.

feel free to have a look at the relations on the exit road of merthyr bus station. i've already mapped all but routes that start or terminate there. on side note this also shows one of my issues with PTv2 since that one road has 32 relations on it due to all the variants.

you also appear to be missing a route_master relation for the T4 which is required in PTv2

Hi Russ,
referring specifically to merthyr bus station but this prop applies elsewhere too. When available public_transport=stop_position nodes should also be added to the relation directly before the corresponding platform node with the role of "stop" and the service road leading to the stop position should be added a 2nd time so you have it following the service road into the stop position and then following it again as it reverses out of the stop position and forms a continuous path.
Kind regards,
Kits

also sorry for my grammar in the last comment XD

ok thanks for clear this up. you might want to use landuse=brownfield now for this rather than construction but they may be just splitting hairs

its one of these controversial things that the community hasn't really decided on. personally. i think limiting residential areas to blocks gives extra information that makes the map easer to read

Hi there Edit,
I've seen a few of your changeset pass through in the last few days and I would just like to saying that your mapping is very good. I would like to make a couple of recommendations:

first houses should be tagged with building=house. you can then add extra detail with house=detached/semi-detached/terrace.

I can see you've already added a few address and the tags look good. I would encourage you to add more address since they are possible the single most useful thing that can be added to osm.

Thanks for contribution to OSM and feel free to ask any questions you may have.
Kind regards,
Kits :)

Hi there,
correct me if I'm wrong but as far as i can tell all these structures and roads are still standing, just unoccupied at present. I dont believe they should be removed unless they have been destroyed.
---

Published using OSMCha: https://osmcha.org/changesets/155218638

Hi there,
why did you remove all the block level residential areas and replace them with a wide area one. the normal target is to eventually replace all wide area residential areas with block ones.
---

Published using OSMCha: https://osmcha.org/changesets/155221969