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

netconvert OSM turn:lanes with 1.16 vs dev. version #12978

Closed
ambuehll opened this issue Apr 3, 2023 · 7 comments
Closed

netconvert OSM turn:lanes with 1.16 vs dev. version #12978

ambuehll opened this issue Apr 3, 2023 · 7 comments
Assignees
Labels
a:netconvert bug dev_regression Works previously in this version
Milestone

Comments

@ambuehll
Copy link

ambuehll commented Apr 3, 2023

netconvert does not build the right connections for OSM way 371732298.

turn-lanes and vehicle-lanes from OSM:

	<tag k="turn:lanes" v="through|slight_right|through"/>
	<tag k="vehicle:lanes" v="yes|yes|no"/>

in 1.15 this worked like a charm where private lanes were correctly mapped (with the bus lane being unconnected though).
Now, in 1.16, it generates a "slight_right" for the bus-lane (vehicle:lanes "no") and two "through" connections.

see below (sorry for the colours)

1.15:
image

1.16:
Image 03 04 23 at 12 40

**SUMO-version:**1.15 vs current

files

@m-kro
Copy link
Contributor

m-kro commented Apr 3, 2023

So in essence you need an intersection where neighbour lanes are in conflict with each other (the bus lane crosses the right turn)? If I remember correctly, this is still a problematic case in SUMO...

@ambuehll
Copy link
Author

ambuehll commented Apr 3, 2023

Yes, that's it. It's correctly tagged in OSM, but then netconvert changes the turn:lanes order somehow.
The thing is it didn't do so in sumo 1.15 - back then the car lane connections were correct. (probably because now the bus lane connectivity is better).

@m-kro
Copy link
Contributor

m-kro commented Apr 4, 2023

I could reproduce the issue only in the development version, not in the v1.16 release.

@m-kro m-kro added the dev_regression Works previously in this version label Apr 4, 2023
@m-kro
Copy link
Contributor

m-kro commented Apr 4, 2023

Introduced in #1446 (whereas #12457 introduced lane access import)

@m-kro m-kro changed the title netconvert OSM turn:lanes with 1.15 vs 1.16 netconvert OSM turn:lanes with 1.16 vs dev. version Apr 4, 2023
@namdre namdre added the bug label Apr 11, 2023
@namdre namdre self-assigned this Apr 11, 2023
@namdre namdre added this to the 1.17.0 milestone Apr 17, 2023
namdre added a commit that referenced this issue Apr 17, 2023
@namdre namdre closed this as completed in cac8ec2 Apr 17, 2023
namdre added a commit that referenced this issue Apr 17, 2023
@ambuehll
Copy link
Author

Sorry to come back to this, but it seems to me as if this is not yet fixed in the newest nightly:
v1_16_0+2119-c93da6fc232
image
not sure, where I went wrong.

@namdre
Copy link
Contributor

namdre commented Apr 20, 2023

you are using an outdated nightly (+2119) but need at least v1_16_0-2131-gcac8ec2

@ambuehll
Copy link
Author

ahhh. downloaded "Windows zip with all extras" from the nightly, which is the one lagging behind by two days. thanks and sorry! :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:netconvert bug dev_regression Works previously in this version
Projects
None yet
Development

No branches or pull requests

3 participants