vorpalblade's Comments
| Changeset | When | Comment |
|---|---|---|
| 72030111 | over 6 years ago | Just so you know, Mesa County GIS has allowed us to use their (much more) up to date imagery for tracing in OpenStreetMap. You can see it by clicking on the "Background Settings" button in iD (keyboard shortcut "b") and select "Mesa County GIS Valleywide 2018". Thank you for contributing, and I hope to see more edits from you in the future,
|
| 72099311 | over 6 years ago | I do not see any major issues with this changeset. That being said, I would like a better changeset comment than "GIS Data". In this case, it looks like you were modifying motorway exits and their links, so I would have expected a comment like "I modified motorway exits in Grand Junction" or something like that. This helps other people look at your changeset and see possible accidents (for example, did you mean to edit the boundary for Clifton? You did in this changeset). Also, Mesa County GIS has allowed us to use their (much more) up to date imagery for tracing in OpenStreetMap. You can see it by clicking on the "Background Settings" button in iD (keyboard shortcut "b") and select "Mesa County GIS Valleywide 2018". Thank you for contributing, and I hope to see more edits from you in the future,
|
| 72196396 | over 6 years ago | Just so you know, Mesa County GIS has allowed us to use their (much more) up to date imagery for tracing in OpenStreetMap. You can see it by clicking on the "Background Settings" button in iD (keyboard shortcut "b") and select "Mesa County GIS Valleywide 2018". Thank you for contributing, and I hope to see more edits from you in the future,
|
| 72196021 | over 6 years ago | This changeset generally looks good. About the only concern I have is that you changed the building=industrial tag on way/497002707 to building=yes. Is there a reason why? ( The original building shape and information was an import from Mesa County GIS ) Also, Mesa County GIS has allowed us to use their (much more) up to date imagery for tracing in OpenStreetMap. You can see it by clicking on the "Background Settings" button in iD (keyboard shortcut "b") and select "Mesa County GIS Valleywide 2018". Thank you for contributing, and I hope to see more edits from you in the future,
|
| 70569180 | over 6 years ago | I'll note that I said that he didn't have to add anything above addr:street. I didn't say he *shouldn't*. I just view it as a waste of time when (most) data consumers look at the surrounding boundaries. In this case, looking at the administrative boundaries, he was probably right to add an "addr:city", although IIRC, I've seen some people use Fruitvale and Grand Junction interchangeably in addresses. You do have a point on the postcode, but I'd rather go through the import process for the zip codes. Part of the issue that I've had with having addr:postcodes and addr:city tags on addresses is that when boundaries change, it seems that no one bothers to update the interior addr tags. (I've seen this with street name changes as well, and I still have to fix that in Palisade, CO). It *might* be an extra step for data consumers, but I would hope they would be taking that extra step anyway, if only to ensure they don't have an address with "addr:city=Palisade" in the middle of Grand Junction. All that being said, I can see why you would leave addr:city and addr:postcode on every single address in an import. With an import, I'd leave as much information as possible. |
| 70556873 | over 6 years ago | This looks good.
|
| 70564735 | over 6 years ago | I presume that the building has a plaque somewhere that says "WCCC Building B". If so, this edit looks good. If the building ref is "BB", then I would move that to "ref=BB". |
| 70569180 | over 6 years ago | This is generally good. You can make it look better by using the "square" tool (keyboard shortcut is "q", you can also right-click on the building outline and click on the "Square" tool. Like I said in my other comment, you don't have to add anything above addr:street, since most software will "guess" it by using the surrounding boundaries. Also, when drawing outlines, I would _highly_ recommend using either the "Mapbox Satellite" imagery or "Mesa County GIS Valleywide 2018" imagery (under "Background Settings" ("b" keyboard shortcut)). |
| 70569281 | over 6 years ago | You did the name/address/payment tags well. One caveat is that you don't have to add addr:city, addr:postcode, and addr:state (most navigation software looks at the administrative boundaries for those). I don't know if "General Nutrition Center" is a supermarket (which you have tagged it as) or a healthcare location. If it is a supermarket, its fine. You poorly formatted the opening_hours tag. You had the right general idea. I use http://openingh.openstreetmap.de/evaluation_tool/ to avoid malformed opening hours. In this case, it should be "opening_hours=Mo-Fr 10:00-20:00; Sa 10:00-18:00; Su 11:00-17:00". Notice that Sat -> Sa and Sun -> Su, there is a separator (";") between the days (a "," may work as well, but for simple cases just use the semi-colon). I hope this helps,
|
| 70303007 | over 6 years ago | Just an FYI, OpenStreetMap is a public map. I'm mentioning this since you have put semi-private information on the map. You might think your garage is the "coolest place to play ping pong in the summer", but do you want other people to know that and randomly drop by? Or do you just want your friends to know that? If you meant to do that, then great!
In any case, welcome to OpenStreetMap. |
| 70052098 | over 6 years ago | Since you've asked for a review, here we go:
I've left everything alone, so you can play around with it and see the difference. Welcome to OpenStreetMap, and if you have any questions, send me a message! |
| 67956816 | almost 7 years ago | *make it not a roundabout |
| 67956816 | almost 7 years ago | This *may be* a bad edit since it is possible for it to have two-way traffic on some sections, which would make it a roundabout. Please direct all comments about this changeset to @vorpalblade77-kaart |
| 49150328 | almost 7 years ago | Are you certain that Ute and Pitkin Avenue are primary instead of trunk? I can see how you might have decided to classify them as primary, since they have quite a few intersections, but then there are trunks starting semi-randomly. |
| 60721009 | over 7 years ago | Honestly, when I was reverting 57870860, I just kept the tags that were existed prior to that changeset and any tags that existed in that changeset that didn't conflict. It looks like they can be removed, since they are the only existences of those tags. |
| 57870860 | over 7 years ago | Absent objections, I'm going to revert this change. Problems with it:
Official street names can be found here:
Again, absent objections, I intend to revert this changeset by July 20, 2018, but probably sooner. |