CitiesSkylinesMods / TMPE

Cities: Skylines Traffic Manager: President Edition
https://steamcommunity.com/sharedfiles/filedetails/?id=1637663252
MIT License
569 stars 85 forks source link

hidden redundant lane transitions #1564

Closed kianzarrin closed 2 years ago

kianzarrin commented 2 years ago

image in the image above routing detector displays a redundant track transition (in addition to a useful road transition) from car+tram lane to tram only lane. We don't need that.

originalfoo commented 2 years ago

What do you mean by "redundant"? Should the car+tram lane not be able to reach the tram-only lane?

kianzarrin commented 2 years ago

You should focus on the lanes that are highlighted and also read the text. image

krzychu124 commented 2 years ago

Question is: what vehicle type the selection represents. If 'both' (car + rail) then it is correct, not redundant

kianzarrin commented 2 years ago

The source lane is car+track and the target lane is car only. that is why there is no need for track transition.

kianzarrin commented 2 years ago

I think redundant is not strong enough. its a bug in routing manager. but path find recovers because it checks the source/target lanes' vehicle types.

originalfoo commented 2 years ago

Tracked lanes should not have track connections to non-tracked lanes. Agreed.

IIRC you added some modifier key shortcuts to toggle between car vs. tracked, and I assume you're holding that key on your keybaord when image is taken? If so, we can't see your keyboard in the image. There's literally nothing in that image that conveys "this is filtered to tracked connections".

So in addition to the identified bug in routing manager, maybe also some enhancecments to the routing diagnostic tool to help illustrate what we're looking at in current view. Either square connectors for tracks, or yellow-text highlight relevant content in the hint panel, or change button icon between road and track...?

kianzarrin commented 2 years ago

you added some modifier key shortcuts to toggle between car vs. tracked

@aubergine10 those modifiers are for lane connection. the tool above is Routing detector. It does not have track/road shortcuts.

enhancecments to the routing diagnostic tool to help illustrate what we're looking

to understand what you are looking at you need to read the text in the hint box (I have pointed to with green and red arrow in https://github.com/CitiesSkylinesMods/TMPE/issues/1564#issuecomment-1121389265).

Its hard to talk about abstract topics. go to the tool and play with it!