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

Yield signs broken (again?) #756

Closed
Marquillotuca89 opened this issue Feb 29, 2020 · 11 comments · Fixed by #761
Closed

Yield signs broken (again?) #756

Marquillotuca89 opened this issue Feb 29, 2020 · 11 comments · Fixed by #761
Labels
BUG Defect detected high priority Affects lots of users in-progress The problem is being solved currently PRIORITY SIGNS Feature: Stop / Yield / Priority signs

Comments

@Marquillotuca89
Copy link

Marquillotuca89 commented Feb 29, 2020

Describe the problem

Yield signs don't work as expected. Cars inside a roundabout (for example) give way to cars incoming to the roundabout even if there is a priority sign for cars on that roundabout and a yield sign for the incoming traffic.

Steps to reproduce

  1. Build a roundabout
  2. Place the priority signs
  3. Wait

Log files

Log_Files.zip

Savegame?

https://steamcommunity.com/sharedfiles/filedetails/?id=2010509718

Notes or questions?

The problem came back with 11.1.1 v.

@Marquillotuca89 Marquillotuca89 added BUG Defect detected triage Awaiting issue categorisation labels Feb 29, 2020
@krzychu124
Copy link
Member

Please set simulation accuracy to Very High and test again :)

@Marquillotuca89
Copy link
Author

Please set simulation accuracy to Very High and test again :)

OMG, if that's the problem, I would be really ashamed. Will try it again.
Thanks a lot.

@krzychu124
Copy link
Member

It is possible because reintroduced Simulation Accuracy option skips additional tests for vehicle priority at lower settings. If not helps, that perhaps I messed up something :)

@Marquillotuca89
Copy link
Author

Marquillotuca89 commented Feb 29, 2020

Ok, just tried again and let it run for 7 minutes. Cars are definitely giving way.

Captura4

Captura1
Captura

All those cars on the roundabout are giving way. That dindn't happen with the previous version.

@Marquillotuca89
Copy link
Author

Marquillotuca89 commented Feb 29, 2020

Thanks a lot, I'll see if it works. (even though, those streets are kind of slow)

@originalfoo
Copy link
Member

I think in your case it might make the issue worse to slow the speed down.

@Marquillotuca89
Copy link
Author

Marquillotuca89 commented Feb 29, 2020

Well, It didn't work actually.

I think it's better if I share a video for you to watch it and see if the Yield Signs look broken for you or not.

https://www.youtube.com/watch?v=0fDS24c7AUw&feature=youtu.be

I would be really thankful if you could check if everything is correct.

What is weird to me is that the previous version worked as expected.

@krzychu124
Copy link
Member

Looks like waitTime problem (or inaccurate priority check), I will investigate it :)

@Marquillotuca89
Copy link
Author

Looks like waitTime problem (or inaccurate priority check), I will investigate it :)

Thanks a lot!

I didn't make a change from yesterday (that worked well with previous version) to today (with the new version).

@krzychu124 krzychu124 added high priority Affects lots of users in-progress The problem is being solved currently PRIORITY SIGNS Feature: Stop / Yield / Priority signs and removed triage Awaiting issue categorisation labels Mar 1, 2020
@originalfoo originalfoo added this to the 11.1.1-hotfix1 milestone Mar 1, 2020
@originalfoo
Copy link
Member

This will be fixed in 11.1.1-hotfix1

@Marquillotuca89
Copy link
Author

This will be fixed in 11.1.1-hotfix1

Pretty great to hear that!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BUG Defect detected high priority Affects lots of users in-progress The problem is being solved currently PRIORITY SIGNS Feature: Stop / Yield / Priority signs
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants