Matt McCutchen's Comments
Changeset | When | Comment |
---|---|---|
110996461 | almost 4 years ago | Hi! I've been trying to keep the construction status of this area up to date on OSM for the last few months, and I noticed that your changeset opens several important roads and closes others. I briefly visited the area yesterday and confirmed that the status of these roads is still as it was in my osm.org/changeset/110811542, not as in this changeset: most importantly, a segment of Executive Boulevard is still closed and the entirety of Grand Park Avenue is still open. Where did you get your information? From imagery? (It looks consistent with the current "Esri World Imagery".) Remember that imagery can be months or years out of date, so you should think carefully before overwriting OSM data recently changed by other users in a survey with your own data based on imagery. Is it OK with you if I go ahead and revert the open/closed status changes? In the process of making those changes, your editor seems to have created, deleted, and repurposed several way objects with no net effect on the resulting geometry, which makes the object-level history more confusing to view. I'd like to take the opportunity to revert those object manipulations as well so that it will be easier for users to compare the object-level state before your changeset and after my partial revert. I'll keep the changes from your changeset that seem plausible to me:
Concretely, the changeset I'm proposing is https://mattmccutchen.net/private/20210927-restore-exec-grand-park.osm . You can open it in JOSM and review the specific changes by searching for "modified OR deleted" with "all objects" enabled and pressing the "History" button on individual objects. Is it OK with you if I submit this now? Thanks for your attention! |
110931126 | almost 4 years ago | Per the discussion at https://help.openstreetmap.org/questions/81647/best-practice-for-mapping-5-month-road-closure-with-approximate-end-date, this may not be the ideal solution, but it should be no worse than the status quo. |
104519460 | about 4 years ago | Unfortunately, this change seems to have made pedestrian routing _less_ accurate in some cases, at least in OsmAnd. From what I read, the difficulty of pedestrian routing is a known problem in OpenStreetMap, and it's acceptable to not alter my mapping to accommodate routing limitations. My hope in mapping sidewalks separately was to provide some information about potential wheelchair accessibility. I guess I'd like to specify that pedestrians are allowed to cross the residential roads anywhere, but there is a kerb, and most sidewalks have frequent breaks in the kerb that I did not want to do the work to map (house driveways), while the sidewalks adjacent to the park have only the mapped kerb breaks. |
104702650 | about 4 years ago | The connectivity and restrictions are from a survey, while the coordinates of added elements are from the imagery.
|
104519460 | about 4 years ago | Sidewalks are traced from Esri Clarity (with extrapolation where the view was blocked by trees) since my GPS receiver is less accurate. Crossings are surveyed. |
104474856 | about 4 years ago | This change is big enough and uses enough mapping conventions new to me that at least some minor things are probably wrong. Coordinates are approximated from imagery and/or my phone's GPS receiver and may be off a bit. |
104382890 | about 4 years ago | In addition to the important correction to the "from" and "via" members, I changed this from "only_left_turn" to "no_straight_on" because OsmAnd had trouble with the "only_left_turn" in my tests. |
104382204 | about 4 years ago | This is an attempt to correct all of my previous changesets according to https://help.openstreetmap.org/questions/80073 . With the help of Bing and Esri imagery, I was able to come up with reasonable estimates of the starting points for all but one turn:lanes tag. I hope I got everything right; it doesn't seem worth the trouble to check it all again. |
102289043 | over 4 years ago | Clarification: Having read osm.org/note/1791672, I confirmed that the sign said "Elementary" and not "High" school, but I didn't check the spelling of the rest of the name. |
101934210 | over 4 years ago | FWIW, I was mistaken: OsmAnd seems to recognize access:conditional. I think this code (https://github.com/osmandapp/OsmAnd/blob/feadaa0da5804d9f3312e6df852ef96093fb82ec/OsmAnd-java/src/main/java/net/osmand/binary/BinaryMapRouteReaderAdapter.java#L237) is involved, though I haven't traced the entire data flow. |
101934210 | over 4 years ago | Apparently access:conditional is well-established, but I can't find a router that supports it to test this change. :( |
101886777 | over 4 years ago | The segment of eastbound O Street that I changed is somewhat artificial to OSM's representation of the complex intersection, and I didn't see any turn:lanes signage for it specifically. But the combination of the turn:lanes I added with the existing value for southbound 1st Street seems to correctly model the reachability of eastbound New York Avenue and O Street from the left three southbound lanes of 1st Street (see osm.wiki/Relation:connectivity#Basic_summary). Indeed, it fixes the lane guidance in OsmAnd for the maneuver from southbound 1st Street to eastbound New York Avenue. It might be better to use connectivity relations directly from southbound 1st Street to eastbound New York Avenue and O Street, but fewer routers support those, and I'm unsure of the exact target lanes on eastbound New York Avenue and O Street, which would need to be stated in the connectivity relations. |
101884419 | over 4 years ago | turn:lanes:conditional seems to be very unofficial (in particular, I'm unsure how I'm supposed to protect the semicolon in the conditional value; I took a look at taginfo for prior art), but that's the fun of OpenStreetMap, right? I'm having fun anyway. If enough instances of turn:lanes:conditional are added, maybe routers will be motivated to learn to parse it. |
101883895 | over 4 years ago | Concerns:
|
101605366 | over 4 years ago | Information about the construction project is at https://www.montgomerycountymd.gov/whiteflint/ . I used the vehicle detour map there to plan where to survey, but I didn't copy any information directly from that map because I was unsure of the licensing. I used GPS traces to locate the closures and tried to get the connectivity right, at least for motor vehicles. Review requested!!! I've probably done some things not according to convention, but I hope this is at least much better than the status quo. And someone who cares about pedestrian and bicycle access is invited to update that part. Discussion of the complete update to this area beyond this initial changeset can take place at osm.org/note/2590993 . |
101464915 | over 4 years ago | Signs I saw:
|
100557581 | over 4 years ago | - The time-based restriction at westbound Blackthorn St is "do not enter" from any direction (surveyed 2021-02-25).
|
97526511 | over 4 years ago | I don't think this change is particularly important: most of the minor roads don't have turn:lanes tags. I'm just having fun because the local transportation department recently signed these turn lanes at my request. |
96163211 | over 4 years ago | I know by survey that a left turn from northbound Citadel Avenue to westbound Old Georgetown Road is blocked by the median on Old Georgetown Road. I don't know how far the median extends; I've taken a guess from the Bing imagery. I wasn't at all confident of how to make this edit, but I hope what I did will be better than the status quo until someone more experienced takes a look. |
63079408 | almost 7 years ago | Oops, the changeset description is backwards: the segment was moved from Greenhill Road to Old Lancaster Road. |