refactor state

lcd_manager.py

@@ -1,7 +1,8 @@
 from time import sleep
 
+
 class Adafruit_CharLCD:
-  
+
     # commands
     LCD_CLEARDISPLAY = 0x01
     LCD_RETURNHOME = 0x02
@@ -231,4 +232,4 @@ class Adafruit_CharLCD:
             if char == '\n':
                 self.write4bits(0xC0)  # next line
             else:
-                self.write4bits(ord(char), True)
+                self.write4bits(ord(char), True)

leds.py

@@ -11,6 +11,11 @@ import lcd_manager
 import light_manager
 import logging
 
+level = 0
+level_max = 14
+level_state = 0
+color_state = 0
+
 
 def lightlevel(lcd, level):
     logging.debug("display level")
@@ -18,9 +23,8 @@ def lightlevel(lcd, level):
     lcd.message("Light Level:\n]" + "-"*level + "[")
 
 
-def querylightlevel():
+def query():
-    logging.debug("NYI")
+    level_state = 2
-    return 7
 
 
 def color(lcd):
@@ -29,33 +33,39 @@ def color(lcd):
     logging.debug("NYI")
 
 
+def get_state():
+    if level_state > 0:
+        level_state -= 1
+        return "level"
+    elif color_state > 0:
+        color_state -= 1
+        return "color"
+    else:
+        return "idle"
+
+
+def displayon(lcd):
+    if lcd.displaycontrol & lcd.LCD_DISPLAYON != lcd.displaycontrol:
+        lcd.display()
+
+
 def loop():
     lcd = lcd_manager.Adafruit_CharLCD()
     lights = light_manager.LightStrip()
-    level = 0
-    level_max = 14
-    idle = 0
-    idle_max = 15
-    cur_color = (255, 255, 255)
     while True:
         logging.debug("loop")
-        query_level = querylightlevel()
-        idle = idle + 1
-        logging.debug("idle value: {}".format(idle))
         lights.tick()
-        if query_level != level:
+        query()
-            level = query_level
+        state = get_state()
-            lights.set_light_level(level / level_max)
+
-            idle = 0
+        if state == "level":
-            if lcd.displaycontrol & lcd.LCD_DISPLAYON != lcd.displaycontrol:
+            if lights.get_light_level() != (level / level_max):
-                lcd.display()
+                lights.set_light_level(level / level_max)
             lightlevel(lcd, level)
-        elif idle >= idle_max:
+        elif state == "color":
-            logging.debug("hit idle threshold")
+            color(lcd)
-            idle = idle_max
-            lcd.noDisplay()
         else:
-            lightlevel(lcd, level)
+            lcd.noDisplay()
         sleep(0.1)
 
 

light_manager.py

@@ -3,54 +3,63 @@ import neopixel
 import board
 import logging
 
+
 def defaultPattern(n, t):
     return (((n + t) * 5) % 255, (t * 42) % 255, (t * 50) % 255)
 
+
 class LightStrip:
-  def __init__(self, data_pin = board.D18, string_length = 300, brightness = 1, pixel_order = neopixel.GRB, max_changes = 5):
+    def __init__(self, data_pin=board.D18, string_length=300, brightness=1, pixel_order=neopixel.GRB, max_changes=5):
         self.data_pin = data_pin
 
-        self.np = neopixel.NeoPixel(self.data_pin, string_length, brightness = brightness, auto_write=True, pixel_order = pixel_order)
+        self.np = neopixel.NeoPixel(
+            self.data_pin, string_length, brightness=brightness, auto_write=True, pixel_order=pixel_order)
         self.pattern = defaultPattern
         self.cur_tick = 0
         self.cur_index = 0
         self.max_changes = max_changes
         self.cur_pattern = []
 
-  def set_light_level(self, level):
+    def get_light_level(self):
-    self.np.brightness = level
+        return self.np.brightness
-  def pattern(self, pattern_callback):
-    self.pattern = pattern_callback
 
-  def get_next_pattern_tick(self):
+    def set_light_level(self, level):
-    n = self.np.n
+        self.np.brightness = level
-    t = self.cur_tick
+
-    if not len(self.cur_pattern) >= n - 1:
+    def pattern(self, pattern_callback):
-      li = [(255, 255, 255)] * (n - len(self.cur_pattern))
+        self.pattern = pattern_callback
-      self.cur_pattern.extend(li)
+
-    for i in range(n):
+    def get_next_pattern_tick(self):
-      self.cur_pattern[i] = self.pattern(i, t)
+        n = self.np.n
-    self.cur_tick = t + 1
+        t = self.cur_tick
-  
+        if not len(self.cur_pattern) >= n - 1:
-  def tick(self):
+            li = [(255, 255, 255)] * (n - len(self.cur_pattern))
-    np = self.np
+            self.cur_pattern.extend(li)
-    t = self.cur_tick
+        for i in range(n):
-    n = np.n
+            self.cur_pattern[i] = self.pattern(i, t)
-    
+        self.cur_tick = t + 1
-    if not len(self.cur_pattern) >= n - 1:
+
-      self.get_next_pattern_tick()
+    def tick(self):
-    changes = 0
+        np = self.np
-    ind = self.cur_index
+        t = self.cur_tick
-    for i in range(ind, n):
+        n = np.n
-      col = self.cur_pattern[i]
+
-      self.cur_index = i
+        if not len(self.cur_pattern) >= n - 1:
-      logging.debug("TEST VALUES: np {} col {} same {}".format(np[i], col, tuple(np[i]) != col))
+            self.get_next_pattern_tick()
-      if tuple(np[i]) != col:
+        changes = 0
-        changes = changes + 1
+        ind = self.cur_index
-        np[i] = col
+        for i in range(ind, n):
-        logging.debug("CHANGE COLOR OF PIXEL {} TO {} ON TICK {}".format(i, col, t))
+            col = self.cur_pattern[i]
-        if changes >= self.max_changes:
+            self.cur_index = i
-          break;
+            logging.debug("TEST VALUES: np {} col {} same {}".format(
-    if self.cur_index >= (n-1):
+                np[i], col, tuple(np[i]) != col))
-      self.get_next_pattern_tick()
+            if tuple(np[i]) != col:
-      self.cur_index = 0
+                changes = changes + 1
+                np[i] = col
+                logging.debug(
+                    "CHANGE COLOR OF PIXEL {} TO {} ON TICK {}".format(i, col, t))
+                if changes >= self.max_changes:
+                    break
+        if self.cur_index >= (n-1):
+            self.get_next_pattern_tick()
+            self.cur_index = 0

logging.py

@@ -1,4 +1,4 @@
-debug_statements = True
+debug_statements = False
 
 def debug(msg):
     if debug_statements: