-
-
Notifications
You must be signed in to change notification settings - Fork 366
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Separately mapped sidewalk is not detected #4494
Comments
(and there are many similar examples relatively nearby; e.g. https://www.openstreetmap.org/way/804793377, https://www.openstreetmap.org/way/629025256, https://www.openstreetmap.org/way/27314747 etc.) |
Sidewalk is only detected as sidewalk if it's close enough, for minimum distance at which a way is recognizes as a possible sidewalk see StreetComplete/app/src/main/java/de/westnordost/streetcomplete/quests/sidewalk/AddSidewalk.kt Line 78 in d215e40
|
hmmm... according to JOSM, there is about Trying to decipher: StreetComplete/app/src/main/java/de/westnordost/streetcomplete/quests/sidewalk/AddSidewalk.kt Lines 78 to 84 in d215e40
If so, it should be So, the comment claiming However, (if I didn't miss something) that calculation does not feel right. If you look at the JOSM picture with aerial background, given that there is no green protection between road and sidewalk, SC seems to estimate that center of footway should be no more than So its minimum average width in this case should IMHO be |
Note that this And if there is actual grass protection verge (e.g. in a middle left part of the picture, marked with |
So if my reasoning is not faulty, would something like this make sense? private fun getMinDistanceToWays(tags: Map<String, String>): Float =
(
(estimateRoadwayWidth(tags) ?: guessRoadwayWidth(tags)) +
(estimateParkingOffRoadWidth(tags) ?: 0f) +
+ 4f // average sidewalk width should also be accounted for (as sidewalk will be mapped in the middle of it, not on the side closest to the road)
(estimateCycleTrackWidth(tags) ?: 0f)
) / 2f +
+ 3f // + allow for kerbs, wider sidewalks, possibly segregated cycleways on sidewalks, lanes little larger than the 3m minimum
4f // + generous buffer for possible grass verge |
Since in the sidewalk quest, you can now actually select "displayed separately on map", having a filter that reduces the number of false positives even in edge cases at the cost of producing many false negatives (i.e. quest is not asked in places where it should be) is not really important anymore. ... to the point where I am entertaining the thought to completely drop this "min distance to ways" thing, as it is always better to specify the situation explicitly rather than relying on some fuzzy logic. On the other hand, of course, the filter makes sense insofar as otherwise a city that is comprehensively mapped with separate sidewalks (but consistently missing So, at least, in my opinion, the filter should at this point rather produce false positives than false negatives. I.e. for the cited calculation, rather not assume generous widths (like 4m for a sidewalk, 3.5 meters for a lane, big grass protection verge etc.) |
OK, I can accept that false positives are preferable here. Still, should some width (instead of current For example, over here legal minimum width for sidewalk is
or this
(conversion: |
I'd accept a PR that:
|
* include sidewalk width in calculation (and document it) to address #4494 * intentionally reduce widths way below realistic estimates see #4494 (comment) * adjust values * adjust values * more adjustment Co-authored-by: Tobias Zwick <newton@westnordost.de>
Sidewalk quest get asked even if there are nearby separate footways marked separately with
highway=footway
+footway=sidewalk
. The sidewalk quest should be skipped in such cases.How to Reproduce
sidewalk
quest and navigate for example to this way: https://www.openstreetmap.org/way/34926444Does this street have a sidewalk
quest gets askedExpected Behavior
Code itself is unfortunately too complex for me to follow, but comments in the code seem to indicate it should work that way:
StreetComplete/app/src/main/java/de/westnordost/streetcomplete/quests/sidewalk/AddSidewalk.kt
Lines 42 to 45 in d215e40
Versions affected
upstream StreetComplete
v47.1
, Android 10Same or similar problem seems to also be present in earlier versions too, e.g. https://www.openstreetmap.org/note/3186207, https://www.openstreetmap.org/note/3186207 etc
The text was updated successfully, but these errors were encountered: