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

Render cycleable sidewalks tagged on roads #464

Merged
merged 7 commits into from
Nov 29, 2020
Merged

Conversation

brainwad
Copy link
Contributor

Adds two new colours of cycletrack, which represent sidewalks that are primarily for pedestrians but either designated for bicycles (drawn in the same lighter blue as a footway designated for bicycles is) or merely allowed for bicycles (drawn in the same green as a footway that allows bicycles).

image
image

The ranking of road-based bicycle features that are drawn in the border of the road is now:

A cycletrack (cycleway=track) or a segregated sidewalk, that allows bicycles (sidewalk:bicycle!=no sidewalk:segregated=yes): solid dark blue
A sidewalk designated for bicycles (sidewalk:bicycle=designated): solid lighter blue
A bike lane (cycleway=lane): dashed dark blue
A sidewalk allowing bicycles (sidewalk:bicycle=yes): solid green
A shared busway or lane: very dashed dark blue

The reasoning for splitting the sidewalks is that a designated off-road path is probably more comfortable than an on-road lane, but a non-designated sidewalk can potentially quite bad infrastructure and so a dedicated lane is better; but then sharing with pedestrians is still better than sharing with cars, so shared road lanes come after it.

Fixes #409.

sidewalk:bicycle=designated is still preferred over cycleway=lane, because that is probably a decent cycling facility. But a sidewalk:bicycle=yes has no guarantee of quality, even though it is off-road. It is porbably still better than a mere shared lane, however.
@Florimondable
Copy link
Member

There is 7 sidewalk:bicycle=designated don't worth a special color for it.

And 679 for sidewalk:bicycle, does it worth the effort and the rendering cost ?

https://taginfo.openstreetmap.org/keys/sidewalk%3Abicycle#values

@brainwad
Copy link
Contributor Author

brainwad commented Nov 28, 2020 via email

@Phyks
Copy link
Member

Phyks commented Nov 29, 2020

sidewalk:bicycle isn't a valid tagging, that's why its so uncommon (and I don't even bother to render it). Look at sidewalk:(both/left/right):bicycle : https://overpass-turbo.eu/s/10Fl

There are around 10k sidewalk:[left/right/both]:bicycle=yes tags around, this is definitely something worth rendering.

However, there are only 32 sidewalk:both:bicycle=designated + 292 (left case) + 416 (right case) which is < 1k objects worldwide.

Given the tag adoption, I think it's not worth making the distinction (perf-wise). I would simply check the value against anything but no and dismount and have a single "sidewalk" render.

project.mml Outdated
@@ -675,10 +683,16 @@ Layer:
END AS oneway,
CASE
WHEN tags->'cycleway:left' IN ('track', 'opposite_track') THEN 'track'
WHEN tags->'sidewalk:left:bicycle' <> 'no' and tags->'sidewalk:left:segregated' = 'yes' then 'track'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor (stylistic) comment but could we have an homogeneous SQL code convention here?

Almost all of CyclOSM SQL queries are written using != operator (instead of <>) and uppercase keywords (AND / THEN). Thanks!

@Phyks
Copy link
Member

Phyks commented Nov 29, 2020

I have a SQL query style comment, and I'd like to drop the dedicated sidewalk:*:bicycle=designated render. Once done, I think it is a go for merge.

@brainwad
Copy link
Contributor Author

brainwad commented Nov 29, 2020 via email

@Florimondable
Copy link
Member

sidewalk:bicycle isn't a valid tagging, that's why its so uncommon (and I don't even bother to render it). Look at sidewalk:(both/left/right):bicycle : https://overpass-turbo.eu/s/10Fl

My bad.
So yes let's try with a "solid lighter blue".

@brainwad
Copy link
Contributor Author

I think path green would be better, since most usages are not designated, so having the same colour as highway=footway bicycle=yes makes more sense. As a bonus, it's more distinguishable from cycletracks when its green.

@Florimondable
Copy link
Member

ok

@brainwad
Copy link
Contributor Author

The area above, with both designated and non-designated cycleable sidewalks now looks like this:

image

@Phyks
Copy link
Member

Phyks commented Nov 29, 2020

Thanks!

@Phyks Phyks merged commit 1371c41 into cyclosm:master Nov 29, 2020
@brainwad brainwad deleted the sidewalk branch November 29, 2020 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Render sidewalk= similar to cycleway=track
3 participants