From 8e7e539407b22528d02c910dc614e654614a7a5d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Stefan=20Schr=C3=B6der?= Date: Thu, 16 Nov 2017 12:45:19 +0100 Subject: [PATCH 1/7] Update Other-Projects.md --- docs/Other-Projects.md | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/docs/Other-Projects.md b/docs/Other-Projects.md index f61ccecb..20e9a196 100644 --- a/docs/Other-Projects.md +++ b/docs/Other-Projects.md @@ -15,6 +15,9 @@ VROOM is an optimization engine written in C++14 that aim at providing good solu #### [Hipster4j](http://www.hipster4j.org/) Hipster is an easy to use yet powerful and flexible type-safe library for heuristic search, written in pure Java. It relies on a flexible model with generic operators to define search problems. So you can also model and solve vehicle routing problems. +#### [OscaR](https://bitbucket.org/oscarlib/oscar/wiki/Home) +OscaR, an Open Source Toolbox for Optimising Logistics and Supply Chain Systems. + ### Territory Design #### [OpenDoorLogistics](http://www.opendoorlogistics.com) @@ -23,4 +26,4 @@ standalone open source application for performing geographic analysis of your cu -If you know another promising open source implementation, report it. \ No newline at end of file +If you know another promising open source implementation, report it. From fb78e1574b75292885f679d2c337dcef30e9b833 Mon Sep 17 00:00:00 2001 From: oblonski Date: Tue, 21 Nov 2017 19:59:26 +0100 Subject: [PATCH 2/7] add human readable reason --- .../jsprit/core/util/UnassignedJobReasonTracker.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/util/UnassignedJobReasonTracker.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/util/UnassignedJobReasonTracker.java index a4577a05..0ab34dec 100644 --- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/util/UnassignedJobReasonTracker.java +++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/util/UnassignedJobReasonTracker.java @@ -29,8 +29,11 @@ import java.util.*; */ public class UnassignedJobReasonTracker implements JobUnassignedListener { + private final static String NO_REASON = "cannot determine a particular reason"; + public static String getMostLikelyFailedConstraintName(Frequency failedConstraintNamesFrequency) { - if (failedConstraintNamesFrequency == null) return "no reason found"; + if (failedConstraintNamesFrequency == null || failedConstraintNamesFrequency.getUniqueCount() == 0) + return NO_REASON; Iterator, Long>> entryIterator = failedConstraintNamesFrequency.entrySetIterator(); long maxCount = 0; String mostLikely = null; @@ -54,6 +57,7 @@ public class UnassignedJobReasonTracker implements JobUnassignedListener { Set failedConstraintNamesToBeIgnored = new HashSet<>(); public UnassignedJobReasonTracker() { + codesToHumanReadableReason.put(-1, NO_REASON); codesToHumanReadableReason.put(1, "cannot serve required skill"); codesToHumanReadableReason.put(2, "cannot be visited within time window"); codesToHumanReadableReason.put(3, "does not fit into any vehicle due to capacity"); @@ -165,8 +169,9 @@ public class UnassignedJobReasonTracker implements JobUnassignedListener { * @return */ public String getMostLikelyReason(String jobId) { - if (!this.failedConstraintNamesFrequencyMapping.containsKey(jobId)) return "no reason found"; + if (!this.failedConstraintNamesFrequencyMapping.containsKey(jobId)) return codesToHumanReadableReason.get(-1); Frequency reasons = this.failedConstraintNamesFrequencyMapping.get(jobId); + String mostLikelyReason = getMostLikelyFailedConstraintName(reasons); int code = toCode(mostLikelyReason); if (code == -1) return mostLikelyReason; From 76aa6938db31d07664628e84b2a03b456de6ed5c Mon Sep 17 00:00:00 2001 From: oblonski Date: Tue, 28 Nov 2017 15:47:33 +0100 Subject: [PATCH 3/7] refine error messages --- .../jsprit/core/problem/vehicle/VehicleImpl.java | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleImpl.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleImpl.java index f77f7b9c..809f2adf 100644 --- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleImpl.java +++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleImpl.java @@ -145,7 +145,7 @@ public class VehicleImpl extends AbstractVehicle { * @throws IllegalArgumentException if type is null */ public Builder setType(VehicleType type) { - if (type == null) throw new IllegalArgumentException("type cannot be null."); + if (type == null) throw new IllegalArgumentException("Vehicle type must not be null."); this.type = type; return this; } @@ -213,7 +213,7 @@ public class VehicleImpl extends AbstractVehicle { */ public Builder setEarliestStart(double earliest_startTime) { if (earliest_startTime < 0) - throw new IllegalArgumentException("earliest start of vehicle " + id + " must not be negative"); + throw new IllegalArgumentException("The earliest start time of vehicle " + id + " must not be negative."); this.earliestStart = earliest_startTime; return this; } @@ -226,7 +226,7 @@ public class VehicleImpl extends AbstractVehicle { */ public Builder setLatestArrival(double latest_arrTime) { if (latest_arrTime < 0) - throw new IllegalArgumentException("latest arrival time of vehicle " + id + " must not be negative"); + throw new IllegalArgumentException("The latest arrival time of vehicle " + id + " must not be negative."); this.latestArrival = latest_arrTime; return this; } @@ -254,17 +254,17 @@ public class VehicleImpl extends AbstractVehicle { */ public VehicleImpl build() { if (latestArrival < earliestStart) - throw new IllegalArgumentException("latest arrival of vehicle " + id + " must not be smaller than its start time"); + throw new IllegalArgumentException("The latest arrival time of vehicle " + id + " must not be smaller than its start time."); if (startLocation != null && endLocation != null) { if (!startLocation.getId().equals(endLocation.getId()) && !returnToDepot) - throw new IllegalArgumentException("this must not be. you specified both endLocationId and open-routes. this is contradictory.
" + - "if you set endLocation, returnToDepot must be true. if returnToDepot is false, endLocationCoord must not be specified."); + throw new IllegalArgumentException("You specified both the end location and that the vehicle " + id + " does not need to return to its end location. This must not be. " + + "Either specify end location and return to depot or leave end location unspecified."); } if (startLocation != null && endLocation == null) { endLocation = startLocation; } if (startLocation == null && endLocation == null) - throw new IllegalArgumentException("vehicle requires startLocation. but neither locationId nor locationCoord nor startLocationId nor startLocationCoord has been set"); + throw new IllegalArgumentException("Every vehicle requires a start location, but vehicle " + id + " does not have one."); skills = skillBuilder.build(); return new VehicleImpl(this); } From b610626b1d2e307b63cc0c2bd12ca79efb59f410 Mon Sep 17 00:00:00 2001 From: oblonski Date: Thu, 30 Nov 2017 09:49:00 +0100 Subject: [PATCH 4/7] refine error messages --- .../core/problem/VehicleRoutingProblem.java | 19 +++++++------ .../jsprit/core/problem/job/Job.java | 1 + .../jsprit/core/problem/job/Service.java | 16 +++++------ .../jsprit/core/problem/job/Shipment.java | 28 +++++++++++-------- .../core/problem/vehicle/VehicleTypeImpl.java | 14 ++++++---- 5 files changed, 43 insertions(+), 35 deletions(-) diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/VehicleRoutingProblem.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/VehicleRoutingProblem.java index 99a1788e..c08e05cc 100644 --- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/VehicleRoutingProblem.java +++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/VehicleRoutingProblem.java @@ -232,9 +232,9 @@ public class VehicleRoutingProblem { */ public Builder addJob(AbstractJob job) { if (tentativeJobs.containsKey(job.getId())) - throw new IllegalArgumentException("vehicle routing problem already contains a service or shipment with id " + job.getId() + ". make sure you use unique ids for all services and shipments"); + throw new IllegalArgumentException("The vehicle routing problem already contains a service or shipment with id " + job.getId() + ". Please make sure you use unique ids for all services and shipments."); if (!(job instanceof Service || job instanceof Shipment)) - throw new IllegalArgumentException("job must be either a service or a shipment"); + throw new IllegalArgumentException("Job must be either a service or a shipment."); job.setIndex(jobIndexCounter); incJobIndexCounter(); tentativeJobs.put(job.getId(), job); @@ -285,10 +285,11 @@ public class VehicleRoutingProblem { for (Vehicle v : uniqueVehicles) { if (v.getBreak() != null) { if (!uniqueBreakIds.add(v.getBreak().getId())) - throw new IllegalArgumentException("problem already contains a vehicle break with id " + v.getBreak().getId() + ". choose unique ids for each vehicle break."); + throw new IllegalArgumentException("The vehicle routing roblem already contains a vehicle break with id " + v.getBreak().getId() + ". Please choose unique ids for each vehicle break."); hasBreaks = true; List breakActivities = jobActivityFactory.createActivities(v.getBreak()); - if(breakActivities.isEmpty()) throw new IllegalArgumentException("at least one activity for break needs to be created by activityFactory"); + if (breakActivities.isEmpty()) + throw new IllegalArgumentException("At least one activity for break needs to be created by activityFactory."); for(AbstractActivity act : breakActivities){ act.setIndex(activityIndexCounter); incActivityIndexCounter(); @@ -351,7 +352,7 @@ public class VehicleRoutingProblem { private void addShipment(Shipment job) { if (jobs.containsKey(job.getId())) { - logger.warn("job " + job + " already in job list. overrides existing job."); + logger.warn("The job " + job + " has already been added to the job list. This overrides the existing job."); } addLocationToTentativeLocations(job); // tentative_coordinates.put(job.getPickupLocation().getId(), job.getPickupLocation().getCoordinate()); @@ -367,7 +368,7 @@ public class VehicleRoutingProblem { * */ public Builder addVehicle(Vehicle vehicle) { if (!(vehicle instanceof AbstractVehicle)) - throw new IllegalArgumentException("vehicle must be an AbstractVehicle"); + throw new IllegalArgumentException("A vehicle must be an AbstractVehicle."); return addVehicle((AbstractVehicle) vehicle); } @@ -379,7 +380,7 @@ public class VehicleRoutingProblem { */ public Builder addVehicle(AbstractVehicle vehicle) { if(addedVehicleIds.contains(vehicle.getId())){ - throw new IllegalArgumentException("problem already contains a vehicle with id " + vehicle.getId() + ". choose unique ids for each vehicle."); + throw new IllegalArgumentException("The vehicle routing problem already contains a vehicle with id " + vehicle.getId() + ". Please choose unique ids for each vehicle."); } else addedVehicleIds.add(vehicle.getId()); if (!uniqueVehicles.contains(vehicle)) { @@ -443,7 +444,7 @@ public class VehicleRoutingProblem { } boolean hasBreaks = addBreaksToActivityMap(); if (hasBreaks && fleetSize.equals(FleetSize.INFINITE)) - throw new UnsupportedOperationException("breaks are not yet supported when dealing with infinite fleet. either set it to finite or omit breaks."); + throw new UnsupportedOperationException("Breaks are not yet supported when dealing with infinite fleet. Either set it to finite or omit breaks."); return new VehicleRoutingProblem(this); } @@ -510,7 +511,7 @@ public class VehicleRoutingProblem { // tentative_coordinates.put(service.getLocation().getId(), service.getLocation().getCoordinate()); addLocationToTentativeLocations(service); if (jobs.containsKey(service.getId())) { - logger.warn("service " + service + " already in job list. overrides existing job."); + logger.warn("The service " + service + " has already been added to job list. This overrides existing job."); } jobs.put(service.getId(), service); return this; diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Job.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Job.java index 55728235..76b65084 100644 --- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Job.java +++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Job.java @@ -30,6 +30,7 @@ import com.graphhopper.jsprit.core.problem.Skills; */ public interface Job extends HasId, HasIndex { + /** * Returns the unique identifier (id) of a job. * diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Service.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Service.java index af4e6b85..1f4f436e 100644 --- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Service.java +++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Service.java @@ -92,7 +92,7 @@ public class Service extends AbstractJob { protected Object userData; protected double maxTimeInVehicle = Double.MAX_VALUE; - + Builder(String id){ this.id = id; timeWindows = new TimeWindowsImpl(); @@ -135,7 +135,7 @@ public class Service extends AbstractJob { */ public Builder setServiceTime(double serviceTime) { if (serviceTime < 0) - throw new IllegalArgumentException("serviceTime must be greater than or equal to zero"); + throw new IllegalArgumentException("The service time of a service must be greater than or equal to zero."); this.serviceTime = serviceTime; return this; } @@ -167,20 +167,20 @@ public class Service extends AbstractJob { * @throws IllegalArgumentException if dimensionValue < 0 */ public Builder addSizeDimension(int dimensionIndex, int dimensionValue) { - if (dimensionValue < 0) throw new IllegalArgumentException("capacity value cannot be negative"); + if (dimensionValue < 0) throw new IllegalArgumentException("The capacity value must not be negative."); capacityBuilder.addDimension(dimensionIndex, dimensionValue); return this; } public Builder setTimeWindow(TimeWindow tw){ - if(tw == null) throw new IllegalArgumentException("time-window arg must not be null"); + if (tw == null) throw new IllegalArgumentException("The time window must not be null."); this.timeWindows = new TimeWindowsImpl(); timeWindows.add(tw); return this; } public Builder addTimeWindow(TimeWindow timeWindow) { - if(timeWindow == null) throw new IllegalArgumentException("time-window arg must not be null"); + if (timeWindow == null) throw new IllegalArgumentException("The time window must not be null."); if(!twAdded){ timeWindows = new TimeWindowsImpl(); twAdded = true; @@ -205,7 +205,7 @@ public class Service extends AbstractJob { * @throws IllegalArgumentException if neither locationId nor coordinate is set. */ public T build() { - if (location == null) throw new IllegalArgumentException("location is missing"); + if (location == null) throw new IllegalArgumentException("The location of service " + id + " is missing."); this.setType("service"); capacity = capacityBuilder.build(); skills = skillBuilder.build(); @@ -246,13 +246,13 @@ public class Service extends AbstractJob { */ public Builder setPriority(int priority) { if (priority < 1 || priority > 10) - throw new IllegalArgumentException("incorrect priority. only priority values from 1 to 10 are allowed where 1 = high and 10 is low"); + throw new IllegalArgumentException("The priority value is not valid. Only 1 (very high) to 10 (very low) are allowed."); this.priority = priority; return this; } public Builder setMaxTimeInVehicle(double maxTimeInVehicle){ - throw new UnsupportedOperationException("maxTimeInVehicle is not yet supported for Pickups and Services (only for Deliveries and Shipments)"); + throw new UnsupportedOperationException("The maximum time in vehicle is not yet supported for Pickups and Services (only for Deliveries and Shipments)."); // if(maxTimeInVehicle < 0) throw new IllegalArgumentException("maxTimeInVehicle should be positive"); // this.maxTimeInVehicle = maxTimeInVehicle; // return this; diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Shipment.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Shipment.java index a3eeaaf6..60085b3a 100644 --- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Shipment.java +++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Shipment.java @@ -148,7 +148,8 @@ public class Shipment extends AbstractJob { * @throws IllegalArgumentException if servicTime < 0.0 */ public Builder setPickupServiceTime(double serviceTime) { - if (serviceTime < 0.0) throw new IllegalArgumentException("serviceTime must not be < 0.0"); + if (serviceTime < 0.0) + throw new IllegalArgumentException("The service time of a shipment must not be < 0.0."); this.pickupServiceTime = serviceTime; return this; } @@ -164,7 +165,7 @@ public class Shipment extends AbstractJob { * @throws IllegalArgumentException if timeWindow is null */ public Builder setPickupTimeWindow(TimeWindow timeWindow) { - if (timeWindow == null) throw new IllegalArgumentException("delivery time-window must not be null"); + if (timeWindow == null) throw new IllegalArgumentException("The delivery time window must not be null."); this.pickupTimeWindows = new TimeWindowsImpl(); this.pickupTimeWindows.add(timeWindow); return this; @@ -193,7 +194,8 @@ public class Shipment extends AbstractJob { * @throws IllegalArgumentException if serviceTime < 0.0 */ public Builder setDeliveryServiceTime(double deliveryServiceTime) { - if (deliveryServiceTime < 0.0) throw new IllegalArgumentException("deliveryServiceTime must not be < 0.0"); + if (deliveryServiceTime < 0.0) + throw new IllegalArgumentException("The service time of a delivery must not be < 0.0."); this.deliveryServiceTime = deliveryServiceTime; return this; } @@ -209,7 +211,7 @@ public class Shipment extends AbstractJob { * @throws IllegalArgumentException if timeWindow is null */ public Builder setDeliveryTimeWindow(TimeWindow timeWindow) { - if (timeWindow == null) throw new IllegalArgumentException("delivery time-window must not be null"); + if (timeWindow == null) throw new IllegalArgumentException("The delivery time window must not be null."); this.deliveryTimeWindows = new TimeWindowsImpl(); this.deliveryTimeWindows.add(timeWindow); return this; @@ -224,7 +226,8 @@ public class Shipment extends AbstractJob { * @throws IllegalArgumentException if dimVal < 0 */ public Builder addSizeDimension(int dimensionIndex, int dimensionValue) { - if (dimensionValue < 0) throw new IllegalArgumentException("capacity value cannot be negative"); + if (dimensionValue < 0) + throw new IllegalArgumentException("The capacity value must not be negative, but is " + dimensionValue + "."); capacityBuilder.addDimension(dimensionIndex, dimensionValue); return this; } @@ -245,8 +248,8 @@ public class Shipment extends AbstractJob { * is set */ public Shipment build() { - if (pickupLocation_ == null) throw new IllegalArgumentException("pickup location is missing"); - if (deliveryLocation_ == null) throw new IllegalArgumentException("delivery location is missing"); + if (pickupLocation_ == null) throw new IllegalArgumentException("The pickup location is missing."); + if (deliveryLocation_ == null) throw new IllegalArgumentException("The delivery location is missing."); capacity = capacityBuilder.build(); skills = skillBuilder.build(); return new Shipment(this); @@ -271,7 +274,7 @@ public class Shipment extends AbstractJob { } public Builder addDeliveryTimeWindow(TimeWindow timeWindow) { - if(timeWindow == null) throw new IllegalArgumentException("time-window arg must not be null"); + if (timeWindow == null) throw new IllegalArgumentException("The time window must not be null."); if(!deliveryTimeWindowAdded){ deliveryTimeWindows = new TimeWindowsImpl(); deliveryTimeWindowAdded = true; @@ -291,7 +294,7 @@ public class Shipment extends AbstractJob { } public Builder addPickupTimeWindow(TimeWindow timeWindow) { - if(timeWindow == null) throw new IllegalArgumentException("time-window arg must not be null"); + if (timeWindow == null) throw new IllegalArgumentException("The time window must not be null."); if(!pickupTimeWindowAdded){ pickupTimeWindows = new TimeWindowsImpl(); pickupTimeWindowAdded = true; @@ -319,7 +322,7 @@ public class Shipment extends AbstractJob { */ public Builder setPriority(int priority) { if (priority < 1 || priority > 10) - throw new IllegalArgumentException("incorrect priority. only 1 (very high) to 10 (very low) are allowed"); + throw new IllegalArgumentException("The priority value is not valid. Only 1 (very high) to 10 (very low) are allowed."); this.priority = priority; return this; } @@ -331,7 +334,8 @@ public class Shipment extends AbstractJob { * @return */ public Builder setMaxTimeInVehicle(double maxTimeInVehicle){ - if(maxTimeInVehicle < 0) throw new IllegalArgumentException("maxTimeInVehicle should be positive"); + if (maxTimeInVehicle < 0) + throw new IllegalArgumentException("The maximum time in vehicle must be positive."); this.maxTimeInVehicle = maxTimeInVehicle; return this; } @@ -436,7 +440,7 @@ public class Shipment extends AbstractJob { return pickupTimeWindows.getTimeWindows(); } - + /** * Returns a string with the shipment's attributes. *

diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleTypeImpl.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleTypeImpl.java index 3e8da61d..112c137c 100644 --- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleTypeImpl.java +++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleTypeImpl.java @@ -151,7 +151,8 @@ public class VehicleTypeImpl implements VehicleType { * if velocity is smaller than zero */ public VehicleTypeImpl.Builder setMaxVelocity(double inMeterPerSeconds) { - if (inMeterPerSeconds < 0.0) throw new IllegalArgumentException("velocity cannot be smaller than zero"); + if (inMeterPerSeconds < 0.0) + throw new IllegalArgumentException("The velocity of a vehicle (type) cannot be smaller than zero."); this.maxVelo = inMeterPerSeconds; return this; } @@ -166,7 +167,7 @@ public class VehicleTypeImpl implements VehicleType { * @throws IllegalArgumentException if fixedCost is smaller than zero */ public VehicleTypeImpl.Builder setFixedCost(double fixedCost) { - if (fixedCost < 0.0) throw new IllegalArgumentException("fixed costs cannot be smaller than zero"); + if (fixedCost < 0.0) throw new IllegalArgumentException("Fixed costs must not be smaller than zero."); this.fixedCost = fixedCost; return this; } @@ -181,7 +182,8 @@ public class VehicleTypeImpl implements VehicleType { * @throws IllegalArgumentException if perDistance is smaller than zero */ public VehicleTypeImpl.Builder setCostPerDistance(double perDistance) { - if (perDistance < 0.0) throw new IllegalArgumentException("cost per distance must not be smaller than zero"); + if (perDistance < 0.0) + throw new IllegalArgumentException("Cost per distance must not be smaller than zero."); this.perDistance = perDistance; return this; } @@ -260,9 +262,9 @@ public class VehicleTypeImpl implements VehicleType { * @throws IllegalArgumentException if capacity dimension is already set */ public Builder addCapacityDimension(int dimIndex, int dimVal) { - if (dimVal < 0) throw new IllegalArgumentException("capacity value cannot be negative"); + if (dimVal < 0) throw new IllegalArgumentException("The capacity value must not be negative."); if (capacityDimensions != null) - throw new IllegalArgumentException("either build your dimension with build your dimensions with " + + throw new IllegalArgumentException("Either build your dimension with build your dimensions with " + "addCapacityDimension(int dimIndex, int dimVal) or set the already built dimensions with .setCapacityDimensions(Capacity capacity)." + "You used both methods."); dimensionAdded = true; @@ -283,7 +285,7 @@ public class VehicleTypeImpl implements VehicleType { */ public Builder setCapacityDimensions(Capacity capacity) { if (dimensionAdded) - throw new IllegalArgumentException("either build your dimension with build your dimensions with " + + throw new IllegalArgumentException("Either build your dimension with build your dimensions with " + "addCapacityDimension(int dimIndex, int dimVal) or set the already built dimensions with .setCapacityDimensions(Capacity capacity)." + "You used both methods."); this.capacityDimensions = capacity; From 7ae5de35924c88f16a22b96ff99a9ad2c621b970 Mon Sep 17 00:00:00 2001 From: Tom Ingold Date: Fri, 8 Dec 2017 14:47:05 -0600 Subject: [PATCH 5/7] updated getting started to fix formatting removed html escaping for markdown --- docs/Getting-Started.md | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/docs/Getting-Started.md b/docs/Getting-Started.md index aea9645f..e666ef52 100644 --- a/docs/Getting-Started.md +++ b/docs/Getting-Started.md @@ -13,11 +13,11 @@ jsprit is a multi-module project and consists of: If you want to use the latest release of jsprit-core, add the following lines to your pom: ``` -<dependency> - <groupId>com.graphhopper</groupId> - <artifactId>jsprit-core</artifactId> - <version>{version}</version> -</dependency> + + com.graphhopper + jsprit-core + {version} + ``` Find the latest versions here: [mvn repository](https://mvnrepository.com/artifact/com.graphhopper/jsprit-core) From f218b9a7387e63c97b3d494cf1461abdedad8d8a Mon Sep 17 00:00:00 2001 From: oblonski Date: Mon, 11 Dec 2017 13:29:44 +0100 Subject: [PATCH 6/7] throw exception if vehicle id is null --- .../graphhopper/jsprit/core/problem/vehicle/VehicleImpl.java | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleImpl.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleImpl.java index 809f2adf..3686d703 100644 --- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleImpl.java +++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleImpl.java @@ -135,6 +135,7 @@ public class VehicleImpl extends AbstractVehicle { private Builder(String id) { super(); this.id = id; + if (id == null) throw new IllegalArgumentException("Vehicle id must not be null."); } /** @@ -196,6 +197,8 @@ public class VehicleImpl extends AbstractVehicle { * @return start location */ public Builder setStartLocation(Location startLocation) { + if (startLocation == null) + throw new IllegalArgumentException("Start location of vehicle " + id + " must not be null."); this.startLocation = startLocation; return this; } @@ -232,6 +235,7 @@ public class VehicleImpl extends AbstractVehicle { } public Builder addSkill(String skill) { + if (skill == null) throw new IllegalArgumentException("Skill of vehicle " + id + " must not be null"); skillBuilder.addSkill(skill); return this; } From cd6477ce146ce989f64020a86b37487c8fb8a69a Mon Sep 17 00:00:00 2001 From: oblonski Date: Thu, 14 Dec 2017 16:10:17 +0100 Subject: [PATCH 7/7] add helper methods for skills --- .../com/graphhopper/jsprit/core/problem/job/Service.java | 9 ++++++--- .../graphhopper/jsprit/core/problem/job/Shipment.java | 9 ++++++--- .../jsprit/core/problem/vehicle/VehicleImpl.java | 8 ++++++++ 3 files changed, 20 insertions(+), 6 deletions(-) diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Service.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Service.java index 1f4f436e..99608c23 100644 --- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Service.java +++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Service.java @@ -222,10 +222,13 @@ public class Service extends AbstractJob { return this; } + public Builder addAllRequiredSkills(Collection skills) { + skillBuilder.addAllSkills(skills); + return this; + } + public Builder addAllRequiredSkills(Skills skills){ - for(String s : skills.values()){ - skillBuilder.addSkill(s); - } + skillBuilder.addAllSkills(skills.values()); return this; } diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Shipment.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Shipment.java index 60085b3a..8d356a02 100644 --- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Shipment.java +++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/job/Shipment.java @@ -261,10 +261,13 @@ public class Shipment extends AbstractJob { return this; } + public Builder addAllRequiredSkills(Collection skills) { + skillBuilder.addAllSkills(skills); + return this; + } + public Builder addAllRequiredSkills(Skills skills) { - for (String s : skills.values()) { - addRequiredSkill(s); - } + addAllRequiredSkills(skills.values()); return this; } diff --git a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleImpl.java b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleImpl.java index 3686d703..0731abf2 100644 --- a/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleImpl.java +++ b/jsprit-core/src/main/java/com/graphhopper/jsprit/core/problem/vehicle/VehicleImpl.java @@ -24,6 +24,8 @@ import com.graphhopper.jsprit.core.problem.job.Break; import org.slf4j.Logger; import org.slf4j.LoggerFactory; +import java.util.Collection; + /** * Implementation of {@link Vehicle}. @@ -234,6 +236,12 @@ public class VehicleImpl extends AbstractVehicle { return this; } + public Builder addAllSkills(Collection skills) { + if (skills == null) throw new IllegalArgumentException("Skills of vehicle " + id + " must not be null"); + skillBuilder.addAllSkills(skills); + return this; + } + public Builder addSkill(String skill) { if (skill == null) throw new IllegalArgumentException("Skill of vehicle " + id + " must not be null"); skillBuilder.addSkill(skill);