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

allow custom error handling for routes via a disallowed rerouter-closed edge #13494

Closed
namdre opened this issue Jun 24, 2023 · 3 comments
Closed
Assignees
Milestone

Comments

@namdre
Copy link
Contributor

namdre commented Jun 24, 2023

currently, loading such routes (with the expectation of seeing jammed traffic) requires the use of option --ignore-route-errors. However, the use of this option potentially hides errors that are different and relevant.

@namdre
Copy link
Contributor Author

namdre commented Feb 22, 2024

closely related to #14203 and #14205

@namdre
Copy link
Contributor Author

namdre commented Feb 23, 2024

I think we should fix this via #8898 with the following expected behavior

  • routing ignores rerouter-induced permission changes by default
  • closingLaneReroute are analyzed for potential edgeClosing effects and added to closed vector of edges (currently only closingReroute are registered with the router)
  • we add a new bit to the (traci) routing mode which defines whether rerouter-changes are visible or not - this solves routing without rerouter blockage #14205
  • we allow setting the routingMode for a trip/vehicle via xml (to distinguis vehicles with/without knowledge of current closing effects). The default is "vehicles know everything" for backward compatibility

namdre added a commit that referenced this issue Apr 12, 2024
@namdre namdre assigned namdre and unassigned behrisch Apr 12, 2024
namdre added a commit that referenced this issue Apr 12, 2024
namdre added a commit that referenced this issue Apr 12, 2024
namdre added a commit that referenced this issue Apr 12, 2024
@namdre namdre closed this as completed Apr 12, 2024
namdre added a commit that referenced this issue Apr 12, 2024
namdre added a commit that referenced this issue Apr 12, 2024
namdre added a commit that referenced this issue Apr 16, 2024
@namdre
Copy link
Contributor Author

namdre commented Apr 23, 2024

routing still fails to ignore transient permissions in some cases due to missing successors

@namdre namdre reopened this Apr 23, 2024
namdre added a commit that referenced this issue Apr 24, 2024
@namdre namdre closed this as completed in d42030e Apr 24, 2024
namdre added a commit that referenced this issue Apr 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants