In this changeset, you added a Defibrillator at node/10857037358 . However, there was already one mapped a few metres away at node/4681850123 . Do you know if there are genuinely two AEDs this close together here, or should there just be one?

Looking at aerial imagery, it's presumably an auto-complete error, when I intended to tag it as oneway=yes . Looks like it's already been fixed by someone else though:
way/27547919 .

There is no gate here and no legal restriction on deliveries using this road. So I have removed the erroneous tagging added to node/27509634 . I'm not sure exactly what "driver feedback" could have led to it.

The only restriction on this road is a the (already correctly tagged) height limit of 2.2m under the railway bridge.

In this changest you added a node node/10555428087 with name=Pure Gym and not:brand:wikidata=Q18345898 .

The latter tag states that the gym is *not* part of the "PureGym" chain described at https://www.wikidata.org/wiki/Q18345898 , i.e. isn't not part of https://www.puregym.com/ .

Can I ask if this is correct? It seems odd that someone would be able to call their gym "Pure Gym" if it wasn't part ofhte chain. It's possible that the tag was added acidentally by mis-clicking something in the OSM editor.

If the gym is part of that chain, then I think the name should be "PureGym" (no space) and it would be good to add brand:wikidata=Q18345898 to capture this.

You've got the wrong changeset here. shop=discount_store was added to the original node in changeset/126859411 by a different mapper. This changeset was just merging that node into the building polygon.

But anyway, I'd say discount_store is probably synonymous with variety_store, and certainly is in the case of this brand. So I've fixed this instance.

Thanks - I've added a "fixme".

In this changeset, you added "Poundland" brand tags to node/420783189 but left the name as "99p Stores".

https://en.wikipedia.org/wiki/99p_Stores suggests after Poundland bought the chain they changed the branding to "Poundland". Do you know if this is the case here, i.e. should the name also be updated to "Poundland"?

And that's what I've done. I've used highway=no to map the 'ground truth' that there is no physical highway following that part of the line of the public footpath.

From a legal point of view the pedestrian route there certainly exists, and it is physically possible for it to be used, despite there not being a dedicated physical path occupying that section.

These three aspects are all important, and can/should be recorded in OSM. This can't be done without the way being present to hold the tags (designation=public_footpath + foot=designated + highway=no). Without the way, we would not have this information, and there would be no way to tell whether the route exists or not or is accessible or not on the ground. i.e. it would not distinguish between the route not existing there and that part just not having been surveyed.

You're presumably talking about way/1154315514 .

The tagging highway=link was me mis-remembering highway=footway + footway=link. Though possibly highway=no would be better here. Either way though, the segment is needed to be able to map the legal route of the Public Footpath, so it shouldn't be deleted.

(It would have been better to wait for a response here rather than just deleting a way whose tagging you didn't understand. And your changeset comment of "fixing tags" is not a good description when the only action is to completely delete an object.)

Yes, it is the same. Not sure why I didn't find that key value in the wiki at the time. Now fixed, thanks.

Also, you added designation=public_footpath to way/235428293 . But in this case, I believe that the footpath has been diverted around the farm. See https://assets.publishing.service.gov.uk/government/uploads/system/uploads/attachment_data/file/926782/row_3238952_od_.pdf .

Hi! In this changeset, you added designation=public_footpath to this way: way/152909708 . Could I check if this was a mistake, or if not, why you think it's a Public Footpath?

According to the Council data I have Blickling FP 14 and Aylsham FP 9 follow a path to the north of this track. See https://osm.mathmos.net/prow/progress/norfolk/broadland/blickling/ and https://osm.mathmos.net/prow/progress/norfolk/broadland/aylsham/ .

As you can see from https://osm.mathmos.net/chains/Q116779207/ the far bigger issue is that previously mapped Martin's stores have since closed. But now we have the brand tags added, it means that it's straightforward for QA tools to flag up discrepancies for editors to re-survey. Have you had a look at https://osm.mathmos.net/survey/ in your local area?

To answer your question, I'm sure I've made hundreds of errors while editing OSM. But hopefully the small relative number is more the outweighed by the positive contributions.

But on these particular changes, I'd be surprised if there were any errors on existing objects that were correctly tagged. As I already said, it would be very unlikely for a convenience store or newsagent to be called "Martin's" and not be part of the McColl's owned chain. For objects that were incorrectly tagged (such as the shop you pointed out, where the name in OSM was incorrect), then mistakes are more likely. But I think that's less of an issue, as in some sense the error was already there.

Anyway, prompted by your concerns I reviewed all the "OSM Objects not matched to list" at https://osm.mathmos.net/chains/Q116779207/ . (We can be confident all the matched objects are genuine Martin's stores.) There were only 5 where I could not verify (using e.g. Google Streetview and/or other OSM-incompatible sources) that there was a McColl's owned Martin's store at the site previously. Three locations are in pedestrianised shopping areas with no imagery available, but the brand tags on those weren't added by me. For the other two, imagery showed them to be similarly-named non-McColl's stores. One was the one you've pointed out already. The other had the brand tags added by someone following iD's suggestion.

So I think we can say that there were most likely no other "false amendments" in this changeset.

Ok, thanks. I've removed the brand tags, added a not:brand:wikidata (to prevent iD suggesting an incorrect change again) and added a human-readable note to alert other editors.

I'm really sorry if I've mis-undersrood the issue here. I assumed you were complaining about the addition of the brand:wikidata tag that pointed to the Wikidata entry for McColls. That was added following the NSI preset at the time, based on an exact name match of "Martin's". (Given UK trademark law, I though it very unlikely that a convenience store called "Martin's" could be anything other than part of the McColl-owned Martin's branded chain.) That wikidata tag now been altered by another mapper to the newly created wikidata entry for Martin's.

But presumably that's not the issue you were raising originally. Apologies again for misunderstanding. Perhaps you could explain exactly what the issue is, as I'm still not sure. Are you saying it's an independent shop and not part of the Martin's brand?

(If so, it would have been helpful if you said this explicitly in the changeset comments, or left a note=* tag on the object, when you removed the brand tags from way/676788755/history . Also, should the name be put back to "Martins Local", as you mapped it originally? Note that I haven't altered the name. Had it been named "Martin's Local", or if there had been a note tag explaining it wasn't part of the Martin's brand, it would not have been included in my recent changeset. )

I'm not sure if it was deliberate or not, but both of the two nodes above have amenity=bank on them. This is wrong, as it means that the OSM data is recording two banks there, when in reality there is only one.

To capture the situation you describe, I would probably have one amenity=bank node for the bank itself with (with atm=yes on it if you want), and then one amenity=atm for each of the physical ATMs. The one inside the bank could have indoor-=yes to distinguish it. You can add appropriate opening_hours to all three objects.

Is the brand=* needed if brand:wikidata=* is present? Feel free to add it if you want...

In terms of how the edit was done, it was an overpass turbo search on the name, results exported to JOSM, followed by a manual sanity check on the objects returned before adding the brand:wikidata tags.

Hi Phil,

In this changeset you created the node node/10080826333 . This appears to duplicate a pre-existing bank node node/6871715012 .

I presume this was a mistake. But should the first node above be deleted, or did you e.g. intend to add a second ATM at that position?