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

ExtNetInfo class for lane grouping #1598

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

Elesbaan70
Copy link
Contributor

Resolves #1589

This code analyzes NetInfo objects and groups lanes according to directionality and detected medians. It will be the foundation for new routing logic that is more median-aware.

This is all new code, not yet wired up to any existing features. I am submitting this work in stages.

@Elesbaan70 Elesbaan70 self-assigned this Jun 9, 2022
@Elesbaan70 Elesbaan70 added enhancement Improve existing feature feature A new distinct feature LANE ROUTING Feature: Lane arrows / connectors labels Jun 9, 2022
@Elesbaan70 Elesbaan70 removed the enhancement Improve existing feature label Jun 9, 2022
using System.Linq;
using System.Text;

namespace CSModLib.GameObjects {
Copy link
Collaborator

@kianzarrin kianzarrin Jun 11, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is a lot like TrackUtil.cs but smaller.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure but IsRoadLane might be ambiguous because does not tell us if it is only road lane or it includes road lane (like mixed tram+car lane). That is why I chose to use MatchesRoad() and IsRoadOnly() to differentiate.

@DaEgi01 DaEgi01 self-requested a review August 25, 2022 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new distinct feature LANE ROUTING Feature: Lane arrows / connectors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ExtNetInfo for lane grouping
2 participants