Frank Peng's Comments
| Changeset | When | Comment |
|---|---|---|
| 162891977 | 11 days ago | Why did you tag those one-way roads as not one-way? It's a divided highway, and the carriageways are clearly one-way roads. What is the source for this change? Thanks. |
| 175671709 | 11 days ago | Also, please respond when someone comments on any of your changesets pointing out issues to you or requesting further information about your edits. This helps avoid issues and builds trust between you and other mappers on OSM. Users who never respond may be treated as less trustworthy.
I and a couple of other mappers have commented on your changesets pointing out issues and requesting further information about your edits, and you have not responded to them. You can see your discussed changesets here: https://resultmaps.neis-one.org/osm-discussion-comments?uid=22361283
|
| 174818967 | 12 days ago | Why did you remove the gates from this driveway? I can see in the Bing Streetside street-level photos from 2021 that there are gates on both sides of the driveway. Did those gates recently get removed? If those gates are still there and open, then you should not delete the gates from this driveway way. You could've tagged those gates as barrier=gate and locked=no so routing software will know that those gates are unlocked and access through them can be allowed without a key. |
| 175671709 | 12 days ago | Also, I can see in the Bing Streetside imagery from 2021 that this driveway appears to be one-way based on signs on the ground. Did this driveway recently stop being a one-way road? What is the source for this? Thanks. |
| 175669829 | 12 days ago | Also, I can see that you added a non-existent bridge, which I don't think there is really a bridge here: way/1456912459 |
| 175671709 | 12 days ago | I don't think this driveway should be tagged as an unclassified road since it serves school buses and a school. This should be tagged as highway=service and service=driveway. I think you have the tags wrong on the driveway way (way/752320461).
|
| 175669829 | 12 days ago | Why did you change valid administrative boundaries into roads? Those are very clearly administrative boundaries and must not be tagged as roads. |
| 175737446 | 12 days ago | This changeset did not disconnect the node of the road from the administrative boundary as I thought it would. Instead, it moved the node back to its previous position along with the administrative boundary. The node of the road was actually disconnected from the administrative boundary in changeset/175738531. changeset/175738531 |
| 129425915 | 12 days ago | One node adjusted in this changeset has been reverted in changeset/175737446. changeset/175737446 |
| 174997736 | 15 days ago | When someone comments on any of your changesets, you should almost always respond, even if that answer might be "I don't know" - or if you fixed the mistakes yourself, as it helps prevent further issues on OSM and builds trust with other mappers. Edits of people who never respond may be treated as less trustworthy. Per the OSM Wiki, it states: >"You should almost always respond, even if that response might be "I don't know" – or if you fixed the mistakes yourself. Public communication between mappers is crucial to building trust. Edits of people who never respond may be treated as less trustworthy." Please keep in mind that you will not be banned just for making mistakes. We understand it takes time to learn how OpenStreetMap works. However, unresponsive users who continue questionable edits despite being contacted multiple times first will get a warning (so-called 0-day block), then progressively longer blocks on their account. |
| 109048711 | 17 days ago | Just to follow up, I don't think "Unknown Building" would be a good name for a building you don't know the type of or don't know the name of. If you don't know what type of building it is or don't know the building's name after mapping it, I don't think you need to put "Unknown Building" in the name tag of buildings. You can just leave it without a name for now. Once you know the type of a building or its name, you can specify the type of building, add the building's name, and other building tags. |
| 174997736 | 17 days ago | Also, don't remove tags you don't understand. Just because you come across elements with tags you don't understand or have no meaning to you doesn't automatically mean you should remove them. They may have been added for a specific purpose. If you think they might be junk, then try to contact the author first. You can also contact me if you have any questions or concerns about my map edits. My map edits are usually adding valid map data and modifying existing map data to reflect changes in the real world while preserving the history on OSM (e.g. not deleting existing map elements and redrawing them). osm.wiki/Good_practice#Don't_remove_tags_that_you_don't_understand Don't remove objects that you don't need or don't like. Be brave in what you add, but careful in what you delete. You should certainly not remove map details that someone else added. It is known that not everything is always needed by everyone, but you should not remove something just because you do not need it, you think it is stupid, or the validator ordered it so. Per the OSM Wiki, it states: >"Be brave in what you add but careful in what you delete. Someone added a catenary mast or a manhole? And you think it's stupid or unnecessary because it doesn't display on the map? Inaccurate mapping is not forbidden, but you certainly should not work the other way and remove details that someone else added. It is known that not everything is always needed by everyone, but you should not remove something just because you do not need it / you think it is stupid / the validator ordered it so."
|
| 174997736 | 17 days ago | You don't need to delete existing roads and redraw them anew. It's better to modify their geometry or change their tags instead of deleting the roads. This helps preserve the history on OSM. I talked to you about deleting existing elements and redrawing them anew on the map in a few of your changesets (You can find your discussed changesets here: https://resultmaps.neis-one.org/osm-discussion-comments?uid=9370870). You just have to modify their geometry or edit their tags instead of deleting the elements. I saw in this changeset that you deleted a driveway as seen here: way/20597830 and then drew an alley in its place. You can just change the road's tags instead of deleting the road and redrawing it anew. For the driveway you deleted, you could've just changed its tags to highway=service and service=alley and modified its geometry, instead of deleting the driveway and drawing a new alley in its place. I do encourage you to read this OSM Wiki article on keeping the history on OSM, so this is brought to your attention to keep in mind for the future: osm.wiki/Keep_the_history I also encourage you to read the OSM Wiki article on good practice on OSM, so you can be a better OSM mapper next time: osm.wiki/Good_practice |
| 174996289 | 17 days ago | It looks like you edited or adjusted the commercial area in this changeset rather than just "Add" it since you already added it. |
| 164057264 | 18 days ago | I corrected the direction of the Gateway Lane Northeast one-way roads in changeset/175475244 by clicking on the "Change Direction" button in the "One Way" field of the ways in iD editor. |
| 164057264 | 18 days ago | I noticed that you tagged the Gateway Lane Northeast ways way/1371012842 and way/1371012843 as oneway=yes, but tagged them as going in the wrong direction (I can see the error you made on the map). As a result, those Gateway Lane Northeast ways appear to go in the direction opposite of where they're actually supposed to be going. The oneway tag is defined in relation to the direction the linear feature ("way") is drawn in OpenStreetMap. If the oneway restriction is in the opposite direction to the drawn way, the fix in most cases is to turn the way around ("reverse way" tool in the map editors) and apply oneway=yes, or by clicking on the "Change Direction" button in the "One Way" field of the selected way after applying the oneway=yes tag. Per the OSM Wiki article on the oneway tag, it states: >"Note that the oneway tag is defined in relation to the direction the linear feature ("way") is drawn in OpenStreetMap. Identifying the direction of a way describes how to determine the direction of a way in the main OpenStreetMap editors."
In the United States, we drive on the right-hand side of the road, not the left-hand side. If you tag a road as one-way but tag it as going in the wrong direction, this could cause routing software to direct drivers to go on the wrong side of the road or go the wrong way on a one-way road, and potentially cause an accident or cause a driver to get stopped by police, all because of the error you made on OSM. Many different data consumers use OpenStreetMap data, and rideshare services such as Uber and Lyft also use OSM data for routing. When tagging roads as one-way, make sure to tag them as going in the correct direction based on signs or pavement markings seen on the ground or street-level photos like Bing Streetside and Mapillary, so routing software doesn't direct drivers to go on the wrong side of the road or go the wrong way on a one-way road. I would encourage you to read this OSM Wiki article on the oneway tag: oneway=* As a future reference, here is an OSM Wiki article on major OpenStreetMap consumers, containing some of the companies that use OSM data for their maps or services: osm.wiki/Major_OpenStreetMap_consumers
|
| 113457771 | 20 days ago | Just an FYI, you could have tagged the former Chickahominy Drive way (way/59149099) as old_name=Chickahominy Drive and then renamed it to Givens Lane, rather than merging the former Chickahominy Drive way into its adjoining Givens Lane way. The former street name is also noted in the way's import tags ("bburg:"). It would be good to take a few steps to preserve the history on OSM. It's generally better to modify the existing elements instead of deleting them. osm.wiki/Keep_the_history |
| 172918134 | 23 days ago | I looked at Bing Streetside street-level imagery of this location and saw that the no left-turn restriction is for certain hours on school days only. You could've tagged the turn restriction relation as "restriction:conditional=no_left_turn @ <condition>" rather than removing a way from the relation. osm.wiki/Conditional_restrictions#Turn_restrictions |
| 172918134 | 23 days ago | I looked at Bing Streetside street-level imagery of this location and saw that the no left-turn restriction is for certain hours on school days only. You could've tagged the turn restriction relation as "restriction:conditional=no_left_turn @<condition>" rather than removing a way from the relation. osm.wiki/Conditional_restrictions#Turn_restrictions |
| 172918134 | 23 days ago | What do you mean by "no restriction"? In this changeset, you removed a way from a turn restriction relation. |