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

More advance roundabout quick-setup. #578

Closed
4 of 5 tasks
kianzarrin opened this issue Nov 29, 2019 · 11 comments · Fixed by #863
Closed
4 of 5 tasks

More advance roundabout quick-setup. #578

kianzarrin opened this issue Nov 29, 2019 · 11 comments · Fixed by #863
Labels
feature A new distinct feature
Milestone

Comments

@kianzarrin
Copy link
Collaborator

kianzarrin commented Nov 29, 2019

In issue #539 I implemented the first phase of roundabout quick-setup This issue is the next phase to handle stations like this:
Cities_HdgHjyccJG
Cities_PtXixifMAF
rabout image0

Tasks:

  • option to setup enter blocked junction for roads joining the roundabout.
  • More clever path detection: Instead of following a preferred direction (forward) follow all possible paths in all directions. if the path is not a roundabout then discard it. among the paths that are roundabout I choose the one that has the list amount of left/right turns within the roundabout. if the number of turns are equal then choose the shortest paths With the introduction of road selection panel, this is no longer important.
  • short term undo short-term undo for bulk edit tools. #860
  • ban parking on roundabout
  • Speed Limit Automatically lower speed limits on roundabouts [Suggestion] #793

related : #541 #542

@kianzarrin kianzarrin added feature A new distinct feature triage Awaiting issue categorisation labels Nov 29, 2019
@kianzarrin
Copy link
Collaborator Author

I want to add option to setup entering blocked junctions for roads joining the roundabout. but our options are already a mess (#853) (sorry that's on me!) and I am worried I might make them worse.

So here is what I think:

  • in a highway roundabout we want to allow cars to enter blocked junctions
  • in an urban roundabout we want cars to yield.

This is what the game does by default. so in theory I don't need to add an option for this. But in practice, when Biffa is fixing cities he has to turn off enter blocked junctions manually because

  • sometimes highway roads join urban roundabouts.
  • sometimes user has modified enter blocked junction manually before hand.

So I think I need to add option like this:

O modify enter blocked junctions for in-coming cars
   O allow(on)/ ban(off)

@originalfoo
Copy link
Member

There's also case where users build their urban roundabouts out of highway roads so they can watch cars driving round them at warp speed.

I suspect road size in terms of lane counts will be a factor, but haven't thought that through yet.

@kianzarrin
Copy link
Collaborator Author

Default settings for entering blocked junctions:

  • joining highway : yes
  • joining and leaving highway at the same node: No
  • urban: No
    Screenshot (859)
    Screenshot (864)
    Screenshot (860)

@originalfoo
Copy link
Member

My focus was drawn to the pedestrian crossing icons - on highway roads... Is that a bug?

@kianzarrin
Copy link
Collaborator Author

I suspect road size in terms of lane counts will be a factor, but haven't thought that through yet.

entering blocked junction is good if you are joining the outer lane but not good when you are joining the inner lane (because you can block cars trying to exit the roundabout. so for one lane roundabout is good but for 2+ lane roundabout is bad.

hmmm ... maybe we can have an option in policies to enter blocked junction only for the first lane when pedestrian crossing does not exists.

@originalfoo
Copy link
Member

Regardading lane-wise enter-blocked junction, see also: #565

Actually, for enter blocked junction in general, I keep thinking it maybe needs a complete rethink: #424

@kianzarrin
Copy link
Collaborator Author

My focus was drawn to the pedestrian crossing icons - on highway roads... Is that a bug?

I did brought this up int he pull request #845 (comment).

Fixing it would be as easy as setting IsPedestrianCrossingAllowedConfigurable() to false

if the road does has 2+ pedestrian lanes:
    IsPedestrianCrossingAllowedConfigurable = true
    GetDefaultPedestrianCrossingAllowed = true
else if the road to the right has pedestrian lanes AND the road to the left has pedestrian lanes
    // maybe this logic should be more sophisticated to check that the pedestrian lanes is on innder/outter side of the road
    IsPedestrianCrossingAllowedConfigurable = true
    GetDefaultPedestrianCrossingAllowed = true
else
    IsPedestrianCrossingAllowedConfigurable =  false
    GetDefaultPedestrianCrossingAllowed =  false

The GUI automatically does not show pedestrian crossing overlays if they are disabled. Actually in my mod I have patched TMPE to hide pedestrian crossing overlays when appropriate.

@originalfoo
Copy link
Member

I split the ped crossing stuff to new ticket ^^

@kianzarrin
Copy link
Collaborator Author

The ticket started with focus on better roundabout detection but now I have abandoned that because road selection panel does the deed.

better roundabout detection should move to another ticket:

  • detection and setup of roads that split when joining roundabout.
    • the code for split road setup already exists.
    • my Pedestrian bridge builder already is capable of detecting split roads.
  • detection of magic roundabout
    • at every intersection try to see if we can find a mini roundabout. then try to continue on the main roundabout without going through the mini roundabouts.
  • graph
    • prefer straight roundabouts.
    • prefer short roundabouts
    • prefer roundabouts without sharp turns

@originalfoo
Copy link
Member

Ok, create new ticket.

Should this #578 be closed (treat it as completed by the road selection panel)?

@kianzarrin
Copy link
Collaborator Author

kianzarrin commented Apr 23, 2020

Its working!

kianzarrin added a commit that referenced this issue May 19, 2020
Added new features to roundabout quick setup (fixes #578). They can be setup in the settings.
- ban parking on roundabout/branches.
- lower speed limit on small roundabouts (fixes #793
- allow/ban entering blocked roundabout
- Undo feature (fixes #860)
- improved priority signs cycle : #860 (comment)
@originalfoo originalfoo added this to the 11.6.0 milestone May 19, 2020
@originalfoo originalfoo removed the triage Awaiting issue categorisation label Apr 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature A new distinct feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants