Udarian's Comments
| Changeset | When | Comment |
|---|---|---|
| 159269420 | about 1 year ago | Please stop rounding the ramps of, that is in no way representative of the shape of the ramp. |
| 159269331 | about 1 year ago | please stop mapping turn:lane tagging as geometry. |
| 159282486 | about 1 year ago | this was already mapped correctly because we shouldn't map turn:lane tagging as geometry. |
| 159256079 | about 1 year ago | next time please don't name things with a description since names aren't descriptions, for more info see osm.wiki/Names#Names_are_not_for_descriptions . |
| 159016131 | about 1 year ago | also entrances don't go in the middle of a road, they describe places you can enter a building, so usually they are at the spot a footway meets a building. Happy mapping,
|
| 159016131 | about 1 year ago | if you have access=private on a road or gate it is implied that the other uses are also private, as in if you have access=private you don't need motor_vehicle=private. you only need to specify a value for motor vehicles, bicycles or any other mode of transportation of it is different than the default or is only usable by certain modes like a bicycle and foot path. also names aren't descriptions. Happy mapping,
|
| 158953471 | about 1 year ago | so then logically we should also remove footway=* tags. that's the pattern here, you have a tag telling you what the feature is and then then a tag that starts with what it is from the previous tag telling you what type of what it is it is.
just because the tag crossings=* has in the passed been inconsistently interpreted in the passed doesn't mean it has to now. the change is rather simple in fact, all we have to do is deprecate all crossings=* values other than uncontrolled and traffic_signals (as laid out above); and although that change would make it abundantly clear what the meaning of crossings=* is we specify in the wiki that it is purely for whether or not it is signalized. that is anyways basically the state of things now. again all we have to do is deprecate all 910 other tags apparently. again, as to keep the tagging internally consistent either we deprecate crossing:signals or both crossings=* and footway=*. pick your poison apparently. Happy mapping,
|
| 158953471 | about 1 year ago | If we deprecate crossing we would be going against all the conventions that have already been established elsewhere in the tagging, for example we don't describe sidewalks as a set of properties added to ways with the tag highway=footway, we add footway=sidewalk to describe that something is a sidewalk. by the same logic we shouldn't describe different types of crossings with different tagging, we should just say the type of crossing through a crossing tag. in the same way as footway=* tells you the type of footway crossing=* tells you the type of crossing.
I'll add a note here that there are really ony two types of crossings, uncotrolled and controlled(traffic_signals), I agree that the tagging still needs some work, I just think that we should deprecate crossing=unmarked as we did with crossing=marked since crossing=unmarked and crossings:markings=no encode the same information and since they are duplicate information one should be removed. Due to the fact that crossing=uncotroled and traffic_signals (what I'm calling controlled) do not encode the type of marking present at a crossing, crossing:markings=* is neccessary for cases were crossinga:markings is not "no" and ∴ crossings=unamrked should be deprecated. this does not apply to crossing:signals=* since if crossing=unmarked is deprecated then the two values of crossing=* would already encode the same infomation as crossings:signals=* and since crossings=* already exists we dont need a new tag and ∴ crossings:signals=* should be depredcated. essentially we either deprecate crossing:markings or crossings:signals and since it is significantly harder (it takes exponetially more tags) to encode crossings:markings fully into crossings=* it should be the one thats kept since all it takes to encode the same information as crossing:signals into crossing=* is two tags that we already have. Happy mapping,
|
| 158953471 | about 1 year ago | please in the future do not add the crossing:signals=* because it is implied the by the crossings=* value; this is because crossings=uncontrolled and crossing=unmarked both imply that the crossings isn't signalized (aka crossing:signals=no)(literally the definition of uncontrolled in this context) and crossings=traffic_signals implies crossing:signals=yes because that means that the crossings has signals. Happy mapping,
|
| 158802545 | about 1 year ago | as a note, the refs are at least somewhat incorrect, I don't know how to check what they should be, I just traced based on newest imagery. |
| 158683112 | about 1 year ago | next time please don't remove footway=sidewalk from ways along the side of a road (even if it is a service roads) that are tagged as sidewalks. |
| 158678681 | about 1 year ago | the bicycle=designated foot= designated means that this is bicycle and foot path specifically marking those as private stops that, also these aren't for cars so the general access=private doesn't really apply to the bicycle and foot paths. |
| 158468109 | about 1 year ago | what the source for this because on the Bing aerial imagery Okeechobee Frontage Road connects between the two carriageways of Northwest 138th Street Extension. |
| 158370879 | about 1 year ago | sorry miss read the changeset comment, my brain registered "colour = White -> roof:colour = white" instead of what you write, my bad. I'll be more careful next time. |
| 158370879 | about 1 year ago | were these checked with aerial imagery before uploading, I ask because some of these may be the color of the walls of the building not the roof. |
| 157893178 | about 1 year ago | as mentioned previously we (OSM) don't have the right to use Waze data as a source and since that is the source for this commit I have reverted it, for more info see osm.wiki/Waze . |
| 157991991 | about 1 year ago | I saw a world spanning commit so I translated the changeset comment to see if it properly described what was in the commit and it wasn't so I left the comment about that and because from my experience when there are geographically large commits made by new contributors there's a high chance of the commits being vandalism so I translated some of the name changes to check. the changed names came out sus so I left a comment pointing out that there's possibly an issue here so that someone with more experience and knows the language can check them and see if they are vandalism. I just didn't call it vandalism from the get go since from other commits you seems to be mostly well intentioned so I left it implied. I was just tying to be nice about it, I always like to give others the benefit of the doubt on these things unless it is really clear what it is. it isn't ignorance to point out possible vandalism, the whole point of me pointing out that I don't know the language was to point out that there might be vandalism in this commit and other who know the language can fix any issues; ignorance would be me calling it vandalism without pointing out that I don't know the language.
|
| 157938999 | about 1 year ago | please respond |
| 157991655 | about 1 year ago | it seams like the changes made to the names of these countries translate to completely nonsensical things |
| 157991991 | about 1 year ago | next time can you please leave a better changeset comment, for m ore info see osm.wiki/Good_changeset_comments .
|