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

one-way residential street do not show up with bicycle quest #1013

Closed
RubenKelevra opened this issue Apr 10, 2018 · 31 comments
Closed

one-way residential street do not show up with bicycle quest #1013

RubenKelevra opened this issue Apr 10, 2018 · 31 comments
Assignees

Comments

@RubenKelevra
Copy link
Contributor

RubenKelevra commented Apr 10, 2018

I found a bicycle lane which I seemed to have overseen before, so I looked at SC and it showed no quest.

I've tooked a look at the data with Vespucci and it revealed no tag for bicycle lanes.

I've voided the cache for quests, just activated the bicycle lane quest and reload the data. Still no quest-popup.

The street is a quite usual 4 way residential street with a center divider, that's why it's mapped as two one-way lanes, but from the tags I don't see a reason why it shouldn't be suitable for a bicycle lane quest.

Did I overlooked something?

Version is v5.0b1

I've added a yellow marking is next to the way:

@matkoniecz
Copy link
Member

Low maxspeed roads are excluded from this quest on assumption that cycleway infrastructure is unlikely in that cases.

@matkoniecz
Copy link
Member

It may or may not be a good idea.

@westnordost
Copy link
Member

I think low maxspeed roads are very unlikely to have bicycle infrastructure. I have no statistics, just my personal experience.

@RubenKelevra
Copy link
Contributor Author

Well, this street got one on both sides, actually not that uncommon 🤔

@rugk
Copy link
Contributor

rugk commented Apr 10, 2018

Hmm, also cannot follow the logic that "low maxspeed" makes it unlikely to have bicycle infrastructure. Actually, It's kinda the reverse, IMHO, asa high maxspeed may endanger bicyclists, so when there is a high maxspeed it is rather unlikely to have bicycle infrastructure there. E.g. on autobahns/highways there is no bicycle infrastructure… 😉 (i know, there are other reasons for that, but just so you get my point)

@matkoniecz
Copy link
Member

matkoniecz commented Apr 10, 2018

when there is a high maxspeed it is rather unlikely to have bicycle infrastructure there

That is exact reason why this roads may have cycleways along them, separate from traffic (or sidewalks with legal cycling or at least cycle lanes).

Are you seriously claiming that road with maxspeed 70 is less likely to have cycling infrastructure than residential road in maxspeed 30 zone?

Motorways are a special case and are excluded anyway...

@rugk
Copy link
Contributor

rugk commented Apr 10, 2018

separate from traffic

Okay, that's a good argument. I only thought of cycleway lanes or so. In these cases maxspeed 70 is likely less likely to have such a lane.

Then, however, one still has to explain why maxspeed 30 (or lower) is unlikely to have bicycle infrastructure. These streets are common in cities or villages and I think such infrastructure (even if it is just a bike lane) is not really rare there.

@westnordost
Copy link
Member

Present statistics that show otherwise and I will drop this requirement.

@westnordost westnordost added the feedback required more info is needed, issue will be likely closed if it is not provided label Apr 10, 2018
@rugk
Copy link
Contributor

rugk commented Apr 11, 2018

Maybe we could do since parsing of the data already mapped in OSM?

@matkoniecz
Copy link
Member

matkoniecz commented Apr 11, 2018

Maybe we could do since parsing of the data already mapped in OSM?

Yes, but I am not planning on doing that. I am 99,9% sure that (except highway=trunk, motorway and links) road with maxspeed above 30 are significantly more likely to have cycling infrastructure* than roads with maxspeed below 30.

*except oneway:bicycle=no in some places - if you count it as "infrastructure"

@westnordost
Copy link
Member

So, I will close this. If anyone wants to try to research there more and find some statistics (through overpass?) that show otherwise, you are welcome to open another issue.

@rugk
Copy link
Contributor

rugk commented Apr 11, 2018

"I'm planning to do that"_so statistics incoming.😉
In other words you could have left this open for the time until this final stat is made, to solve this issue for good and all!

@matkoniecz
Copy link
Member

"I'm planning to do that"_so statistics incoming

Who is quoted here? Because nobody in this thread posted that.

you could have left this open

One of prioritization methods is opening/closing issues. By closing this issue, with "you are welcome to open another issue." (if there are statistics confirming this) @westnordost communicated that it is not considered as high priority but if there is a proof that it is a problem then it would be a good reason to investigate it further.

@westnordost westnordost reopened this Apr 11, 2018
@rugk
Copy link
Contributor

rugk commented Apr 11, 2018

Who is quoted here?

You: "I am not planning on doing that" – sorry, my quote was not literally. 😃

And yes, I know it is okay to call for a new issue and close this one. It's just only one statistic is missing, then this issue can be closed finally. And as you pledged to provide that, why not let it open until you do? (Hope it's not too much work.)

@RubenKelevra
Copy link
Contributor Author

RubenKelevra commented Apr 12, 2018

That is exact reason why this roads may have cycleways along them, separate from traffic (or sidewalks with legal cycling or at least cycle lanes).

Are you seriously claiming that road with maxspeed 70 is less likely to have cycling infrastructure than residential road in maxspeed 30 zone?

Yes I am.

This one got a cycleway (not tagged in OSM):
https://www.openstreetmap.org/way/113482748

This one has not:
https://www.openstreetmap.org/way/37178440

They are actually directly next to each other and the second one has less elevation - since it's an old railway. Also, there's a cycleway over the whole old railway line, except this part which is a 70 zone. They decided it would be better to route it thru the 30 zone with a cycleway on the street, which has a lot more elevation.

Present statistics that show otherwise and I will drop this requirement.

@westnordost

