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

considerate bus stop behaviour #1690

Merged
merged 5 commits into from
Nov 23, 2022
Merged

considerate bus stop behaviour #1690

merged 5 commits into from
Nov 23, 2022

Conversation

kianzarrin
Copy link
Collaborator

@kianzarrin kianzarrin commented Nov 17, 2022

fixes #1688. see issue for details
TMPE.zip

@kianzarrin kianzarrin self-assigned this Nov 17, 2022
@kianzarrin kianzarrin added Vehicle Behavior Vehicle behavior related issue enhancement Improve existing feature feature A new distinct feature and removed feature A new distinct feature labels Nov 17, 2022
@krzychu124 krzychu124 added this to the 11.8.0 milestone Nov 17, 2022
Copy link
Member

@krzychu124 krzychu124 left a comment

Choose a reason for hiding this comment

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

I've tested with bus, trolleybus, and trams (obviously shouldn't change anything, but just in case), everything seems to be working nicely. Noticeably better than vanilla. Performance wise most likely in margin of error territory which is nice.

One thing I noticed, but I haven't had time to compare it to scenario without the change is that when you put bus stop at segment joined directly to big intersection node busses going from other segment may overshoot stop position and run into each other instead of forming a waiting queue especially if they are turning from different direction.

ugly example when approaching green bus is not able to stop without hitting blue bus that is already waiting at the bus stop
image

@kianzarrin
Copy link
Collaborator Author

@krzychu124 if you have a test city I'd like to take a look. I'd be surprised if my changes do anything to contribute to this situation since I am only changing the sideways position by a little bit.

@krzychu124
Copy link
Member

@krzychu124 if you have a test city I'd like to take a look. I'd be surprised if my changes do anything to contribute to this situation since I am only changing the sideways position by a little bit.

Yeah, but maybe changes to path are increasing a chance to collision detection issues e.g.: because target positions are generated differently further exposing its poor accuracy at high speed

@kianzarrin
Copy link
Collaborator Author

@krzychu124 do you have saved city? it might be hard to reproduce

@krzychu124
Copy link
Member

@krzychu124 do you have saved city? it might be hard to reproduce

Yeah, I have, but I don't see any difference between this build and current master (on both sometimes they do weird things) so I think we can ignore that for now.

@kianzarrin
Copy link
Collaborator Author

kianzarrin commented Nov 23, 2022

t1 and t2 should be modifiable : #1695

@krzychu124 krzychu124 modified the milestones: 11.8.0, 11.7.3.0 Nov 23, 2022
@kianzarrin kianzarrin merged commit ebf86c6 into master Nov 23, 2022
@kianzarrin kianzarrin deleted the 1688-considerate-buses branch November 23, 2022 17:05
@krzychu124 krzychu124 mentioned this pull request Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improve existing feature Vehicle Behavior Vehicle behavior related issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve bus stop path
3 participants