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.

Sign up for GitHub

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

Jump to bottom

fixed LC raycasting for overlapping lanes #1706

Merged
merged 1 commit into from
Dec 5, 2022
Merged

fixed LC raycasting for overlapping lanes #1706

merged 1 commit into from
Dec 5, 2022

Conversation

kianzarrin
Copy link
Collaborator

@kianzarrin kianzarrin commented Dec 5, 2022 •
edited
Loading

fixes: https://discord.com/channels/545065285862948894/1049088238486294568
not possible to connect overlapping bus+tram lanes.

solution: don't try to intersect mouse ray for the wrong lanes.

TMPE.zip

TEST:

  • place a intersection with vanilla bus+tram road and try to connect overlapping car/tram lanes individually.

@kianzarrin kianzarrin self-assigned this Dec 5, 2022
kvakvs
kvakvs approved these changes Dec 5, 2022
View reviewed changes
Copy link
Collaborator

@kvakvs kvakvs left a comment

Choose a reason for hiding this comment

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

Fine

@kianzarrin kianzarrin added this to the 11.7.3.0 milestone Dec 5, 2022
@kianzarrin kianzarrin merged commit 965e100 into master Dec 5, 2022
@kianzarrin kianzarrin deleted the LC-raycast branch December 5, 2022 23:20
@krzychu124 krzychu124 mentioned this pull request Dec 12, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@krzychu124 krzychu124 krzychu124 approved these changes

@kvakvs kvakvs kvakvs approved these changes

@chameleon-tbn chameleon-tbn Awaiting requested review from chameleon-tbn

Assignees

@kianzarrin kianzarrin

Labels
None yet
Projects
None yet
Milestone
11.7.3.0
Development

Successfully merging this pull request may close these issues.
3 participants
@kianzarrin @kvakvs @krzychu124