Well, this doesn't work this time. SC had 2% of all changesets last year, so much details were added on the ground by this app. Many cycle ways might not be tagged, because of this filter. So any statistic based on this data might be biased, because of this bug. But my typical example shows, that you can't assume this.

Want more proof? Wuppertal, Germany got 174 ways with cycleway=* tag and maxspeed=30, just 11 of them are tagged with cycleway=no. The most common tag is "opposite" which shows cycleways in real oneway streets are a common thing, even in a 30 kph area.

In comparison, there are 224 ways with a cycleway=* tag (excepting the cycleway=no ways) and maxspeed=50 in Wuppertal, Germany and zero cycleway with maxspeed tagged as 60,70,80,90 or 100.

There are even two ways with a cycleway and maxspeed=10.

@rindlerblabla
Copy link

In my experience (from Sweden) it's connected to the classification of the highway when it comes to highways with speed lower than 50 (whether there is a cycle lane or not). Residential highways very seldom have cycle lanes, but unclassified highways, tertiary highways and so on can have a cycle lane even if the maxspeed is low.

In this case obviously the highway is classified as a residential highway though. Maybe the classification is wrong, hehe?

@RubenKelevra
Copy link
Contributor Author

RubenKelevra commented Apr 13, 2018

@rindlerblabla tagging looks fine to me. That's a picture of the street. You can see that it's a residential street, maxspeed=30 and that there's a bicycle lane. :)

The small sign below shows that this street is part of the bicycle network called "Panorama Radweg" (previously a railway line)

4114910571

@rindlerblabla
Copy link

Hehe, thanks. Maybe it's out of the scope for this issue, but I think that highway could have been tagged as tertiary as well. Compared to other residential highways I think that highway has a higher importance for routing etc. But i havent seen that place IRL.

Ordinary residential highways seldom have bicycle lanes when the maxspeed is low, in my experience, but when it comes to residential highways with greater importance (=tertiary) the maxspeed is not as important.

At least I have used tertiary highways that way.

@rugk
Copy link
Contributor

rugk commented Apr 14, 2018

So you propose to limit that maxspeed limit to only some types of highways?

@rindlerblabla
Copy link

Yes, or just exclude residential and skip the maxspeed thing but maybe it's just the area where I am mapping this is the situation.

@westnordost
Copy link
Member

Ok so I will exclude highway=residential and only exclude speed limits of 20km/h and below.

@westnordost westnordost self-assigned this Apr 14, 2018
@westnordost westnordost removed the feedback required more info is needed, issue will be likely closed if it is not provided label Apr 14, 2018
@RubenKelevra
Copy link
Contributor Author

@rindlerblabla well, I got your point, but using this tag would fool routers to use it, which makes no sense, using the parallel Kölner Straße is much faster. That's the reason why it's nearly ignored by locals which lowers it's importance. :)

westnordost added a commit that referenced this issue Apr 22, 2018
westnordost added a commit that referenced this issue Apr 22, 2018
@RubenKelevra
Copy link
Contributor Author

RubenKelevra commented Apr 23, 2018

Well, version 5.0 allow streets which are not residential to be included to this quest, but my issue was about residential roads with bicycle infrastructure and 30 kph speed restrictions. So actually the issue was not fixed, but very edgy scenarios for streets which are not residential but have a speed restriction below 50 kph. 😐

@westnordost
Copy link
Member

I know. The app will not ask for residential 30kphs

@RubenKelevra
Copy link
Contributor Author

But this was the point of this ticket, that I found a residential street with 30 kph and cycle way which the app does not show. And I also showed, that this case is pretty common.

@RubenKelevra
Copy link
Contributor Author

RubenKelevra commented Apr 24, 2018

@westnordost so if you did not want to fix this issue with your change, why did you closed this ticket with it?

I totally don't get your change... streets which aren't residential but maxspeed=30 are pretty uncommon. And I showed that on the other hand streets with maxspeed=30 and residential have bike lanes.

So what exactly did I miss here?

@ENT8R
Copy link
Contributor

ENT8R commented Apr 24, 2018

And I showed that on the other hand streets with maxspeed=30 and residential have bike lanes.

IMO the amount of residential roads with maxspeed=30 and a cycle way really depends on the area you live in. In urban areas you will maybe encounter more of these than in rural areas, where (just speaking from my experience) those roads do nearly never exist...

@RubenKelevra
Copy link
Contributor Author

RubenKelevra commented Apr 25, 2018

@ENT8R well, that might be true for your area, but my data here, as shown, draw a different picture.

Furthermore we do ask for residential streets for bicycle lanes, in case the maxspeed was previously not determined the maxspeed quest is followed by the bicycle quest even when the maxspeed is 30 kph. 🤔

https://www.openstreetmap.org/changeset/58398593

@matkoniecz
Copy link
Member

matkoniecz commented Apr 25, 2018

Furthermore we do ask for residential streets for bicycle lanes, in case the maxspeed was previously not determined the maxspeed quest is followed by the bicycle quest even when the maxspeed is 30 kph. thinking

@RubenKelevra See #746

@ENT8R
Copy link
Contributor

ENT8R commented Apr 25, 2018

that might be true for your area, but my data here, as shown, draw a different picture.

But it is also not possible to disable this question for low maxspeeds in rural areas and enable it only in urban areas... So how should we handle this?

@RubenKelevra
Copy link
Contributor Author

@ENT8R what about splitting this into two categories? A standard and an extended bicycle quest, with the extended disabled by default without this restriction. I don't see an issue with adding more definitive answers to the database, stating that this residential road has no bicycle lane. But I understand that those quests might cause many additionally "clicks" for users in the wrong areas.

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

No branches or pull requests

6 participants