Simplify line layer for now

kylecorry31 · Dec 9, 2023 · c5c1bd5 · c5c1bd5
1 parent 1ccfffd
commit c5c1bd5
Showing 1 changed file with 30 additions and 16 deletions.
diff --git a/app/src/main/java/com/kylecorry/trail_sense/tools/augmented_reality/ARLineLayer.kt b/app/src/main/java/com/kylecorry/trail_sense/tools/augmented_reality/ARLineLayer.kt
@@ -54,25 +54,39 @@ class ARLineLayer(
         for (line in lines) {
             path.reset()
             drawer.stroke(color)
-            val pixels = getLinePixels(
-                view,
-                line,
-                resolutionDegrees.toFloat(),
-                maxDistance.toFloat()
-            )
-            // TODO: This should clip the lines to the view, just like
-            for (pixelLine in pixels) {
-                var previous: PixelCoordinate? = null
-                for (pixel in pixelLine) {
-                    if (previous != null) {
-                        path.lineTo(pixel.x, pixel.y)
-                    } else {
-                        path.moveTo(pixel.x, pixel.y)
-                    }
-                    previous = pixel
+
+            // TODO: This should split the lines into smaller chunks (which will allow distance splitting) - keeping it this way for now for the clinometer
+            var previous: PixelCoordinate? = null
+            for (point in line){
+                val pixel = view.toPixel(point.getHorizonCoordinate(view))
+                // TODO: This should split the line if the distance is too great
+                if (previous != null) {
+                    path.lineTo(pixel.x, pixel.y)
+                } else {
+                    path.moveTo(pixel.x, pixel.y)
                 }
+                previous = pixel
             }
 
+            // Curved + increased resolution
+//            val pixels = getLinePixels(
+//                view,
+//                line,
+//                resolutionDegrees.toFloat(),
+//                maxDistance.toFloat()
+//            )
+//            for (pixelLine in pixels) {
+//                var previous: PixelCoordinate? = null
+//                for (pixel in pixelLine) {
+//                    if (previous != null) {
+//                        path.lineTo(pixel.x, pixel.y)
+//                    } else {
+//                        path.moveTo(pixel.x, pixel.y)
+//                    }
+//                    previous = pixel
+//                }
+//            }
+
             drawer.path(path)
         }