-
Notifications
You must be signed in to change notification settings - Fork 344
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
max_travel_time parameter has no effect on optimization #884
Comments
Thanks for reporting, we indeed provide a solution that does not account for this constraint. Moreover when solving in debug mode, the matching assert is triggered all right upon route formatting:
The problem is "simply" that this instance is wrongly considered a TSP, so we run the ad-hoc TSP solving code on it, and the Simple work-around before this gets fixed: adding any extra constraint (capacity, TW etc.), even a non-binding one, will send this instance to the right solving path. |
You are right, it works when adding TW |
I did this and still not working {
] |
@elkashefehab it is not possible to reproduce your solution without access to your routing stack, can you share a standalone problem instance? |
@jcoupey , Hi there I also have an issue in using max_travel_time, for example in my problem I was expecting that the engine will exclude the only job I have as the max_travel_time equal "1"
Also I tried defining TW in absolute values, but still the max_travel_time is not taken into consideration
|
@zahert are you using a version of VROOM that supports the |
@jcoupey I'm using v1.12.0 I will need to upgrade, thank you 👍 |
Running on the initial instance above now results in all jobs being unassigned as expected:
|
Hello!
I am using the following input data for the optimization with vroom v1.13.0
{ "vehicles":[ { "id":0, "start_index":0, "end_index":3, "max_travel_time":100, "profil":"car", "costs":{ "fixed":0, "per_hour":600000 } } ], "jobs":[ { "id":1414, "location_index":1 }, { "id":1515, "location_index":2 } ], "matrices":{ "car":{ "durations":[ [ 0, 21040, 1970, 12990 ], [ 21030, 0, 22550, 31520 ], [ 1970, 22560, 0, 11020 ], [ 12990, 31530, 11020, 0 ] ] } } }
However, I noticed that the
max_travel_time
constraint for the vehicle is not being taken into account in the resultThe text was updated successfully, but these errors were encountered: