comments

src/main/java/org/matsim/freight/logistics/example/lsp/multipleChains/ProximityStrategyFactory.java

@@ -18,6 +18,9 @@
 import org.matsim.freight.logistics.shipment.LSPShipment;
 
 final class ProximityStrategyFactory {
+  //This is ok so as long as it is **non-public**.
+  //Before making it public, it should be configurable either via config or Injection.
+  //KMT, KN (Jan'24)
 
   private ProximityStrategyFactory() {} // class contains only static methods; do not instantiate
 

src/main/java/org/matsim/freight/logistics/example/lsp/multipleChains/RandomDistributionAllShipmentsStrategyFactory.java

@@ -15,6 +15,9 @@
 import org.matsim.freight.logistics.shipment.LSPShipment;
 
 final class RandomDistributionAllShipmentsStrategyFactory {
+  //This is ok so as long as it is **non-public**.
+  //Before making it public, it should be configurable either via config or Injection.
+  //KMT, KN (Jan'24)
 
   private
   RandomDistributionAllShipmentsStrategyFactory() {} // class contains only static methods; do not

src/main/java/org/matsim/freight/logistics/example/lsp/multipleChains/RandomShiftingStrategyFactory.java

@@ -16,6 +16,9 @@
 import org.matsim.freight.logistics.shipment.LSPShipment;
 
 class RandomShiftingStrategyFactory {
+  //This is ok so as long as it is **non-public**.
+  //Before making it public, it should be configurable either via config or Injection.
+  //KMT, KN (Jan'24)
 
   private
   RandomShiftingStrategyFactory() {} // class contains only static methods; do not instantiate.

src/main/java/org/matsim/freight/logistics/example/lsp/multipleChains/RebalancingStrategyFactory.java

@@ -16,6 +16,9 @@
 import org.matsim.freight.logistics.shipment.LSPShipment;
 
 class RebalancingStrategyFactory {
+  //This is ok so as long as it is **non-public**.
+  //Before making it public, it should be configurable either via config or Injection.
+  //KMT, KN (Jan'24)
 
   private RebalancingStrategyFactory() {} // class contains only static methods; do not instantiate
 

src/main/java/org/matsim/freight/logistics/example/lsp/multipleChains/RoundRobinDistributionAllShipmentsStrategyFactory.java

@@ -35,6 +35,9 @@
 import org.matsim.freight.logistics.shipment.LSPShipment;
 
 /*package-private*/ class RoundRobinDistributionAllShipmentsStrategyFactory {
+  //This is ok so as long as it is **non-public**.
+  //Before making it public, it should be configurable either via config or Injection.
+  //KMT, KN (Jan'24)
 
   private
   RoundRobinDistributionAllShipmentsStrategyFactory() {} // class contains only static methods; do