Hi there, landuse=grass is incorrect here. landuse=grass is used for maintained areas of grass. if this area is used for grazing animals then it should be landuse=meadow, if its used for grow crop including grow grass for hay then it should be landuse=farmland. if its wild area mainly consisting of grass then it should be natural=grassland
---

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

Hi there drh,
its appears you've dragged a node of a wall by mistake, way/1279602177#map=16/53.37311/-4.53246. please fix it
kind regards,
Kits
---

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

Hi there,
Forgive me, normally when I do a review I guide a person through fixing their own changes but I'm short on time today so I've fixed it myself. please study what I've changed and feel free to ask any questions.

changeset/156252426

Here are the main points:

Buildings should follow the outline of the actual building not the land. I've added a landuse=retail to indicate the land the pub occupies.

The carpark was somehow a mult-polygon of one. Not sure how this happened but I've removed the relation and made it a normal area again.

you had two elements representing the same pub. There should only be one element for each real world object so since you've mapped the pub building there is no need for the pub point to exist anymore.
osm.wiki/One_feature,_one_OSM_element

overall your tagging is good. the only error was on the opening_hours. Days should be represented with 2 letters, eg Su=sunday and each entry should be separated with a semi-colon ";". you can also use day ranges eg "Mo-Su" for monday to sunday. so the new value for opening_hours is "Su 12:00-22:00; Mo-Sa 11:00-22:00;" you can also use this tool to help you with them since the opening_hours syntax is not the simplest. https://projets.pavie.info/yohours/?oh=Su%2012:00-22:00;%20Mo-Sa%2011:00-22:00;

I hope this is of some help and please feel free to ask any questions.

Kind regards,
Kits

this reverts changeset/56146484

hi Lamb, the new little section of road should prob be a unclassified road instead of a trunk link
---

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

vandalism reverted in changeset: 156197126

hi there, this is not correct. I've removed all instances of srcName and official_name from power lines. srcName was the name of the substation where the power line originated, it wasnt meant to be included as the origin substation can be found with the "wpd_source_site" tag and it name derived from that. i'll correct my import scripts to avoid this in future.

Sorry I should have include the node. Here it is node/12142984445

Hi there, I would you like to say that you could avoid OS Maps when contributing to OSM. With exception of OS OpenData and out of copyright maps, OS Maps are copyrighted and are not compatible with OSM's licence. osm.wiki/Ordnance_Survey

Hi there,
What is your source for this change?
---

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

Hi there, it appears you've connected your new path to the power cable
---

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

Hi there Nathan,
Could you please provide a source for the overtaking restriction. On imagery, the road appears to have a broken centre line, allowing overtaking.
---

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

Hi Ade,
Could you please add more descriptive comments to your changesets in future. its hard to tell what you are changing.

Hi Changsta,
You should try to limit changesets to a single local area.

this reverts changeset #155834346

for future reference in that case you would mark a section of a stream with `tunnel=culvet` and `layer=-1`. for now i would recommend placing a note here to make the survey request more visible

there's no point waiting for a survey for stuff like this especially when a lot of OSM was imported rather than surveyed.

In this case you have enough information to justify the change. a Ford is somewhere where a waterway crosses a roadway and at present the waterway crosses the roadway without any node which is considered incorrect.

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