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

initial pseudo route should not be written by vehroute output #5639

Closed
yunpangfloetteroed opened this issue May 15, 2019 · 6 comments

Comments

@yunpangfloetteroed
Copy link
Contributor

commented May 15, 2019

SUMO does not select the right route in the given route set of the route file when using the meso mode.

@yunpangfloetteroed

This comment has been minimized.

Copy link
Contributor Author

commented May 15, 2019

the files are under Bugreports\5639_route_not_correctly_selected_with_meso

@behrisch

This comment has been minimized.

Copy link
Contributor

commented May 22, 2019

@yunpangfloetteroed I cannot reproduce it here. Can you please add the sumocfg.

@yunpangfloetteroed

This comment has been minimized.

Copy link
Contributor Author

commented May 23, 2019

I added the required files in the directory "20190523" under Bugreports\5639_route_not_correctly_selected_with_meso.

@behrisch

This comment has been minimized.

Copy link
Contributor

commented May 23, 2019

The problem is not sumo but the duarouter. It considers every route in the input and even a route with 0 probability in the input can be chosen in the output if the cost are low enough (which they are for such a short route). Unfortunately it does not check whether the route is valid at all. Maybe adding --repair already fixes that, I'll check. furtheremore we should modify vehroute output such that it does not include the invalid first "route" consisting of two edges only.

@behrisch behrisch added a:DUAROUTER a:SUMO and removed c:meso labels May 23, 2019

@yunpangfloetteroed

This comment has been minimized.

Copy link
Contributor Author

commented May 23, 2019

I think/suggest that sumo should not write out the set of the start and end links as a route under the routeDistribution when running one-shot. It is logical that all routes in the routeDistribution should be vaild as prerequisite. Under this condition, what the current duarouter does ist right.

@behrisch behrisch removed the a:DUAROUTER label May 23, 2019

@behrisch behrisch changed the title route not correctly selected with meso initial pseudo route should not be written by vehroute output May 23, 2019

behrisch added a commit that referenced this issue May 24, 2019
behrisch added a commit that referenced this issue May 24, 2019
namdre added a commit that referenced this issue May 24, 2019
lcodeca pushed a commit to lcodeca/sumo that referenced this issue May 24, 2019
namdre added a commit that referenced this issue May 25, 2019
@behrisch

This comment has been minimized.

Copy link
Contributor

commented May 27, 2019

We should still add an option to get the old behavior back.

behrisch added a commit that referenced this issue Jun 4, 2019

@behrisch behrisch closed this in d19bd0a Jun 4, 2019

namdre added a commit that referenced this issue Jun 5, 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.