Xvtn's Comments
Changeset | When | Comment |
---|---|---|
140036750 | over 1 year 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 | over 1 year 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 | over 1 year 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 | over 1 year 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 | over 1 year 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 | over 1 year ago | Looks like there were already some Wells Fargo tags on the building here. To avoid duplication, I merged them. |
142214732 | over 1 year ago | In OpenStreetMap tags, it's best to avoid abbreviations. So for addresses, it should be "West 11000 North" rather than "W 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 | over 1 year 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 | over 1 year 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 | over 1 year 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 | over 1 year ago | Hi, and welcome to OpenStreetMap! This looks great, no complaints from me. Thanks for your contributions! |
142402041 | over 1 year 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 | over 1 year ago | Oops, I had my filter tool set up wrong. You didn't request a review, haha. |
145086789 | over 1 year 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 | over 1 year ago | Hi, and welcome to OSM! Looks great. |
144679046 | over 1 year 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 | over 1 year ago | I went ahead and made that change. |
143439615 | over 1 year 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 | over 1 year ago | Looks great! Thanks for contributing. See my note on your other changeset regarding multipolygons. |
143443437 | over 1 year 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. |