Reemal_lyft's Comments
Changeset | When | Comment |
---|---|---|
170124 | almost 5 years ago | Excellent work done on the modeling of the intersection of San Tomas Expressway and Monroe Street, all your splits and lane count tags are correct. When modeling ways with center turn lanes, please ensure that the way is drawn along the middle of the center turn lane and not towards the sides (Please refer to the screen shot and you have drawn it correctly here https://api06.dev.openstreetmap.org/way/4305501732). Please correct lane direction tags on (https://api06.dev.openstreetmap.org/way/4305501726), the correct tags should be lanes:forward=3 and lanes:backward=1. Please split the way at (https://api06.dev.openstreetmap.org/node/4319572591) and then correct lane direction tags on (https://api06.dev.openstreetmap.org/way/4305501711), the correct tags should be lanes:forward=1 and lanes:backward=2. Please split at the start of the turn lane (https://api06.dev.openstreetmap.org/way/4305501735) at nodes (https://api06.dev.openstreetmap.org/node/4319572682 and https://api06.dev.openstreetmap.org/node/4319572680), please move these nodes 1st. |
170118 | almost 5 years ago | Warburton Avenue needs to be split at this intersection (https://api06.dev.openstreetmap.org/node/4319571081#map=19/37.35579/-121.96497). The lane count should be removed West of the intersection (https://api06.dev.openstreetmap.org/way/4305501352#map=17/37.35579/-121.96390) as there are no road markings present. Please remember that lane count tags cannot be added without lane markings (Please refer to confluence for more guidance). Please refer to the screenshot on where to split and add lane count for Bento Street, San Tomas Expressway and El Camino. You have correctly split and add tags to this segment of the way (https://api06.dev.openstreetmap.org/way/4305501664 and https://api06.dev.openstreetmap.org/way/4305501663), use this as a reference. Please reverse the lane direction for this way (https://api06.dev.openstreetmap.org/way/4305501654). |
169990 | almost 5 years ago | Hi, very good job on this change set. Keep up the good work. |
169989 | almost 5 years ago | Hi, you have modeled the parking lots well. Please change the direction of these roads to oneway (https://api06.dev.openstreetmap.org/way/4305501606), ((https://api06.dev.openstreetmap.org/way/4305501583) (https://api06.dev.openstreetmap.org/way/4305501596) and (https://api06.dev.openstreetmap.org/way/4305501583), always remember to look for painted arrows in the parking lots and the alignment of parking spaces to determine the direction of the way. Overall good work. |
169965 | almost 5 years ago | Hi, your road classification looks correct, if you ever have doubts on what classification should the road be, always refer to confluence or OSM wiki to be sure. Street names also look correct, it is always a good practice to use the latest street level imagery like mapillary to check for street signs.
|
169950 | almost 5 years ago | Hi, your geometry is very good and road classification (highway=service) has been correctly applied. This service road that you have modeled is slightly incorrect (https://api06.dev.openstreetmap.org/edit?way=4305501223#map=20/37.35149/-121.96216), if you look closer, you will see there are 2 arrow painted on the entrance of the parking lot, which indicates that you can only enter and there is a EXIT ONLY painted on road just above this node (https://api06.dev.openstreetmap.org/node/4319570605). The way needs to be changed into a one way.
|
169944 | almost 5 years ago | Hi, very good work on your modeling of El Camino Real. The geometry is really good, it is a good practice to draw the ways in the middle of the roads. You current geometry can be improved by eliminating extra nodes that you have drawn in. The extra node (it appears as a white node in OSM) like (https://api06.dev.openstreetmap.org/edit?way=4305501158#map=21/37.35243/-121.96319) can be either deleted or merged with another node close to it example this node (https://api06.dev.openstreetmap.org/edit?way=4305501157#map=21/37.35228/-121.96063) can be merged with this node (https://api06.dev.openstreetmap.org/edit?way=4305501157#map=21/37.35228/-121.96063).
|