update the progress method to only fill pixels that need to be changed. change simpletest to use for loop counting ints to avoid floating point math issue

Author: FoamyGuy

adafruit_progressbar.py

@@ -117,12 +117,18 @@ def progress(self, value):
         ), "Progress value must be a floating point value."
         if self._progress_val > value:
             # uncolorize range from width*value+margin to width-margin
-            for _w in range(int(value * self._width + 2), self._width - 2):
+            # from right to left
+            _prev_pixel = max(2, int(self._width * self._progress_val - 2))
+            _new_pixel = max(int(self._width * value - 2), 2)
+            for _w in range(_prev_pixel, _new_pixel - 1, -1):
+                print("w {}".format(_w))
                 for _h in range(2, self._height - 2):
                     self._bitmap[_w, _h] = 0
         else:
-            # fully fill progress bar color
-            for _w in range(2, self._width * value - 2):
+            # fill from the previous x pixel to the new x pixel
+            _prev_pixel = max(2, int(self._width * self._progress_val - 3))
+            _new_pixel = min(int(self._width * value - 2), int(self._width * 1.0 - 3))
+            for _w in range(_prev_pixel, _new_pixel + 1):
                 for _h in range(2, self._height - 2):
                     self._bitmap[_w, _h] = 2
         self._progress_val = value

examples/progressbar_simpletest.py

@@ -27,10 +27,10 @@
 
 current_progress = 0.0
 while True:
-    while current_progress <= 1.0:
-        print("Progress: {}%".format(current_progress * 100))
-        progress_bar.progress = current_progress
-        current_progress += 0.05
-        if current_progress >= 1.0:
-            current_progress = 0.0
-        time.sleep(0.01)
+    for current_progress in range(0, 21):
+        print("Progress: {}%".format(current_progress * 0.05))
+        progress_bar.progress = current_progress * 0.05
+        time.sleep(0.02)
+    time.sleep(0.3)
+    progress_bar.progress = 0.0
+    time.sleep(0.3)