TMPE icon indicating copy to clipboard operation
TMPE copied to clipboard

Buggy lane routing on complex intersections

Open Elesbaan70 opened this issue 2 years ago • 7 comments

When there are multiple outgoing segments in the same direction, RoutingManager doesn't know what to do with it.

Note how in these examples, both left-turn lanes are routed to both outgoing lanes.

image image

Elesbaan70 avatar Jun 05 '22 10:06 Elesbaan70

I am loving the routing detector tool. how did we live so far without it?!

kianzarrin avatar Jun 05 '22 10:06 kianzarrin

This overlaps some of the lane routing needs for displaced lanes, so I'll fix it at the same time.

Elesbaan70 avatar Jun 05 '22 10:06 Elesbaan70

I think its better to fix separately. divide the work into smaller pieces to avoid a large PR that is hard to manage.

kianzarrin avatar Jun 05 '22 14:06 kianzarrin

Actually, the lane routing logic for lane groups will inherently resolve this, since each outgoing segment's lanes will be treated as a group.

And yes, it will be a big PR, but mostly limited to one class. The size and scope of the lane routing changes is what made me finally decide to make lane grouping an epic.

Elesbaan70 avatar Jun 05 '22 16:06 Elesbaan70

i hope there is not too many merge conflicts with my PRs

kianzarrin avatar Jun 05 '22 18:06 kianzarrin

there is a all-kian branch that includes all those changes in one branch.

kianzarrin avatar Jun 05 '22 18:06 kianzarrin

If you're talking about your RoutingManager work, I will be writing completely new code, so there shouldn't be many conflicts. See #1593.

Elesbaan70 avatar Jun 05 '22 20:06 Elesbaan70