Ignore NaN pixels in line clipper

kylecorry31 · Jan 28, 2024 · 520d162 · 520d162
1 parent 64a61f4
commit 520d162
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/app/src/main/java/com/kylecorry/trail_sense/shared/canvas/LineClipper.kt b/app/src/main/java/com/kylecorry/trail_sense/shared/canvas/LineClipper.kt
@@ -51,6 +51,8 @@ class LineClipper {
         for (idx in filteredIndices) {
             val pixel = pixels[idx]
             val vector = vectors[idx]
+            // Remove points that are NaN
+            if (pixel.x.isNaN() || pixel.y.isNaN()) continue
             // Remove lines that cross the entire screen (because they are behind the camera)
             val isLineInvalid = preventLineWrapping && previous != null &&
                     (pixel.x < minX && previous.x > maxX ||

diff --git a/app/src/main/java/com/kylecorry/trail_sense/shared/extensions/AndromedaExtensions.kt b/app/src/main/java/com/kylecorry/trail_sense/shared/extensions/AndromedaExtensions.kt
@@ -68,6 +68,11 @@ fun Alerts.cancelableLoading(
 }
 
 fun PixelCoordinate.isSamePixel(other: PixelCoordinate): Boolean {
+    // If either contains a NaN, return false
+    if (x.isNaN() || y.isNaN() || other.x.isNaN() || other.y.isNaN()) {
+        return false
+    }
+
     return x.roundToInt() == other.x.roundToInt() && y.roundToInt() == other.y.roundToInt()
 }