CjMalone's Comments
| Changeset | When | Comment |
|---|---|---|
| 87122938 | over 3 years ago | And laughably, the PO website contradicts the PO sign and Southern Coop website by saying "Please Note: Not all services are available outside of the dedicated Post Office operating hours. Please check with your local branch." I'm losing the will to live on this one, I'm going to ignore opening hours on post offices until there is a clear answer. I'll still use PO data to add addresses, add/remove POs in OSM (OS OpenMap Local helps too), and ref:pol_id. |
| 87122938 | over 3 years ago | My local post office is is more complicated. It's one of the bigger ones, a real counter. PO website and pol data say Mo-Su 07:00-20:00. But the counter is signed "Main counter opening times Mo-Fr 08:30-17:30; Sa 08:30-14:30 if the main counter is not open, please use our serviced from the main retail point". node/1309359635
Also, that's not the correct coop opening hours, printed on the door is "Mo-Su 07:00-22:00", I did think they permanently changed from 23:00 to 22:00 semi recently. The coop website says "Mo-Su 07:00-23:00", but the Southern Coop website says "Mo-Su 07:00-22:00 Post Office services are available throughout this store's opening hours." https://www.coop.co.uk/store-finder/PO37-7AA/42-regent-street
So, I guess, is the opening times for the counter, or for the service? |
| 87122938 | over 3 years ago | This is a 2 and a half year old changeset... Considering I only really started mapping in 2019 I'm sure there are lots of bad edits from around this time. I would not have added "opening_hours:post_office" now adays. I tent to stay away from shop=* amenity=post_office combinations, unless "untangling" them. The validation of POL branch data is interesting. I'm going to look at some ones close to me. |
| 123421872 | over 3 years ago | I was talking about this one, changeset/105877402. There are plenty more bad changset descriptions: #117490994, #113128712, #113128735, #113129191, #111990185, #111990163, #changeset/112839818. But it's so not important, I don't know why we're even talking about it. |
| 123427949 | over 3 years ago | Note, check_date:opening_hours is now broken. It should presumably be check_date:opening_hours=2019-12-07, unless you've checked them more recently? Also the source is wrong, but that doesn't matter. Thanks, Cj |
| 123421872 | over 3 years ago | Cheers Phil, But I reject you labelling this a "mechanical edit". I looked at every POI in this changeset. I reject you calling this edit unhelpful. I think it has undeniably improved post offices in this area, even if there were one or 2 mistakes. Mistakes happen when humans are involved. I can't help you with "my local post office", but if I overrode recent surveyed opening hours it was a mistake. Sorry. I was only adding opening hours because the data is less that 2 months old, I thought I checked object history to make sure I wasn't overriding recent data, and added check_date:opening_hours for to help in the future. I reject "The co-op has a scale thing that may be available if they have a member of staff who knows how to use it", while it's true the manager on shift also has to be fully PO trained. As long as you're nice and respectful they can do it all. That being said, I do believe with Post offices in shops, the PO node should be the counter times, and the shop should have the full times. Please point this node out to me or fix it yourself. I don't believe you can criticise my changeset description, you have 20+ with just "https://maproulette.org/browse/challenges/26015" or "Fix tagging", "(no comment)", "Yet another barber shop". Please don't forget OSM is a do-ocracy. Edits are more valuable that potential edits. This changeset used a community approved dataset to improve OSM. Cj |
| 117296466 | over 3 years ago | There are advantages to having the businesses on nodes, I'd prefer it if you didn't move them to buildings. |
| 116486869 | over 3 years ago | Hey, when you add disused:amenity or disused:shop, you also need to remove amenity or shop. I'll fix this. |
| 121883931 | over 3 years ago | Hey, I don't think Post Office Ltd operate this post office, it's a franchise to Central. |
| 112969735 | over 3 years ago | This also added a CeX in a residential area. Seems unlikely. |
| 121805274 | over 3 years ago | I've requested a review, not for the alignment, but for someone to check all the link roads, connections, etc. I don't usually map this stuff so I've no idea if it's right or wrong. Bing has just been updated to show the new layout. |
| 99230973 | over 3 years ago | Yeah, I don't think it should separated, but I don't know enough about lanes, cycle lanes or routing to delete it. |
| 118521964 | over 3 years ago | Doubt it. Removed in changeset/119553875 |
| 119495871 | over 3 years ago | Thanks bro but you're about 9 years to late. |
| 119381866 | over 3 years ago | That one does have a name and brand tag. I could do with spending some time on Morrisons Daily though, I think FHRS is unusually up to date on the McColl's -> Morrisons Daily fascia change. |
| 113342544 | over 3 years ago | Hey, just seen this. No, it doesn't look like you added it to the relation in this, but I'm considering deleting the relation, or removing the individual points from the relation. It's not the kind of data OSM usually stores. |
| 118324129 | over 3 years ago | I think it i's clearly vegetables, but if you want to change it to sells:vegetables, I wont argue against it. |
| 118324129 | over 3 years ago | I think sells:veg is OK tagging, it fits with sells:tobacco and other sells:*. They aren't that popular but it's useful data. |
| 118335522 | over 3 years ago | Nice one, fixed the name in #118907705 |
| 118129406 | almost 4 years ago | I've should have cleaned up this changeset in #118775388 |