Skip to content

Add support for vehicle breaks in routing #88

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.

Sign up for GitHub

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

Open
MarkSpit wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add support for vehicle breaks in routing #88

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 15 additions & 2 deletions ext/or-tools/routing.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -202,7 +202,10 @@ void init_routing(Rice::Module& m) {
.define_method("objective_value", &Assignment::ObjectiveValue)
.define_method("value", &Assignment::Value)
.define_method("min", &Assignment::Min)
.define_method("max", &Assignment::Max);
.define_method("max", &Assignment::Max)
.define_method("start_value", &Assignment::StartValue)
.define_method("end_value", &Assignment::EndValue)
.define_method("performed_value", &Assignment::PerformedValue);

// not to be confused with operations_research::sat::IntVar
rb_cIntVar
Expand Down Expand Up @@ -257,7 +260,12 @@ void init_routing(Rice::Module& m) {
.define_method("set_cumul_var_soft_lower_bound", &RoutingDimension::SetCumulVarSoftLowerBound)
.define_method("cumul_var_soft_lower_bound?", &RoutingDimension::HasCumulVarSoftLowerBound)
.define_method("cumul_var_soft_lower_bound", &RoutingDimension::GetCumulVarSoftLowerBound)
.define_method("cumul_var_soft_lower_bound_coefficient", &RoutingDimension::GetCumulVarSoftLowerBoundCoefficient);
.define_method("cumul_var_soft_lower_bound_coefficient", &RoutingDimension::GetCumulVarSoftLowerBoundCoefficient)
.define_method(
"set_break_intervals_of_vehicle",
[](RoutingDimension& self, std::vector<operations_research::IntervalVar*> breaks, int vehicle, std::vector<int64_t> node_visit_transits) {
self.SetBreakIntervalsOfVehicle(breaks, vehicle, node_visit_transits);
});

Rice::define_class_under<RoutingDisjunctionIndex>(m, "RoutingDisjunctionIndex");

Expand Down Expand Up @@ -291,6 +299,11 @@ void init_routing(Rice::Module& m) {
[](operations_research::Solver& self, operations_research::IntVar& start_variable, int64_t duration, const std::string& name) {
return self.MakeFixedDurationIntervalVar(&start_variable, duration, name);
})
.define_method(
"fixed_duration_interval_var",
[](operations_research::Solver& self, int64_t start_min, int64_t start_max, int64_t duration, bool optional, const std::string& name) {
return self.MakeFixedDurationIntervalVar(start_min, start_max, duration, optional, name);
})
.define_method(
"cumulative",
[](operations_research::Solver& self, Array rb_intervals, const std::vector<int64_t>& demands, int64_t capacity, const std::string& name) {
Expand Down
42 changes: 42 additions & 0 deletions test/routing_test.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -920,4 +920,46 @@ def test_apply_locks
routing = ORTools::RoutingModel.new(manager)
assert_nil routing.apply_locks([])
end

def test_breaks
# 5 locations, 2 vehicles, depot 0; transit = travel time + visit duration of the departure node
time_matrix = [[0, 6, 9, 8, 7], [6, 0, 8, 3, 2], [9, 8, 0, 11, 10], [8, 3, 11, 0, 1], [7, 2, 10, 1, 0]]
service_time = [0, 5, 5, 5, 5]
manager = ORTools::RoutingIndexManager.new(5, 2, 0)
routing = ORTools::RoutingModel.new(manager)
transit_callback_index = routing.register_transit_callback(lambda do |from_index, to_index|
from_node = manager.index_to_node(from_index)
time_matrix[from_node][manager.index_to_node(to_index)] + service_time[from_node]
end)
routing.set_arc_cost_evaluator_of_all_vehicles(transit_callback_index)
routing.add_dimension(transit_callback_index, 10, 100, true, "Time") # slack so a break fits in a transit
time_dimension = routing.mutable_dimension("Time")

# visit duration per index; start and end indices map to the depot, which has none
node_visit_transits = Array.new(routing.size) { |index| service_time[manager.index_to_node(index)] }
breaks = 2.times.map do |vehicle|
brk = routing.solver.fixed_duration_interval_var(10, 20, 5, false, "Break for vehicle #{vehicle}")
time_dimension.set_break_intervals_of_vehicle([brk], vehicle, node_visit_transits)
brk
end

solution = routing.solve(first_solution_strategy: :path_cheapest_arc)
refute_nil solution

breaks.each_with_index do |brk, vehicle|
assert_equal 1, solution.performed_value(brk)
start = solution.start_value(brk)
assert_includes 10..20, start
assert_equal start + 5, solution.end_value(brk)

# a break never overlaps a visit
index = routing.start(vehicle)
until routing.end?(index)
node = manager.index_to_node(index)
arrival = solution.min(time_dimension.cumul_var(index))
refute start < arrival + service_time[node] && start + 5 > arrival, "break overlaps visit of node #{node}"
index = solution.value(routing.next_var(index))
end
end
end
end