trans/graphhopper-jsprit
1
0
Fork 0
mirror of https://github.com/graphhopper/jsprit.git synced 2020-01-24 07:45:05 +01:00
Code Issues Releases Wiki Activity

#354 - fixed - Cost recalculated for solution after jobs added (also a copy is made from the solution)

Browse source
This commit is contained in:
Balázs Vissy 2017-06-12 22:38:35 +02:00
parent eb78625e1f
commit 1af7532939
1 changed files with 3 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

16
jsprit-core/src/main/java/com/graphhopper/jsprit/core/algorithm/VehicleRoutingAlgorithm.java
View file

@ -142,6 +142,8 @@ public class VehicleRoutingAlgorithm {
* @param solution the solution to be added
*/
public void addInitialSolution(VehicleRoutingProblemSolution solution) {
// We will make changes so let's make a copy
solution = VehicleRoutingProblemSolution.copyOf(solution);
verify(solution);
initialSolutions.add(solution);
}
@ -164,19 +166,7 @@ public class VehicleRoutingAlgorithm {
}
solution.getUnassignedJobs().addAll(allJobs);
// Doesn't work
// for (Vehicle v : problem.getVehicles()) {
// boolean found = false;
// for (VehicleRoute vr : solution.getRoutes())
// if (vr.getVehicle().getId().equals(v.getId())) {
// found = true;
// break;
// }
// if (!found) {
// solution.getRoutes().add(VehicleRoute.Builder.newInstance(v).build());
// }
// }
solution.setCost(getObjectiveFunction().getCosts(solution));
// if (nuJobs != problem.getJobs().values().size()) {
// logger.warn("number of jobs in initial solution ({}) is not equal nuJobs in vehicle routing problem ({})" +