Xvtn's Comments
| Changeset | When | Comment |
|---|---|---|
| 140036750 | about 2 years ago | Hello Vaasan and commenters! I'm going through changesets that have requested a review. Looking over this I'm thinking highway 139/50 going north from exit 15 should not be one-way. But I can't be sure I guess - Can any locals weigh in? |
| 140434171 | about 2 years ago | Hi, and thanks for your contribution! Since you requested a review, I looked over your changes. For the surface tag on that sidewalk, I think the correct one would be concrete rather than concrete_sidewalk. I went ahead and changed that. Thanks again and welcome to OSM! |
| 141003547 | about 2 years ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I went over your changes. Looks great to me, no problems. Thanks for your contributions! |
| 141367051 | about 2 years ago | Hi, and thanks for your contributions! In the future, please use a descriptive changeset comment. An example here might be "Added buildings, fixed Sunset Drive road name, and other minor adjustments". That helps other mappers review what you changed. Thanks again and welcome to OSM! |
| 141388399 | about 2 years ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I looked over your recent changes. Looks like someone fixed it already, but in the future it's best to avoid abbreviations in OSM tags, in addresses for example. I also added a more descriptive office=educational_institution tag. Thanks for your contributions! |
| 142214607 | about 2 years ago | Looks like there were already some Wells Fargo tags on the building here. To avoid duplication, I merged them. |
| 142214732 | about 2 years ago | In OpenStreetMap tags, it's best to avoid abbreviations. So for addresses, it should be "West 11000 North" rather than "way/11000 N". That's because it's easy for computers to shorten/abbreviate words, but can be more tricky and error-prone to expand them. Here's some more info: osm.wiki/Abbreviations I fixed the address tag. Thanks! |
| 142214866 | about 2 years ago | Since removing features is somewhat of a more serious action, in the future I'd suggest adding a source for the info when deleting things -- such as an in-person observation, using their website, etc. |
| 142242773 | about 2 years ago | Hi, and welcome to OpenStreetMap! Since you requested a review, I went over your recent changes. Looks great overall, I did leave comments with tips on a couple of your other changesets. |
| 142214915 | about 2 years ago | For this building, at first glance it looks like it's missing a name, but the outer school grounds already are named. IMO, it's best to avoid duplication in situations like that. |
| 142850683 | about 2 years ago | Hi, and welcome to OpenStreetMap! This looks great, no complaints from me. Thanks for your contributions! |
| 142402041 | about 2 years ago | Hi, and welcome to OpenStreetMap! One request - It's best to keep your changesets small - that means saving your work in one area before moving on to the next. That makes it easier for others to review things.
|
| 145086789 | about 2 years ago | Oops, I had my filter tool set up wrong. You didn't request a review, haha. |
| 145086789 | about 2 years ago | Hi, and thanks for your contributions! Since you requested a review, I looked over those recent gate changes of yours. I might be mistaken, but I think a gate as a way is fine too, as long as there is an intersecting node, and that node is marked as a gate too.
|
| 145087360 | about 2 years ago | Hi, and welcome to OSM! Looks great. |
| 144679046 | about 2 years ago | Oh, I'm not sure how it works in your system, AFAIK checking the box just adds review_requested=yes tag to your changeset, which anyone can view. I'm using a tool called OSMCha to review changesets with that tag. (I'm just some rando.) But perhaps your OSM.us tasks setup has people reviewing all changes specific to that too? |
| 143439615 | about 2 years ago | I went ahead and made that change. |
| 143439615 | about 2 years ago | For situations like this where it's clear that parking is operated by two separate businesses, I think it's best to make them separate lots. (That way someone can come through later and give them potentially different tags like access=* or name=*.) |
| 143441367 | about 2 years ago | Looks great! Thanks for contributing. See my note on your other changeset regarding multipolygons. |
| 143443437 | about 2 years ago | For areas with voids in them like this parking lot, you can use a multipolygon. If you don't know already, they're features that have some number of outer and inner members. I made one with your original lot outline as the outer, and some other lines as inner. An easy way to create these in iD is to draw the outer, tag it as appropriate, then use the Line tool to draw the inner. Then select both of them and press "C". That'll auto-create a relation and move the tags over. |