perf: ⚡ In ThermalEventInstance.from_polygon, do not perform polygon …

…simplification if polygon is short enough
IRFM · Aug 10, 2023 · 44ea599 · 44ea599
1 parent eba3b17
commit 44ea599
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/thermal_events/thermal_event_instance.py b/thermal_events/thermal_event_instance.py
@@ -352,11 +352,12 @@ def from_polygon(
         instance = cls(timestamp_ns=timestamp_ns)
 
         # If needed, simplify the polygon
-        nb_points = len(polygon) - 1
-        simplifier = VWSimplifier(np.array(polygon, dtype=float))
-        while len(polygon_to_string(polygon)) > max_polygon_string_length:
-            polygon = simplifier.from_number(nb_points).astype(int)
-            nb_points -= 1
+        if len(polygon_to_string(polygon)) > max_polygon_string_length:
+            nb_points = len(polygon) - 1
+            simplifier = VWSimplifier(np.array(polygon, dtype=float))
+            while len(polygon_to_string(polygon)) > max_polygon_string_length:
+                polygon = simplifier.from_number(nb_points).astype(int)
+                nb_points -= 1
 
         polygon = np.array(np.round(polygon), dtype=np.int32)