amapanda ᚛ᚐᚋᚐᚅᚇᚐ᚜ 🏳️⚧️'s Comments
| Changeset | When | Comment |
|---|---|---|
| 55979332 | In iD, if you click on a node or way, there's a "View on openstreetmap.org" link on the bottom left. Clicking on that brings you to the 'view' page for a node/way/relation ( node/453344874 ). You can click on the 'View history' to show you details of the history of that node ( node/453344874/history ). It's not a great view. There are some graphical/better ways to view the history of an object.This page ( https://pewu.github.io/osm-history/#/node/453344874 ) shows the edit history of that node. From the regular 'view history' page, you can see the changesets. And this ( changeset/55875241) is the changeset that moved that traffic light node. (they look like a Pokemon Go player) achavi is a service which shows, graphically, changeset. Here ( https://overpass-api.de/achavi/?changeset=55875241 ) you can see the movement of the node. |
|
| 55979332 | You asked for a a review, so here it is. This looks great. Thanks for spotting, and fixing, this mistake. 👍😁 |
|
| 55935292 | Hi there. You asked for a review of this. I'm not sure what a "ball wall" is. I don't know what (if any!) OSM tag there is for that. It's OK to make up your own tags. For some things unique to Ireland, there aren't any "correct" tags yet. So this looks fine. |
|
| 55934512 | Hello and welcome to OSM! You asked for a review of this changeset. This change looks fine. It's always good to have more houses entered (despite the project being called Open STREET Map, it actually has much more buildings than streets). The web editor (iD) has the ability to square building corners, to make them right angled, which is what they are in the real world. It can make the map data better. I'll have to look at the aerial imagery later, but if you have a semi-d house, or a terrace, then it's better in OSM to split that into 2 buildings which share a wall. This allows you to put separate tags for (say) housenumbers on each one. |
|
| 55934447 | Hi, You asked for a review, so here it is. Thanks for this edit. Lots of good stuff here, with lots of details. 👍🙂 You've added a node (point) here ( node/5375514911 ) for a basketball court. However you put it inside a way (area) of an existing basketball court (this one: way/256113545 ). It's better to have sports facilities like that (pitches) as areas, not points. Since they are areas in real life, and with decent arial imagery you can usually see where the actual court outline is. It's also not good to have 2 things when there is only one basketball court, you should probably remove the node/point you added. I notice you added `hoops=2`. 🙂 As you have figured out, OSM has a flexible tagging system, so you can make up your own tags, there's no central authority. But I dunno if we need to specify that there is 2 hoops (since that's what one bastketball court usually has). Information on lighting and building sizes is very helpful, again it's data that's hard to get. So thanks for adding it. |
|
| 55934043 | You've also moved a post box to the other side of the street. Which is OK, but a little odd. Did An Post remove the one beside the dry cleaners? If there are actually 2 post boxes (one near the dry cleaners, the other where the node now is), then you should have 2 postboxes in OSM. 🙂 You've also done a kinda weird thing of "merging"/"glueing" the postbox to the corner of this building ( way/227934966 ). It's almost certainly wrong to glue a postbox there. When somethign is "part of the wall" you "glue"/"attach" it like that. So doors or things are "glued". But if the postbox is freestanding, then you should just have it as a separate, free standing, node in front of the building. Can you stand between the postbox and the building? If yes, then it should be separated. if no (i.e. postbox is built into the wall), then what you've done is right. |
|
| 55934043 | However, there are one or two simple, and common, beginner mistakes. You added the Applewood vets ( node/5375493810 ), and for the phone number it's better to put it in in "international format", i.e. "+353-1-...". Then it's usable to anyone in the world without them needing to know about Irish phone numbers. You added the opening hours, but there's a "proper" format for opening hours ( opening_hours=* ). If you use that, then apps can use it (e.g. OSMAnd can show you that). The format can be complicated due to all the special cases, but you can use a page like YoHours ( http://projets.pavie.info/yohours/ ). You just graphicaly select the hours and then copy/paste the opening hours text at the top. I've changed those details on that vets now, You can see the changes here https://pewu.github.io/osm-history/#/node/5375493810 |
|
| 55934043 | Welcome to OSM! You asked for a review for this, so here it is. It's good that you're adding POIs, things that can only be added by someone on the ground. Please continue to add more of them. 👍😃 It's also get to get more details, phone numbers, opening hours etc. |
|
| 55934997 | Welcome to OSM! You asked for a review so here it is. This looks fine. :) It's always good to have Points of Interest (POI) like shops. We can't get them from aerial imagery. It's good that you put in a little bit more details, rather than just "shop=yes" or something. You can always add more details, phone numbers, opening hours, eircodes, etc. :) |
|
| 55847110 | Hi there. Welcome to OSM!You asked for a review, so here it is. I'm a little confused by this. You've added a (weirdly shaped) pond in a middle of some back gardens. I can't see any ponds there in the Bing aerial imagery. What's going on? I've removed it in this changeset ( changeset/55927886 ). |
|
| 55844231 | A review was requested, and for the record, I did this (& the other) on the other changeset: changeset/55844257 |
|
| 55844257 | Hi! Welcome to OSM! You asked for a review on this edit (and the previous one changeset/55844231 ), so here it is. Thanks for adding this, and the house number. We can't get addresses from the aerial imagery, so we need someone local to add them. Thanks for this. 👍 *buuuuut* you added your back garden as `leisure=park`, that's mostly for actual parks maintained by the council and all that. Adding a name of "back garden" is wrong as well, since the name should be the sort of thing that would be on a sign, not just a description. It's OK if something doesn't have a name, lots of things in the real world don't have a sign with the name. You can also use the 'square' feature to make a building have right angles, which can make buildings more accurate. |
|
| 55801603 | Hi, welcome to OSM! You asked for a review, so here it is. I think you've made one or two newbie mistakes here. You added a part of the path here ( way/556245385#map=18/53.34689/-6.52044 ) and you added the `access=no` tag. That's for things which the public aren't allowed on. Private driveways that kind of thing. Is that right? I suppose this is private grounds so it make sense that way. Maybe you should update the other tracks/roads as well. You've tweaked this little bridge there here ( changeset/55801603#map=17/53.34659/-6.52231 ) and sorta joined it at a sharp angle to the new path. Is that right? It looks like the little bridge should be part of the track you added. Do you know what I mean? Would you like to take a stab at cleaning it up or would you like me to do it? |
|
| 55787974 | Hello and welcome to OSM! You asked for a review of this change. You marked this as "natural=grassland", but that's more for savannah in Africa or something, not a patch of grass in a housing estate! 🙂 I've changed it to landuse=grass which is correct for this. |
|
| 55695808 | You asked for a review, so here it is. This is very good for a new mapper. you're including a lot of details, which is great. eircodes/addresses, opening hours, lots of details on the pub, etc. The only thing I'd question is the farm buildings to the top right, you added building=farm, but I think that's for "farm houses" (which we don't really have in Ireland I think). I would have mapped those buildings as building=barn. But, if you know the area, then that's fine |
|
| 16858468 | I've fixed it hre changeset/55741408 |
|
| 55696655 | Well, are there any houses/buildings there now? If they've all be leveled, then it looks like those numbers are now gone and you can delete them if you want. |
|
| 55696655 | You asked for a review, so here it is. This looks fine. You can make areas have "right angled corners" in iD, sometimes that can be more accurate if they have right angled in the real world. I trust you that these houses have been demolished. *But* there is still some address points there, if they've been demolished, shouldn't they be gone? Is there a number 176 still? a 175? etc. |
|
| 55696624 | Hi! You asked for a review, so here it is. Now, I know the thing you have added. it's a big old, semi-circular, metal yoke, right? It doesn't actually lift the bridge any more, right? So I don't know really *what* type of thing this is. Now, OSM has a free form and flexible tagging system, so you're totally free to make up whatever tag you want ( osm.wiki/Any_tags_you_like ). So man_made=Lifting_bridge is as good as anything. Minor note: the OSM convention is to use lower case letters for "machine" tags, so it should be man_made=lifting_bridge here. You don't need the area=yes tag, since it's a closed area. iD adds them often. It doesn't really matter anyway, since it does no harm. |
|
| 55692408 | Hi, you asked for a review, so here it is. I see you did some HOT work in Bangladesh, you haven't changed the commit message. 😉 You have added the name to a crossroads here. Thanks! 😁 It's that sort of local knowledge that we need. *But* you just put the name on the point (i.e. "node" in OSM-speak). So much software (incl the software which makes the map on osm.org) won't do anything with it. If you add the "junction=yes" tag, then it'll appear on the map. Would you like me to do it? or do you want to have a go at it? Here's an example of the way to do it: node/1496772129 |