Skip to content
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

Do not ask if there is an _opposite_ cycling lane for roundabouts #5959

Closed
gsurrel opened this issue Oct 14, 2024 · 6 comments · Fixed by #5961
Closed

Do not ask if there is an _opposite_ cycling lane for roundabouts #5959

gsurrel opened this issue Oct 14, 2024 · 6 comments · Fixed by #5961
Assignees

Comments

@gsurrel
Copy link

gsurrel commented Oct 14, 2024

StreetComplete asks for cycling lanes in the two directions for one-way streets (which is good as some streets accepts cyclists in the two ways). However, I'm not sure if there is any roundabout that can be taken in the opposite way with a bike.

How to Reproduce
Screenshot_20241014-080213_StreetComplete

Expected Behavior

Versions affected

  • 59.2
@gsurrel gsurrel added the bug label Oct 14, 2024
@gsurrel gsurrel changed the title Do not ask if there is an _opposite_ cycling lane for roundaboits Do not ask if there is an _opposite_ cycling lane for roundabouts Oct 14, 2024
@westnordost westnordost removed the bug label Oct 14, 2024
@westnordost
Copy link
Member

It doesn't "ask". It's an overlay, not a quest. This is a difference. E.g. for private roads, the surface quest is never asked, but in the surface overlay, the information is still marked as missing.

That said, there was a similar issue recently and I thought I changed it so that the inner part of a roundabout is not marked as missing even if it is missing. I'll check.

@westnordost
Copy link
Member

westnordost commented Oct 14, 2024

I thought I implemented this in #5716. But looking closely, it doesn't look like it. At least not this exact feature request.

@FloEdelmann

This comment was marked as resolved.

@westnordost
Copy link
Member

westnordost commented Oct 14, 2024

https://www.openstreetmap.org/way/1084085781

@westnordost
Copy link
Member

Deciding whether cycleway tagging is not expected per-side is more complex that one might think, because it's not only junction=roundabout but also =circular and also things like highway=..._link or dual_carriageway=yes, plus, more importantly, which side of a road is the contra-flow-side depends both on tags (e.g. it could be junction=roundabout + oneway=-1) but also on in which country one is in (left hand traffic in United Kingdom).

So, I have been beginning to doubt if this is worth the added complexity or whether the overlay should just show the situation "as is". On the other hand, I did make the decision to regard this in #5716 (comment) and it is somewhat important to stay consistent in decisionmaking.

@westnordost
Copy link
Member

Implemented in #5961, however, as there are no unit tests for cycleway overlay coloring, I'd feel better if another pair of eyes had a look over the changed code. (I tested it once at a roundabout in Lausanne and then in two roundabouts in Peterborough)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants