Adding another merit type, improving max density objective

merschformann · merschformann · commit 609c08a9de17 · 2025-04-14T00:46:37.000+02:00
diff --git a/SC.ObjectModel/Additionals/ContainerInfo.cs b/SC.ObjectModel/Additionals/ContainerInfo.cs
@@ -29,7 +29,7 @@ public ContainerInfo(COSolution solution, Container container)
         /// <summary>
         /// The container which this info is tracking.
         /// </summary>
-        private Container Container { get; set; }
+        internal Container Container { get; private set; }
 
         /// <summary>
         /// The volume packed inside of the container.
diff --git a/SC.ObjectModel/Additionals/MeritType.cs b/SC.ObjectModel/Additionals/MeritType.cs
@@ -50,5 +50,10 @@ public enum MeritFunctionType
         /// Aims to minimize packing height
         /// </summary>
         H,
+
+        /// <summary>
+        /// Aims to minimize packing height while aiming for fewer containers (Reduce-Height-Reduce-Container)
+        /// </summary>
+        RHRC,
     }
 }
diff --git a/SC.ObjectModel/Additionals/Objective.cs b/SC.ObjectModel/Additionals/Objective.cs
@@ -42,17 +42,14 @@ public double Value
                             return
                                 -Solution.OffloadPieces.Count * _heightBigM * 2
                                 - Solution.ContainerInfos.Count(c => c.NumberOfPieces > 0) * _heightBigM
-                                + Solution.ContainerInfos.Where(c => c.NumberOfPieces > 0).MinOrDefault(c => c.PackingHeight) / _heightBigM;
+                                - Solution.ContainerInfos.Where(c => c.NumberOfPieces > 0).Sum(c => c.Container.Mesh.Height - c.PackingHeight);
                         }
                     default:
                         throw new ArgumentException($"Unknown objective type: {Solution.Configuration.Objective}");
                 }
             }
         }
 
-
-
-
         /// <summary>
         /// The volume packed inside of the containers.
         /// </summary>
diff --git a/SC.ObjectModel/COSolution.cs b/SC.ObjectModel/COSolution.cs
@@ -1177,6 +1177,13 @@ public double ScorePieceAllocation(Container container, VariablePiece piece, int
                         score += position.Z + piece[orientation].BoundingBox.Height;
                     }
                     break;
+                case MeritFunctionType.RHRC:
+                    {
+                        var newHeight = Math.Max(position.Z + piece[orientation].BoundingBox.Height, ContainerInfos[container.VolatileID].PackingHeight);
+                        var newVolume = ContainerInfos[container.VolatileID].VolumeContained + piece.Volume;
+                        score += newVolume / (container.Mesh.Length * container.Mesh.Width * newHeight) + ContainerInfos[container.VolatileID].NumberOfPieces > 0 ? 1 : 0;
+                    }
+                    break;
                 case MeritFunctionType.None:
                 default:
                     break;

Original file line number	Diff line number	Diff line change
`@@ -50,5 +50,10 @@ public enum MeritFunctionType`
`50`	`50`	`/// Aims to minimize packing height`
`51`	`51`	`/// </summary>`
`52`	`52`	`H,`
	`53`	`+`
	`54`	`+ /// <summary>`
	`55`	`+ /// Aims to minimize packing height while aiming for fewer containers (Reduce-Height-Reduce-Container)`
	`56`	`+ /// </summary>`
	`57`	`+ RHRC,`
`53`	`58`	`}`
`54`	`59`	`}`
Original file line number	Diff line number	Diff line change
`@@ -42,17 +42,14 @@ public double Value`
`42`	`42`	`return`
`43`	`43`	`-Solution.OffloadPieces.Count * _heightBigM * 2`
`44`	`44`	`- Solution.ContainerInfos.Count(c => c.NumberOfPieces > 0) * _heightBigM`
`45`		`- + Solution.ContainerInfos.Where(c => c.NumberOfPieces > 0).MinOrDefault(c => c.PackingHeight) / _heightBigM;`
	`45`	`+ - Solution.ContainerInfos.Where(c => c.NumberOfPieces > 0).Sum(c => c.Container.Mesh.Height - c.PackingHeight);`
`46`	`46`	`}`
`47`	`47`	`default:`
`48`	`48`	`throw new ArgumentException($"Unknown objective type: {Solution.Configuration.Objective}");`
`49`	`49`	`}`
`50`	`50`	`}`
`51`	`51`	`}`
`52`	`52`
`53`		`-`
`54`		`-`
`55`		`-`
`56`	`53`	`/// <summary>`
`57`	`54`	`/// The volume packed inside of the containers.`
`58`	`55`	`/// </summary>`