Baloo Uriza's Comments
| Changeset | When | Comment |
|---|---|---|
| 104910495 | It's all good, welcome aboard! Don't take this as being overly critical, my intention is to be constructive. And also save a little work on maintenance down the line. |
|
| 104910495 | Really looks like this could have been a good edit with better attention paid to overlaps, alignment and geometry.
|
|
| 104912920 | http://www.oklegislature.gov/cf_pdf/2003-04%20COMMITTEE%20SUBS/scs/sb350%20cs.pdf
|
|
| 104688698 | highway=construction implies access=no. I usually don't (and generally don't consider it best practice) to retag access=no, as highway=construction, as it's generally tagged and consumed already, already overrides as not being accessible. This also makes reversibility easier, since all you have to do is change construction=* to highway=* after selecting what's now open and remove the opening_date without having to worry about other keys. I would generally suggest the best practice on highway=construction is set the access to what it will be once it opens. This gets especially handy in more complex access situations (such as around weigh stations and emergency crossovers) that are closed for construction. |
|
| 102590620 | Nice catch, I thought it was a school based on top down profile and position relative to a strip-park with a MUP, which tends to be the prime nonmotorized connection to schools. |
|
| 102588824 | Yes, that sounds fine. Not specific to this change but based on the pattern visible, I think duplicating the ref=* in the name=* in general is not a great way of handling route numbers in general. |
|
| 102588824 | I'm fairly sure OK 82A was retired in 2018, and was contained entirely within Langley, connecting OK 82 to OK 28, so I'm not super into the accuracy of this change. Second, name=* is only the name. So even if this was State Highway 82A, then it would be ref=OK 82A, and if it didn't have a name and only a number like that, then it'd be noname=yes instead of name=State Highway 82A.
|
|
| 102489252 | Looks like you added it yourself. Be aware that advertising copy isn't allowed in OSM.
|
|
| 102353631 | landuse=residential is good for invididual lots or contiguous lots, but not for entire subdivisions.
|
|
| 102350093 | Not bad, but areas go to the edge of the area, not attached to the roadway centerlines.
|
|
| 101858935 | This almost certainly needs to be completely redone. Could I get evidence that these hundreds of buildings, none of which have any relation to what appears on the bing imagery, is even remotely possible to be something other than a house? Correct tagging and good drawing is pretty important. |
|
| 101779866 | I feel this, but I'd have waited for authorities to have given up first. |
|
| 101779866 | I don't think this is a permanent enough fixture to qualify to be an object in OSM. |
|
| 101683229 | No physical separation here, it's just a high radius corner.
|
|
| 101683707 | This seems rather inventive to have a graduated curve at a T intersection.
|
|
| 101683940 | name=* is only the name, it is not ref=*. osm.wiki/Names#Name_is_the_name_only
|
|
| 101409139 | For the landuse, I'd probably go "landuse=retail" and remove the name, since it's no longer a Conoco. |
|
| 101409139 | That sounds about right! |
|
| 101458544 | I forget the object, but if there was a relation I missed, that would make sense. |
|
| 101209894 | Does this road no longer go through?
|