From 42456f942a086823982f638f82f5f53a09e11a09 Mon Sep 17 00:00:00 2001
From: jakobst1n <jakob.stendahl@outlook.com>
Date: Wed, 12 Jun 2024 15:37:26 +0200
Subject: Fix downsampling function

---
 src/graph.rs | 34 +++++++++++++++-------------------
 1 file changed, 15 insertions(+), 19 deletions(-)

(limited to 'src/graph.rs')

diff --git a/src/graph.rs b/src/graph.rs
index 8a133d9..446a75d 100644
--- a/src/graph.rs
+++ b/src/graph.rs
@@ -213,9 +213,7 @@ impl GraphBuilder {
 
         if true {
             // && x_values.windows(2).all(|w| w[1] - w[0] == w[0] - w[1]) {
-            if self.y_values[0].len() >= self.draw_width {
-                self.downsample();
-            }
+            self.downsample();
         } else {
             // If the sample size is not consistent, we should interpolate
             todo!("interpolation is not implemented");
@@ -246,23 +244,21 @@ impl GraphBuilder {
     // Make sure to only use one downsampling-algorithm
     fn downsample(&mut self) {
         for g in 0..self.y_values.len() {
+            let mut scale_width = self.draw_width;
             if self.graph_type == GraphType::Braille {
-                let factor = self.y_values[g].len() as f64 / (self.draw_width as f64 * 2.0);
-                let mut new_values = Vec::with_capacity(self.draw_width * 2);
-                for i in 0..self.draw_width * 2 {
-                    let new_value = self.y_values[g][(i as f64 * factor) as usize];
-                    new_values.push(new_value);
-                }
-                self.y_values[g] = new_values;
-            } else {
-                let factor = self.y_values[g].len() as f64 / self.draw_width as f64;
-                let mut new_values = Vec::with_capacity(self.draw_width);
-                for i in 0..self.draw_width {
-                    let new_value = self.y_values[g][(i as f64 * factor) as usize];
-                    new_values.push(new_value);
-                }
-                self.y_values[g] = new_values;
+                scale_width *= 2;
+            }
+            if self.y_values[g].len() < scale_width {
+                continue;
+            }
+
+            let factor = self.y_values[g].len() as f64 / scale_width as f64;
+            let mut new_values = Vec::with_capacity(scale_width);
+            for i in 0..scale_width {
+                let new_value = self.y_values[g][(i as f64 * factor) as usize];
+                new_values.push(new_value);
             }
+            self.y_values[g] = new_values;
         }
     }
 
@@ -391,7 +387,7 @@ impl GraphBuilder {
             );
             self.draw_exact(
                 self.width - 1,
-                self.draw_height - self.y_values[g][self.y_values.len() - 1] as usize,
+                self.draw_height - self.y_values[g][self.y_values[g].len() - 1] as usize,
                 self.color_pixel('┤', |px| GraphPixel::Green(px)),
             );
         }
-- 
cgit v1.2.3