bike-lights/bike/src/main.rs

@@ -33,31 +33,6 @@ const LIGHT_SCALE: I16F16 = I16F16::lit("256.0");
 const DASHBOARD_BRIGHTESS: u8 = 1;
 const BODY_BRIGHTNESS: u8 = 8;
 
-struct DebouncedButton<P: PinId> {
-    debounce: Instant,
-    pin: Pin<P, FunctionSio<SioInput>, PullUp>,
-}
-
-impl<P: PinId> DebouncedButton<P> {
-    fn new(pin: Pin<P, FunctionSio<SioInput>, PullUp>) -> Self {
-        Self {
-            debounce: Instant((0 as u32).into()),
-            pin,
-        }
-    }
-
-    fn is_low(&self, time: Instant) -> bool {
-        if time <= self.debounce {
-            return false;
-        }
-        self.pin.is_low().unwrap_or(false)
-    }
-
-    fn set_debounce(&mut self, time: Instant) {
-        self.debounce = time + Instant((250 as u32).into());
-    }
-}
-
 struct BikeUI<
     D: SpiDevice,
     P: ValidSpiPinout<D>,
@@ -67,10 +42,10 @@ struct BikeUI<
     NextId: PinId,
 > {
     spi: RefCell<Spi<Enabled, D, P, 8>>,
-    left_blinker_button: DebouncedButton<LeftId>,
+    left_blinker_button: Pin<LeftId, FunctionSio<SioInput>, PullUp>,
-    right_blinker_button: DebouncedButton<RightId>,
+    right_blinker_button: Pin<RightId, FunctionSio<SioInput>, PullUp>,
-    previous_animation_button: DebouncedButton<PreviousId>,
+    previous_animation_button: Pin<PreviousId, FunctionSio<SioInput>, PullUp>,
-    next_animation_button: DebouncedButton<NextId>,
+    next_animation_button: Pin<NextId, FunctionSio<SioInput>, PullUp>,
 }
 
 impl<
@@ -91,10 +66,10 @@ impl<
     ) -> Self {
         Self {
             spi: RefCell::new(spi),
-            left_blinker_button: DebouncedButton::new(left_blinker_button),
+            left_blinker_button,
-            right_blinker_button: DebouncedButton::new(right_blinker_button),
+            right_blinker_button,
-            previous_animation_button: DebouncedButton::new(previous_animation_button),
+            previous_animation_button,
-            next_animation_button: DebouncedButton::new(next_animation_button),
+            next_animation_button,
         }
     }
 }
