impiaaa's Comments
| Changeset | When | Comment |
|---|---|---|
| 108146211 | over 4 years ago | This changeset reuses nodes from the border of a wildlife refuge to create a golf sandtrap. Please do not reuse nodes from unrelated objects. |
| 108165278 | over 4 years ago | I see that a number of nodes used to make these new buildings were reused from nearby roads. Please do not reuse nodes from unrelated features. |
| 107829161 | over 4 years ago | The OpenStreetMap community has a concept called "Any tags you like" osm.wiki/Any_tags_you_like . Even if there is no software out there that recognizes a tag, it's still okay to use an undocumented tag, as long as the mapping is physical and verifiable. In this case, mapper Minh Nguyen wanted to indicate that this intersection disallows turning right when the traffic light is red. This fact can be verified by signs at the intersection. There is no documented tag to indicate this, so he invented one. This is allowed by the OpenStreetMap community. There's also nothing wrong, necessarily, with deleting this from the map, but I believe it to be valuable information, and its removal harmful. |
| 108000919 | over 4 years ago | You appear to have changed much more than just adding buildings. Please use more descriptive changeset comments. Also, I see that nodes from a highway were used for a building. Please only reuse nodes when adjusting geometry, not for unrelated new geometry. |
| 107829161 | over 4 years ago | Oh, I misread. You said "OSRM", not "OSM." Still, just because OSRM doesn't support a tag doesn't mean that it should be removed. |
| 107829161 | over 4 years ago | What do you mean "can't be recognised by osm correctly"? There's no such thing as a disallowed OSM tag. If you mean that there is software that does not correctly handle the relation, the software should be updated rather than the tagging changed. |
| 107801992 | over 4 years ago | I attempted to contact the author but they haven't responded in more than a week. This reverts the changesets numbered 107024938, 107024946, 107024950, 107024958, 107024963, 107024970, 107024974, 107024982, 107024991, 107024995, 107025003, 107025007, 107025013, 107025020, 107025036, 107025045, 107025053, 107025057, 107025060, 107025069, 107025073, 107025098, 107025149, 107025163, 107025188, 107025191, 107025200, 107025229, 107025277, 107025327, 107025331, 107025342, 107025355, 107025370, 107025384, 107025393, 107025400, 107025414, 107025420, 107025427, 107025430, 107025437, 107025474, 107025482, 107025494, 107025499, 107025522, 107025528, 107025531, 107025542, 107025545, 107025553, 107025557, 107025567, 107025573, 107025587, 107025593, 107025601, 107025606, 107025615, 107025623, 107025640, 107025646, 107025651, 107025658, 107025663, 107025675, 107025684, 107025696, 107025704, 107025712, 107025720, 107025727, 107025732, 107025740, 107025744, 107025746, 107025761, 107025762, 107025766, 107025775, 107025778, 107025781, 107025791, 107025796, 107025800, 107025805, 107025808, 107025812, 107025816, 107025820, 107025822, 107025826, 107025828, 107025834, 107025837, 107025840, 107025842, 107025845, 107025849, 107025851, 107025853, 107025855, 107025860, 107025870, 107025874, 107025878, 107025885, 107025888, 107025897, 107025902, 107025919, 107025925, 107025928, 107025932, 107025936, 107025941, 107025947, 107025951, 107025956, 107025959, 107025964, 107025966, 107025968, 107025970, 107025972, 107025974, 107025976, 107025979, 107025985, 107025988, 107025991, 107025995, 107025999, 107026000, 107026004, 107026006, 107026008, 107026013, 107026020, 107026024, 107026025, 107026028, 107026036, 107026038, 107026041, 107026046, 107026049, 107026060, 107026065, 107026073, 107026080, 107026083, 107026087, 107026094, 107026097, 107026101, 107026106, 107026110, 107026112, 107026115, 107026122, 107026123, 107026128, 107026135, 107026137, 107026139, 107026145, 107026148, 107026152, 107026156, 107026158, 107026162, 107026167, 107026171, 107026172, 107026178, 107026179, 107026181, 107026182, 107026184, 107026188, 107026189, 107026193, 107026195, 107026197, 107026200, 107026203, 107026205, 107026206, 107026210, 107026211, 107026214, 107026224, 107026227, 107026229, 107026235, 107026237, 107026238, 107026241, 107026244, 107026248, 107026249, 107026251, 107026254, 107026257, 107026259, 107026266, 107026268, 107026273, 107026274, 107026278, 107026282, 107026283, 107026287, 107026289, 107026290, 107026291, 107026295, 107026299, 107026302, 107026305, 107026307, 107026311, 107026315, 107026319, 107026322, 107026326, 107026328, 107026330, 107026334, 107026341, 107026351, 107026355, 107026360, 107026365, 107026369, 107026378, 107026380, 107026383, 107026399, 107026407, 107026409, 107026411, 107026414, 107026418, 107026425, 107026432, 107026437, 107026440, 107026446, 107026448, 107026450, 107026454, 107026458, 107026461, 107026466, 107026469, 107026471, 107026472, 107026475, 107026477, 107026481, 107026483, 107026488, 107026491, 107026494, 107026495, 107026496, 107026500, 107026503, 107026505, 107026510, 107026511, 107026515, 107026520, 107026522, 107026525, 107026526, 107026529, 107026530, 107026534, 107026540, 107026543, 107026549, 107026553, 107026555, 107026556, 107026563, 107026569, 107026576, 107026582, 107026587, 107026593, 107026596, 107026601, 107026605, 107026608, 107026615, 107026617, 107026619, 107026622, 107026625, 107026628, 107026633, 107026635, 107026638, 107026641, 107026645, 107026649, 107026652, 107026657, 107026664, 107026670, 107026678, 107026680, 107026689, 107026691, 107026694, 107026697, 107026699, 107026703, 107026706, 107026709, 107026712, 107026720, 107026724, 107026729, 107026735, 107026739, 107026741, 107026745, 107026747, 107026754, 107026756, 107026760, 107026764, 107026766, 107026769, 107026773, 107026784, 107032925, 107032946, 107033103, 107033109, 107033115, 107033119, 107033126, 107033132, 107033137, 107033140, 107033143, 107033147, 107033149, 107033155, 107033158, 107033160, 107033165, 107033169, 107033176, 107033181, 107033182, 107033184, 107033188, 107033195, 107033197, 107033198, 107033202, 107033207, 107033213, 107033214, 107033219, 107033221, 107033223, 107033225, 107033226, 107033228, 107033231, 107033233, 107033236, 107033240, 107033244, 107033246, 107033249, 107033252, 107033253, 107033255, 107033256, 107033258, 107033262, 107033267, 107033271, 107033273, 107033278, 107033279, 107033282, 107033284, 107033286, 107033288, 107033289, 107033298, 107033309, 107033318, 107033336, 107033506, 107033518, 107033527, 107033562, 107033573, 107033584, 107033595, 107033600, 107033616, 107033620, 107033623, 107033624, 107033626, 107033635, 107033642, 107033643, 107033651, 107033654, 107033657, 107033663, 107033665, 107033666, 107033672, 107033680, 107033682, 107033685, 107033688, 107033692, 107033694, 107033695, 107033701, 107033705, 107033711, 107033716, 107033718, 107033719, 107033725, 107033728, 107033730, 107033736, 107033738, 107033740, 107033743, 107033748, 107033752, 107033755, 107033758, 107033764, 107033769, 107033772, 107033775, 107033777, 107033782, 107033785, 107033787, 107033789, 107033791, 107033793, 107033802, 107033807, 107033809, 107033813, 107033816, 107033821, 107033830, 107033837, 107033842, 107033845, 107033850, 107033857, 107033862, 107033865, 107033868, 107033870, 107033874, 107033876, 107033880, 107033886, 107033890, 107033894, 107033897, 107033900, 107033902, 107033907, 107033909, 107033911, 107033916, 107033917, 107033922, 107033926, 107033930, 107033932, 107033934, 107033936, 107033940, 107033943, 107033944, 107033945, 107033948, 107033951, 107033955, 107033960, 107033966, 107033967, 107033970, 107033977, 107033979, 107033983, 107033987, 107033990, 107033996, 107034001, 107034003, 107034010, 107034015, 107034017, 107034022, 107034026, 107034030, 107034035, 107034038, 107034040, 107034041, 107034049, 107034051, 107034054, 107034056, 107034059, 107034064, 107034066, 107034067, 107034068, 107034071, 107034073, 107034075, 107034080, 107034086, 107034094, 107034097, 107034100, 107034104, 107034107, 107034111, 107034116, 107034120, 107034123, 107034127, 107034129, 107034131, 107034134, 107034137, 107034141, 107034145, 107034146, 107034149, 107034153, 107034156, 107034160, 107034163, 107034166, 107034167, 107034170, 107034172, 107034177, 107034180, 107034182, 107034186, 107034190, 107034192, 107034197, 107034198, 107034201, 107034211, 107034213, 107034217, 107034220, 107034223, 107034224, 107034230, 107034235, 107034238, 107034246, 107034250, 107034252, 107034258, 107034263, 107034267, 107034269, 107034275, 107034278, 107034279, 107034283, 107034285, 107034286, 107034291, 107034293, 107034295, 107034300, 107034303, 107034308, 107034315, 107034317, 107034322, 107034326, 107034327, 107034332, 107034338, 107034341, 107034350, 107034354, 107034361, 107034364, 107034368, 107034373, 107034375, 107034378, 107034384, 107034385, 107034388, 107034393, 107034397, 107034407, 107034410, 107034413, 107034415, 107034418, 107034422, 107034425, 107034429, 107034432, 107034435, 107034439, 107034442, 107034443, 107034446, 107034450, 107034453, 107034455, 107034457, 107034463, 107034464, 107034465, 107034468, 107034471, 107034475, 107034479, 107034482, 107034484, 107034488, 107034495, 107034497, 107034501, 107034511, 107034521, 107034526, 107034529, 107034537, 107034540, 107034543, 107034553, 107034557, 107034560, 107034565, 107034568, 107034570, 107034573, 107034576, 107034577, 107034583, 107034588, 107034592, 107034596, 107034601, 107034602, 107034606, 107034610, 107034613, 107034616, 107034618, 107034622, 107034627, 107034630, 107034635, 107034636, 107034639, 107034645, 107034648, 107034652, 107034654, and 107034657. |
| 107111986 | over 4 years ago | Hello and thanks for your contributions to OpenStreetMap! Unfortunately, there are some issues with your change: The cycleway that you added crosses many roads without an intersecting node, which the editor should have warned you about. More importantly though, it's convention in OpenStreetMap to only map roads or paths if they're physically separated, so a bike lane doesn't really count. The bike lane is actually already mapped as part of the main road--if you look at the tags, you can see it has "cycleway=lane", so a separate way is not needed. |
| 107070360 | over 4 years ago | Thank you for correcting four of your changes. However, you made thousands of similar edits that I didn't comment on. Please fix them as well. |
| 107025866 | over 4 years ago | Another false re-tag. It would also be helpful to be more descriptive of your changes in the description, something like "Removed highway=priority" |
| 107026373 | over 4 years ago | The same thing, again. Making changesets cover only a single node results in hundreds of changes that are hard to review. |
| 107026711 | over 4 years ago | I have to ask the same thing about this changeset. highway=priority seems perfectly reasonable and helpful for this node. Why did you remove it? |
| 107033172 | over 4 years ago | Why did you remove the highway=priority tag? As documented on the wiki at highway=stop#Tag_the_priority_junction_node , it's used to indicate an intersection where some traffic must stop and prioritize other traffic, which seems appropriate here. |
| 106829181 | over 4 years ago | Thanks for your contribution. Please use the changeset description field to describe what you changed and why; "Updated by Mundi" does not describe what changed. Perhaps something like "Added smoothness value and width to a footway". |
| 106831357 | over 4 years ago | This is incorrect. It is correct to have highway=crossing on the node. highway=footway should be on a way, and it is already. I've reverted your change in changeset/106987959. |
| 106405990 | over 4 years ago | Hello. Please know that whatever app you are using is enabling you to make changes to OpenStreetMap, a global, community-built app. If you would like to test out the app without making any changes to the real world map, make the app use the OSM development server instead. |
| 105898499 | over 4 years ago | This partially reverts changesets 105447152 105447007 105446669 105446279 105445954 105445660 105445024 105444970 105444847 105444549 105443878 105443624 105442985 105438193 105437913 105437744 105437446 105437081 |
| 105898525 | over 4 years ago | This partially reverts changesets 105494859, 105494503, 105494098, 105451435, 105451278, 105451030, 105449902, 105449644, 105449347, 105449126, and 105448837 |
| 105704236 | over 4 years ago | Hello,
|
| 105437081 | over 4 years ago | Is this correct? I don't remember these freeways being toll roads. Maybe you meant something like toll=no + toll:express=yes, to cover FasTrak lanes. |