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

tls.discard-simple #5684

Closed
ambuehll opened this issue May 28, 2019 · 2 comments

Comments

@ambuehll
Copy link

commented May 28, 2019

Setting up OSM networks with tls:
When playing around with tls.discard-simple and tls.discard-loaded = false (and tls.guess =true) I noticed that setting tls.discard-simple = true ends up removing certain tls recorded in osm. The following two screenshots illustrate the dilema:
when setting tls.discard-simple = false -> all tls remain instated, as expected.
tls.discard-simple = false
when setting tls.discard-simple = true-> tls that are not at a 4-way intersection are removed, even if they satisfy the tls.guess.treshold value. It seems as non-standard intersections (such as T-intersections) are classified as geometry-like nodes.
tls.discard-simple = true
I would like to remove tls that are truly located at geometry nodes, but not the ones at non-standard intersections. Any workaround appreciated!

Reproducible with:
files and SUMO v1_2_0+0624-36fd61ea35 at junction:402294227

P.S. any way to make the intersection look nicer in terms of connectivity (the white, unconnected angle space)?

namdre added a commit that referenced this issue May 28, 2019

@namdre namdre closed this in 02dc643 May 28, 2019

@namdre

This comment has been minimized.

Copy link
Contributor

commented May 28, 2019

The white space at junctions is due to #5687
If you don't set option --rectangular-lane-cut it will look somewhat better.

@ambuehll

This comment has been minimized.

Copy link
Author

commented May 28, 2019

thanks for the tip and for the fix! quick as always!
Cheers.

namdre added a commit that referenced this issue Jun 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.