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

Unexpectedly Straight -> TSLL and TR -> C #16752

Closed
razor74 opened this issue Mar 20, 2023 · 15 comments
Closed

Unexpectedly Straight -> TSLL and TR -> C #16752

razor74 opened this issue Mar 20, 2023 · 15 comments
Assignees
Milestone

Comments

@razor74
Copy link
Contributor

razor74 commented Mar 20, 2023

Description

When i drive in the direction from screenshots below, a wrong turn lane count( arrows) is show on the top of the screen - 5 lanes , but the real lane count is 3. You can see the exact location where i was and in other screenshot the turn lane schematics from JOSM.

Steps to reproduce

When i drive from Soseaua Vitan Barzesti road to Soseaua Oltenitei road.

Actual result

5 turn lanes

Expected result

only 3 turn lanes

Your Environment

OsmAnd Version: Osmand+ Pro 4.4.3
Android/iOS version: Android 13
Device model: Xiaomi Mi11

incorrect lane count

@Zirochkabila
Copy link
Collaborator

@razor74 What maps of this territory and country do you have downloaded, do you use live updates?

@razor74
Copy link
Contributor Author

razor74 commented Mar 20, 2023

I have Romania map from 1 march 2023 and i use live updates.

@Zirochkabila
Copy link
Collaborator

Zirochkabila commented Mar 24, 2023

OsmAnd 4.4.4, released: 2023-03-22

1

Moved to #16279

2

Moved to #13829


3

53.90069, 27.61206
Map
OSM
OsmAnd

telegram-cloud-document-2-5256129505956733648.mp4

@razor74
Copy link
Contributor Author

razor74 commented Mar 24, 2023

Osmand still shows wrong lane count. This is the real turn lane configuration. Nowhere are 5 lanes per direction. Just before the segment where now it shows correct lane nr. it shows 5 lanes, but there are only 4.
image

@SamRuzhowski
Copy link

SamRuzhowski commented Mar 25, 2023

OsmAnd 4.4.4, released: 2023-03-22

44.386406, 26.139163 Map OSM OsmAnd

Same 53.884649, 27.619833 Map OSM Osmand

53.90069, 27.61206 Map OSM OsmAnd

And Уральская street from 3rd video at the end has turn:lanes:backward=reverse;left|through;left|right. Way 868135783. So lanes need to be like this.
Screenshot_20230325-112446_1_1

@vshcherb
Copy link
Member

vshcherb commented Apr 6, 2023

To do: No 1 (left|through|through|right and instead we see only C|C|C|C|C), No 2 - should be fixed. No 3 - not reproduced.

@razor74
Copy link
Contributor Author

razor74 commented May 8, 2023

Hi. the problem is present on map release from may. The map show now 6 lanes instead of 3.

@vshcherb vshcherb changed the title Incorrect /different turn lane count compared with map database ( only in Osmand) Unexpectedly Straight -> TSLL and TR -> C May 25, 2023
@vshcherb
Copy link
Member

TODO

1

Through (Straight) turn indicator is replaced turn slightly left.


2

44.386406, 26.139163
Map
OSM
OsmAnd

telegram-cloud-document-2-5256129505956733619.mp4

@ivanPyrohivskyi
Copy link
Contributor

ivanPyrohivskyi commented Jun 15, 2023

PR #17421
Test case 53.884649, 27.619833, before/after.
OsmAnd do not show left turn and reverse at same time (in UI), but code instruction is consist reverse (TU) 34233649 Go ahead (+TL,TU|+TL,C|TR):

@ivanPyrohivskyi
Copy link
Contributor

ivanPyrohivskyi commented Jun 15, 2023

1

Test case 44.386406, 26.139163 after PR #17421
Before/after
Lanes configuration of intersection are right, but this way to fuel station also counting as possible lane. I suggest to avoid all service=* roads (but not all highway=service) in lane counting. (Is wrong, see place 35.83824, -78.64158) Will be in separate pull request.

After data fix:

@ivanPyrohivskyi
Copy link
Contributor

ivanPyrohivskyi commented Jun 15, 2023

2

Test case 50.24268, 18.9957
Need to fix data. Way 234238770 (through|through|through;right) is not connected with way 128275340, therefore OsmAnd generate syntetic lanes instruction.

Before/after fix data:

JOSM:
Screenshot 2023-06-15 at 11 56 54

@ivanPyrohivskyi
Copy link
Contributor

ivanPyrohivskyi commented Jun 15, 2023

3

Test case 49.57615, 11.0159 (test "21.Äußere Nürnberger Straße C" with new data is ignore=true in PR)
Need to fix data or create virtual syntetic lanes for way 430007222 like TL|C or TL|TL|C|C (from way 515591061).
Ways with lanes:
Way 30405356 left|left|through|through|right|right that use for instruction.
Way 515591061 left|left|through|through

Present restriction only_straight_on

@vshcherb
Copy link
Member

vshcherb commented Jun 16, 2023

Summary Jun 16 2023

All mentioned cases are fixed except 3

@vshcherb
Copy link
Member

Slightly different fix for a topic starter that we've udpated the map https://www.openstreetmap.org/changeset/137612121 cause service roads could be counted in turn:lanes - https://www.openstreetmap.org/way/693970262

@xmd5a2
Copy link
Contributor

xmd5a2 commented Jun 22, 2023

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

No branches or pull requests

7 participants