ZeiP's Comments
| Changeset | When | Comment |
|---|---|---|
| 73320610 | almost 6 years ago | Changeset has removed at least the valid attribute man_made=water_tower from the Juhannuskukkula school building without any information on why the removal was made. |
| 72603987 | over 6 years ago | This adds maxspeed=* values to roads that already have maxspeed:forward and maxspeed:backward. This is not a good idea, as that makes it impossible to know which of the values is the correct one. |
| 62826754 | over 6 years ago | This changeset has toll=yes for way #360593734 (the national road 9). AFAIK there are no toll roads in Finland, and at least not on that road. Is there a specific meaning to the tagging or is it just a mistake? As it seems erroneous to me, I'll remove the tag for now. |
| 71229282 | over 6 years ago | You've added some names that look like the names of the residents to the building. The name tag should contain the name of the house, not the name of the resident(s) – is it the actual name of the house? If not, I think it would be appropriate to remove the name tag altogether, allowing the building to be shown with it's house number instead. |
| 69607031 | over 6 years ago | You've added some names that look like the names of the residents to the building. The name tag should contain the name of the house, not the name of the resident(s) – is it the actual name of the house? If not, I think it would be appropriate to remove the name tag altogether, allowing the building to be shown with it's house number instead. |
| 38008405 | over 6 years ago | You've added the user name of an OSM user to the building (which was later supplemented by the said user). The name tag should contain the name of the house, not the name of the resident – is bhudi the actual name of the house? If not, I think it would be appropriate to remove the name tag altogether, allowing the building to be shown with it's house number instead. |
| 62207528 | over 6 years ago | En nyt suoraan muista, paitsi että etäinen muistijälki on tällaisen merkinnän miettimisestä. Todennäköisesti kyseessä on ollut jokin siltatyömaahan liittyvä merkintä, koska tuota on pätkät molemmilla puolilla siltaa sekä itse silta. Käytännössä siis tuota merkintää on wikin mukaan käytetty esimerkiksi liikennevalo-ohjattuun ratkaisuun jossa tievälillä pääsee vuorotellen molempiin suuntiin.
|
| 60122652 | about 7 years ago | The library created in this changeset (node/5715340852) does not exist. |
| 29046644 | about 7 years ago | Is this really a cliff? The aerial imagery doesn't look like it. |
| 62676834 | over 7 years ago | Hmh, ok. Is this somewhere documented? I tried to find decent documentation on the correct tags, and the spec I linked was the only one I found. We should document it more thoroughly if there isn't documentation yet, because I'd guess I'm not the only one struggling with the definition. |
| 57652750 | over 7 years ago | I don't think it makes sense to tag it both man_made and building. The rendering is also a bit problematic, because now it looks like there is a tower on top of the building. In the tower tag documentation there is the tower:type key, which shows the type of structures that are meant to be tagged with the tower tag, and a tallish building isn't really in them. If it's ok with you, I'd like to remove the tower tagging from the apartment building. |
| 57652750 | over 7 years ago | I'm not sure this should be tagged as a tower. It does fit the basic description of being higher than wide, but so do many apartment buildings too. It's basically just a tall apartment building, so I suggest removing the tower tagging. |
| 61245906 | over 7 years ago | Thanks for your polite comments. I'll take a look at them when I have time, feel free to edit the rest yourself too, of course. I do believe that even if there were some ways that weren't edited, it's still a step to the right direction, and shouldn't make a mess. |
| 60602849 | over 7 years ago | It doesn't mean it's wrong (you can use any tags you want), but changing another mapper's tagging to an undocumented and very little used tag is. Changed to landuse=institutional: changeset/60607689 |
| 60602849 | over 7 years ago | landuse=government is completely undocumented, so I don't think you should be replacing existing tagging with it without discussion. landuse=institutional would make a bit more sense: landuse=institutional |
| 43975930 | over 7 years ago | Yeah, I noticed that may not be the best one. I would put it down as a residence also, since that's the best one there really is. landuse=government isn't documented at all, so I don't think you should be replacing existing tagging with that. |
| 43994938 | over 7 years ago | IMO it was the best fit for a residence. I looked through the documented landuse=* values again, and I do still think it's the best fit. Do you have other suggestions? |