@@ -108,18 +83,14 @@ impl<
         NextId: PinId,
     > UI for BikeUI<D, P, LeftId, RightId, PreviousId, NextId>
 {
-    fn check_event(&mut self, current_time: Instant) -> Option<Event> {
+    fn check_event(&self) -> Option<Event> {
-        if self.left_blinker_button.is_low(current_time) {
+        if self.left_blinker_button.is_low().unwrap_or(false) {
-            self.left_blinker_button.set_debounce(current_time);
             Some(Event::LeftBlinker)
-        } else if self.right_blinker_button.is_low(current_time) {
+        } else if self.right_blinker_button.is_low().unwrap_or(false) {
-            self.right_blinker_button.set_debounce(current_time);
             Some(Event::RightBlinker)
-        } else if self.previous_animation_button.is_low(current_time) {
+        } else if self.previous_animation_button.is_low().unwrap_or(false) {
-            self.previous_animation_button.set_debounce(current_time);
             Some(Event::PreviousPattern)
-        } else if self.next_animation_button.is_low(current_time) {
+        } else if self.next_animation_button.is_low().unwrap_or(false) {
-            self.next_animation_button.set_debounce(current_time);
             Some(Event::NextPattern)
         } else {
             None
@@ -140,9 +111,9 @@ impl<
         }
         for (idx, rgb) in body_lights.iter().enumerate() {
             lights[(idx + 4) * 4 + 0] = 0xe0 + BODY_BRIGHTNESS;
-            lights[(idx + 4) * 4 + 1] = (I16F16::from(rgb.b) * LIGHT_SCALE).saturating_as();
+            lights[(idx + 4) * 4 + 1] = (I16F16::from(rgb.r) * LIGHT_SCALE).saturating_as();
-            lights[(idx + 4) * 4 + 2] = (I16F16::from(rgb.g) * LIGHT_SCALE).saturating_as();
+            lights[(idx + 4) * 4 + 2] = (I16F16::from(rgb.b) * LIGHT_SCALE).saturating_as();
-            lights[(idx + 4) * 4 + 3] = (I16F16::from(rgb.r) * LIGHT_SCALE).saturating_as();
+            lights[(idx + 4) * 4 + 3] = (I16F16::from(rgb.g) * LIGHT_SCALE).saturating_as();
         }
         let mut spi = self.spi.borrow_mut();
         spi.write(lights.as_slice());

bike-lights/core/src/lib.rs

@@ -33,7 +33,7 @@ fn linear_ease(value: I8F8, frames: U16F0, slope: I8F8) -> I8F8 {
     value_i16f16.saturating_as()
 }
 
-#[derive(Clone, Copy, PartialOrd, Ord, PartialEq, Eq)]
+#[derive(Clone, Copy)]
 pub struct Instant(pub U128F0);
 
 impl Default for Instant {
@@ -61,7 +61,7 @@ impl Sub for Instant {
 pub const FPS: u8 = 30;
 
 pub trait UI {
-    fn check_event(&mut self, current_time: Instant) -> Option<Event>;
+    fn check_event(&self) -> Option<Event>;
     fn update_lights(&self, dashboard_lights: DashboardPattern, body_lights: BodyPattern);
 }
 
@@ -69,15 +69,13 @@ pub trait Animation {
     fn tick(&mut self, time: Instant) -> (DashboardPattern, BodyPattern);
 }
 
-/*
 pub struct DefaultAnimation {}
 
 impl Animation for DefaultAnimation {
     fn tick(&mut self, _: Instant) -> (DashboardPattern, BodyPattern) {
-        (WATER_DASHBOARD, WATER_BODY)
+        (PRIDE_DASHBOARD, PRIDE_BODY)
     }
 }
-*/
 
 pub struct Fade {
     starting_dashboard: DashboardPattern,
@@ -195,10 +193,12 @@ impl Blinker {
     fn new(
         starting_dashboard: DashboardPattern,
         starting_body: BodyPattern,
+        home_dashboard: DashboardPattern,
+        home_body: BodyPattern,
         direction: BlinkerDirection,
         time: Instant,
     ) -> Self {
-        let mut ending_dashboard = OFF_DASHBOARD.clone();
+        let mut ending_dashboard = home_dashboard.clone();
 
         match direction {
             BlinkerDirection::Left => {
@@ -213,7 +213,7 @@ impl Blinker {
             }
         }
 
-        let mut ending_body = OFF_BODY.clone();
+        let mut ending_body = home_body.clone();
         match direction {
             BlinkerDirection::Left => {
                 for i in 0..30 {
@@ -241,8 +241,8 @@ impl Blinker {
                 time,
             ),
             fade_in: Fade::new(
-                OFF_DASHBOARD.clone(),
+                home_dashboard.clone(),
-                OFF_BODY.clone(),
+                home_body.clone(),
                 ending_dashboard.clone(),
                 ending_body.clone(),
                 BLINKER_FRAMES,
@@ -251,8 +251,8 @@ impl Blinker {
             fade_out: Fade::new(
                 ending_dashboard.clone(),
                 ending_body.clone(),
-                OFF_DASHBOARD.clone(),
+                home_dashboard.clone(),
-                OFF_BODY.clone(),
+                home_body.clone(),
                 BLINKER_FRAMES,
                 time,
             ),
@@ -302,9 +302,8 @@ pub enum Event {
     RightBlinker,
 }
 
-#[derive(Clone, Copy, Debug, PartialEq)]
+#[derive(Clone, Copy, PartialEq)]
 pub enum Pattern {
-    Water,
     GayPride,
     TransPride,
 }
@@ -312,23 +311,20 @@ pub enum Pattern {
 impl Pattern {
     fn previous(&self) -> Pattern {
         match self {
-            Pattern::Water => Pattern::TransPride,
+            Pattern::GayPride => Pattern::TransPride,
-            Pattern::GayPride => Pattern::Water,
             Pattern::TransPride => Pattern::GayPride,
         }
     }
 
     fn next(&self) -> Pattern {
         match self {
-            Pattern::Water => Pattern::GayPride,
             Pattern::GayPride => Pattern::TransPride,
-            Pattern::TransPride => Pattern::Water,
+            Pattern::TransPride => Pattern::GayPride,
         }
     }
 
     fn dashboard(&self) -> DashboardPattern {
         match self {
-            Pattern::Water => WATER_DASHBOARD,
             Pattern::GayPride => PRIDE_DASHBOARD,
             Pattern::TransPride => TRANS_PRIDE_DASHBOARD,
         }
@@ -336,14 +332,13 @@ impl Pattern {
 
     fn body(&self) -> BodyPattern {
         match self {
-            Pattern::Water => OFF_BODY,
             Pattern::GayPride => PRIDE_BODY,
             Pattern::TransPride => TRANS_PRIDE_BODY,
         }
     }
 }
 
-#[derive(Clone, Debug, PartialEq)]
+#[derive(Clone, PartialEq)]
 pub enum State {
     Pattern(Pattern),
     Brake,
@@ -364,19 +359,11 @@ pub struct App {
 
 impl App {
     pub fn new(ui: Box<dyn UI>) -> Self {
-        let pattern = Pattern::Water;
         Self {
             ui,
-            state: State::Pattern(pattern),
+            state: State::Pattern(Pattern::GayPride),
-            home_pattern: pattern,
+            home_pattern: Pattern::GayPride,
-            current_animation: Box::new(Fade::new(
+            current_animation: Box::new(DefaultAnimation {}),
-                OFF_DASHBOARD,
-                OFF_BODY,
-                pattern.dashboard(),
-                pattern.body(),
-                DEFAULT_FRAMES,
-                Instant((0 as u32).into()),
-            )),
             dashboard_lights: OFF_DASHBOARD,
             lights: OFF_BODY,
         }
@@ -408,6 +395,8 @@ impl App {
                 self.current_animation = Box::new(Blinker::new(
                     self.dashboard_lights.clone(),
                     self.lights.clone(),
+                    self.home_pattern.dashboard(),
+                    self.home_pattern.body(),
                     BlinkerDirection::Left,
                     time,
                 ));
@@ -416,6 +405,8 @@ impl App {
                 self.current_animation = Box::new(Blinker::new(
                     self.dashboard_lights.clone(),
                     self.lights.clone(),
+                    self.home_pattern.dashboard(),
+                    self.home_pattern.body(),
                     BlinkerDirection::Right,
                     time,
                 ));
@@ -465,7 +456,7 @@ impl App {
     }
 
     pub fn tick(&mut self, time: Instant) {
-        match self.ui.check_event(time) {
+        match self.ui.check_event() {
             Some(event) => {
                 self.update_state(event);
                 self.update_animation(time);

bike-lights/core/src/patterns.rs

@@ -26,26 +26,26 @@ pub const BLINKER_AMBER: RGB<I8F8> = RGB {
 };
 
 pub const PRIDE_RED: RGB<I8F8> = RGB {
-    r: I8F8::lit("0.95"),
+    r: I8F8::lit("0.89"),
-    g: I8F8::lit("0.00"),
+    g: I8F8::lit("0.01"),
-    b: I8F8::lit("0.00"),
+    b: I8F8::lit("0.01"),
 };
 
 pub const PRIDE_ORANGE: RGB<I8F8> = RGB {
     r: I8F8::lit("1.0"),
-    g: I8F8::lit("0.25"),
+    g: I8F8::lit("0.54"),
     b: I8F8::lit("0"),
 };
 
 pub const PRIDE_YELLOW: RGB<I8F8> = RGB {
     r: I8F8::lit("1.0"),
-    g: I8F8::lit("0.85"),
+    g: I8F8::lit("0.92"),
     b: I8F8::lit("0"),
 };
 
 pub const PRIDE_GREEN: RGB<I8F8> = RGB {
     r: I8F8::lit("0"),
-    g: I8F8::lit("0.95"),
+    g: I8F8::lit("0.5"),
     b: I8F8::lit("0.05"),
 };
 
@@ -56,9 +56,9 @@ pub const PRIDE_INDIGO: RGB<I8F8> = RGB {
 };
 
 pub const PRIDE_VIOLET: RGB<I8F8> = RGB {
-    r: I8F8::lit("0.75"),
+    r: I8F8::lit("0.45"),
-    g: I8F8::lit("0.0"),
+    g: I8F8::lit("0.16"),
-    b: I8F8::lit("0.80"),
+    b: I8F8::lit("0.50"),
 };
 
 pub const TRANS_BLUE: RGB<I8F8> = RGB {
@@ -73,33 +73,11 @@ pub const TRANS_PINK: RGB<I8F8> = RGB {
     b: I8F8::lit("0.32"),
 };
 
-pub const WATER_1: RGB<I8F8> = RGB {
-    r: I8F8::lit("0.0"),
-    g: I8F8::lit("0.0"),
-    b: I8F8::lit("0.75"),
-};
-
-pub const WATER_2: RGB<I8F8> = RGB {
-    r: I8F8::lit("0.8"),
-    g: I8F8::lit("0.8"),
-    b: I8F8::lit("0.8"),
-};
-
-pub const WATER_3: RGB<I8F8> = RGB {
-    r: I8F8::lit("0.00"),
-    g: I8F8::lit("0.75"),
-    b: I8F8::lit("0.75"),
-};
-
 pub const OFF_DASHBOARD: DashboardPattern = [RGB_OFF; 3];
 pub const OFF_BODY: BodyPattern = [RGB_OFF; 60];
 
 pub const DEFAULT_FRAMES: U16F0 = U16F0::lit("30");
 
-pub const WATER_DASHBOARD: DashboardPattern = [WATER_1, WATER_2, WATER_3];
-
-pub const WATER_BODY: BodyPattern = [RGB_OFF; 60];
-
 pub const PRIDE_DASHBOARD: DashboardPattern = [PRIDE_RED, PRIDE_GREEN, PRIDE_INDIGO];
 
 pub const PRIDE_BODY: BodyPattern = [
@@ -200,7 +178,7 @@ pub const BRAKES_DASHBOARD: DashboardPattern = [BRAKES_RED; 3];
 
 pub const BRAKES_BODY: BodyPattern = [BRAKES_RED; 60];
 
-pub const BLINKER_FRAMES: U16F0 = U16F0::lit("10");
+pub const BLINKER_FRAMES: U16F0 = U16F0::lit("15");
 
 pub const LEFT_BLINKER_DASHBOARD: DashboardPattern = [BLINKER_AMBER, RGB_OFF, RGB_OFF];
 

bike-lights/simulator/src/main.rs

@@ -152,7 +152,7 @@ struct GTKUI {
 }
 
 impl UI for GTKUI {
-    fn check_event(&mut self, _: Instant) -> Option<Event> {
+    fn check_event(&self) -> Option<Event> {
         match self.rx.try_recv() {
             Ok(event) => Some(event),
             Err(TryRecvError::Empty) => None,