From 610d2dc1a310d918604f758705bab41b8d2af451 Mon Sep 17 00:00:00 2001
From: Henry Lin <henry.ys.lin@gmail.com>
Date: Sun, 17 Apr 2022 10:34:48 +0800
Subject: [PATCH] Use a bigger sample step to calculate slider center of mass

---
 .../Utils/OsuHitObjectGenerationUtils_Reposition.cs    | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/osu.Game.Rulesets.Osu/Utils/OsuHitObjectGenerationUtils_Reposition.cs b/osu.Game.Rulesets.Osu/Utils/OsuHitObjectGenerationUtils_Reposition.cs
index 2abbd61c59..a77d1f8b0f 100644
--- a/osu.Game.Rulesets.Osu/Utils/OsuHitObjectGenerationUtils_Reposition.cs
+++ b/osu.Game.Rulesets.Osu/Utils/OsuHitObjectGenerationUtils_Reposition.cs
@@ -323,13 +323,19 @@ namespace osu.Game.Rulesets.Osu.Utils
         /// <returns>The centre of mass of the slider.</returns>
         private static Vector2 calculateCentreOfMass(Slider slider)
         {
-            if (slider.Distance < 1) return Vector2.Zero;
+            const double sample_step = 50;
+
+            // just sample the start and end positions if the slider is too short
+            if (slider.Distance <= sample_step)
+            {
+                return Vector2.Divide(slider.Path.PositionAt(1), 2);
+            }
 
             int count = 0;
             Vector2 sum = Vector2.Zero;
             double pathDistance = slider.Distance;
 
-            for (double i = 0; i < pathDistance; i++)
+            for (double i = 0; i < pathDistance; i += sample_step)
             {
                 sum += slider.Path.PositionAt(i / pathDistance);
                 count++;