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

introduce options for adding a junction penalty at non-tls nodes (trac #2640) #2640

Closed
behrisch opened this issue Nov 3, 2016 · 4 comments

Comments

@behrisch
Copy link
Contributor

commented Nov 3, 2016

--meso-major-left-penalty (left turns from a major road)
--meso-minor-penalty (any other unprioritized link)

those delays are very traffic-dependent, yet even a flat time penalty is an improvement over no penalty when running without junction control (or with partial junction control).

Whereas --meso-tls-penalty overrides --meso-junction-control.limited at tls controlled intersections, --meso-minor-penalty should be disabled whenever partial control is applicable (otherwise deadlocks could occur).

Migrated from http://sumo.dlr.de/ticket/2640

{
    "status": "closed", 
    "changetime": "2016-11-04T14:25:45Z", 
    "description": "~~--meso-major-left-penalty (left turns from a major road)~~\n--meso-minor-penalty (any other unprioritized link)\n\nthose delays are very traffic-dependent, yet even a flat time penalty is an improvement over no penalty when running without junction control (or with partial junction control).\n\nWhereas --meso-tls-penalty overrides --meso-junction-control.limited at tls controlled intersections, --meso-minor-penalty should be disabled whenever partial control is applicable (otherwise deadlocks could occur).", 
    "reporter": "namdre", 
    "cc": "", 
    "resolution": "fixed", 
    "_ts": "1478269545639934", 
    "component": "simulation general (meso)", 
    "summary": "introduce options for adding a junction penalty at non-tls nodes", 
    "priority": "major", 
    "keywords": "", 
    "time": "2016-11-03T09:26:14Z", 
    "milestone": "0.29.0", 
    "owner": "namdre", 
    "type": "enhancement"
}
@behrisch

This comment has been minimized.

Copy link
Contributor Author

commented Nov 3, 2016

@namdre committed 943f986 (aka r21899):
added tests refs #2640

@behrisch

This comment has been minimized.

Copy link
Contributor Author

commented Nov 3, 2016

  • @namdre committed 1559012(../commit/155901238f61f9d7a13a28c08f3e5c6aa9b8455d) (aka r21900):
    fix #2640
  • @namdre changed owner from "" to "namdre"
  • @namdre changed resolution from "" to "fixed"
  • @namdre changed status from "new" to "closed"
@behrisch

This comment has been minimized.

Copy link
Contributor Author

commented Nov 3, 2016

@namdre committed cc3f9ff (aka r21901):
refined tests refs #2640

@behrisch

This comment has been minimized.

Copy link
Contributor Author

commented Nov 3, 2016

@namdre committed 92ef69f (aka r21902):
no penalty when limited control is active. refs #2640

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.