wip-3

.vscode/tasks.json

@@ -0,0 +1,29 @@
+{
+	"version": "2.0.0",
+	"tasks": [
+		{
+			"type": "cargo",
+			"command": "build",
+			"problemMatcher": [
+				"$rustc"
+			],
+			"group": {
+				"kind": "build",
+				"isDefault": true
+			},
+			"label": "rust: cargo build"
+		},
+		{
+			"type": "cargo",
+			"command": "espflash flash --no-stub --monitor",
+			"problemMatcher": [
+				"$rustc"
+			],
+			"group": {
+				"kind": "build",
+				"isDefault": false
+			},
+			"label": "rust: cargo espflash"
+		}
+	]
+}

Cargo.toml

@@ -32,8 +32,6 @@ alloc = ["esp-idf-svc/alloc"]
 [dependencies]
 log = { version = "0.4", default-features = false }
 esp-idf-svc = { version = "0.49", default-features = false }
-hsv = "0.1.1"
-palette = { version = "0.7.6" }
 running-average = "0.1.0"
 rgb = "0.8.50"
 
@@ -45,9 +43,11 @@ smart-leds-trait = { version = "0.3.0", optional = true }
 smart-leds = { version = "0.4.0", optional = true }
 
 embedded-graphics = { version = "0.8.1", optional = true, features = ["fixed_point", "defmt"] }
-ansi_term = "0.12.1"
 num = "0.4.3"
 chrono = "0.4.38"
+serde_json = "1.0.133"
+paste = "1.0.15"
+serde = { version = "1.0.216", features = ["derive"] }
 
 [build-dependencies]
 embuild = "0.32.0"

espflash.toml

@@ -0,0 +1,2 @@
+partition_table = "partitions.csv"
+baudrate = 460800

partitions.csv

@@ -0,0 +1,5 @@
+# Name,   Type, SubType, Offset,  Size, Flags
+# Note: if you have increased the bootloader size, make sure to update the offsets to avoid overlap
+nvs,      data, nvs,     ,        0x6000,
+phy_init, data, phy,     ,        0x1000,
+factory,  app,  factory, ,        3M,

src/TODO.md

@@ -1,15 +1,18 @@
 [x] cfg macros
 [ ] warnings
 [x] rgb crate
-[ ] Layer blending
-[ ] Refactor idle pattern into test pattern
-[ ] Wifi
+[x] Layer blending
+[x] Refactor idle pattern into test pattern
+[x] Wifi
 [ ] JSON surface map loading
 [ ] Weather
-[ ] Circadian Rhythm
-[ ] NTP
+[x] Circadian Rhythm
+[x] NTP
 [ ] Config to only start a subset of tasks on startup
 [ ] Serial CLI
 [ ] Surface blending API
 [ ] Layer blending equations
 [ ] Surface rotation
+[ ] esp8266 port
+[ ] event system
+[ ] threaded schedulers

src/animations/mod.rs

@@ -0,0 +1,86 @@
+pub mod test;
+
+use rgb::RGB8;
+use crate::lib8::Hsv;
+
+use crate::events::EventBus;
+use crate::geometry::*;
+use crate::render::{Shader, Surface, Surfaces};
+use crate::task::{Environment, Task};
+use crate::lib8::{trig::{sin8, cos8}, noise::inoise8, IntoRgb8};
+
+#[derive(Debug)]
+pub struct IdleTask<T: Surface> {
+    solid: T,
+    surface: T,
+    shimmer: T,
+}
+
+#[derive(Debug)]
+struct SolidShader {}
+
+impl Shader for SolidShader {
+    fn draw(&self, _coords: &VirtualCoordinates, frame: usize) -> RGB8 {
+        Hsv::new((frame % 255) as u8, 255, sin8((frame % 64) as u8)).into_rgb8()
+    }
+}
+
+#[derive(Debug)]
+struct ShimmerShader {}
+
+impl Shader for ShimmerShader {
+    fn draw(&self, coords: &VirtualCoordinates, frame: usize) -> RGB8 {
+        Hsv::new(((coords.x as usize).wrapping_add(frame / 3) % 255) as u8, coords.y, sin8(frame).max(75).min(64)).into_rgb8()
+    }
+}
+
+#[derive(Debug)]
+struct ThinkingShader {}
+
+impl Shader for ThinkingShader {
+    fn draw(&self, coords: &VirtualCoordinates, frame: usize) -> RGB8 {
+        //let noise_x = sin8(sin8((frame % 255) as u8).wrapping_add(coords.x));
+        //let noise_y = cos8(cos8((frame % 255) as u8).wrapping_add(coords.y));
+        let offset_x = sin8(frame.wrapping_add(coords.x as usize));
+        let offset_y = cos8(frame.wrapping_add(coords.y as usize));
+        let noise_x = offset_x / 2;
+        let noise_y = offset_y / 2;
+        //let noise_x = coords.x.wrapping_add(offset_x);
+        //let noise_y = coords.y.wrapping_add(offset_y);
+        Hsv::new(
+            inoise8(offset_x as i16, offset_y as i16),
+            128_u8.saturating_add(inoise8(noise_y as i16, noise_x as i16)),
+            255
+        ).into_rgb8()
+    }
+}
+
+impl<T: Surface> IdleTask<T> {
+    pub fn new<S: Surfaces<Surface = T>>(surfaces: &mut S) -> Self {
+        IdleTask {
+            solid: surfaces.new_surface(Rectangle::everything()).unwrap(),
+            surface: surfaces.new_surface(Rectangle::everything()).unwrap(),
+            shimmer: surfaces.new_surface(Rectangle::everything()).unwrap(),
+        }
+    }
+}
+
+impl<T: Surface> Task for IdleTask<T> {
+    fn name(&self) -> &'static str { "Idle" }
+
+    fn start(&mut self, _env: &mut Environment) {
+        self.solid.set_shader(SolidShader {});
+        self.surface.set_shader(ThinkingShader { });
+        self.shimmer.set_shader(ShimmerShader { });
+
+        self.solid.set_opacity(64);
+        self.surface.set_opacity(128);
+        self.shimmer.set_opacity(64);
+    }
+
+    fn stop(&mut self, _env: &mut Environment) {
+        self.solid.clear_shader();
+        self.surface.clear_shader();
+        self.shimmer.clear_shader();
+    }
+}

src/animations/test.rs

@@ -1,88 +1,9 @@
-use palette::Hsv;
-
+use crate::{lib8::Hsv, task::Environment};
 use rgb::RGB8;
 
-use crate::time::Periodically;
-use crate::geometry::*;
-use crate::render::{Shader, Surface, Surfaces};
-use crate::task::Task;
-use crate::lib8::{trig::{sin8, cos8}, interpolate::scale8, noise::inoise8, IntoRgb8};
+use crate::{events::EventBus, lib8::{interpolate::scale8, trig::{cos8, sin8}, IntoRgb8}, render::{Shader, Surface}, task::Task, time::Periodically};
 
-#[derive(Debug)]
-pub struct IdleTask<T: Surface> {
-    solid: T,
-    surface: T,
-    shimmer: T,
-}
-
-#[derive(Debug)]
-struct SolidShader {}
-
-impl Shader for SolidShader {
-    fn draw(&self, _coords: &VirtualCoordinates, frame: usize) -> RGB8 {
-        Hsv::new_srgb((frame % 255) as u8, 255, sin8((frame % 64) as u8)).into_rgb8()
-    }
-}
-
-#[derive(Debug)]
-struct ShimmerShader {}
-
-impl Shader for ShimmerShader {
-    fn draw(&self, coords: &VirtualCoordinates, frame: usize) -> RGB8 {
-        Hsv::new_srgb(((coords.x as usize).wrapping_add(frame / 3) % 255) as u8, coords.y, sin8(frame).max(75).min(64)).into_rgb8()
-    }
-}
-
-#[derive(Debug)]
-struct ThinkingShader {}
-
-impl Shader for ThinkingShader {
-    fn draw(&self, coords: &VirtualCoordinates, frame: usize) -> RGB8 {
-        //let noise_x = sin8(sin8((frame % 255) as u8).wrapping_add(coords.x));
-        //let noise_y = cos8(cos8((frame % 255) as u8).wrapping_add(coords.y));
-        let offset_x = sin8(frame.wrapping_add(coords.x as usize));
-        let offset_y = cos8(frame.wrapping_add(coords.y as usize));
-        let noise_x = offset_x / 2;
-        let noise_y = offset_y / 2;
-        //let noise_x = coords.x.wrapping_add(offset_x);
-        //let noise_y = coords.y.wrapping_add(offset_y);
-        Hsv::new_srgb(
-            inoise8(offset_x as i16, offset_y as i16),
-            128_u8.saturating_add(inoise8(noise_y as i16, noise_x as i16)),
-            255
-        ).into_rgb8()
-    }
-}
-
-impl<T: Surface> IdleTask<T> {
-    pub fn new<S: Surfaces<Surface = T>>(surfaces: &mut S) -> Self {
-        IdleTask {
-            solid: surfaces.new_surface(&Rectangle::everything()).unwrap(),
-            surface: surfaces.new_surface(&Rectangle::everything()).unwrap(),
-            shimmer: surfaces.new_surface(&Rectangle::everything()).unwrap(),
-        }
-    }
-}
-
-impl<T: Surface> Task for IdleTask<T> {
-    fn start(&mut self) {
-        self.solid.set_shader(Box::new(SolidShader { }));
-        self.surface.set_shader(Box::new(ThinkingShader { }));
-        self.shimmer.set_shader(Box::new(ShimmerShader { }));
-
-        self.solid.set_opacity(64);
-        self.surface.set_opacity(128);
-        self.shimmer.set_opacity(64);
-    }
-
-    fn tick(&mut self) {}
-
-    fn stop(&mut self) {
-        self.solid.clear_shader();
-        self.surface.clear_shader();
-        self.shimmer.clear_shader();
-    }
-}
+use super::{Coordinates, Rectangle, VirtualCoordinates};
 
 #[derive(Clone, Copy, Debug)]
 enum TestShader {
@@ -127,7 +48,7 @@ impl Shader for TestShader {
             Self::Blue => RGB8::new(0, 0, 255),
             Self::White => RGB8::new(255, 255, 255),
             Self::RGB => RGB8::new(coords.x, coords.y, 255 - (coords.x / 2 + coords.y / 2)),
-            Self::HSV => Hsv::new_srgb(coords.x, coords.y, 255).into_rgb8(),
+            Self::HSV => Hsv::new(coords.x, coords.y, 255).into_rgb8(),
             Self::Outline => match (coords.x, coords.y) {
                 (0, 0) => RGB8::new(255, 255, 255),
                 (0, _) => RGB8::new(255, 0, 0),
@@ -167,23 +88,27 @@ impl<T: Surface> TestPattern<T> {
     }
 }
 
+//const Animations: Namespace = Namespace("animations");
+
 impl<T: Surface> Task for TestPattern<T> {
-    fn start(&mut self) {
-        self.surface.set_shader(Box::new(self.pattern.clone()));
+    fn name(&self) -> &'static str { "TestPattern" }
+
+    fn start(&mut self, _env: &mut Environment) {
+        self.surface.set_shader(self.pattern);
     }
 
-    fn tick(&mut self) {
+    fn on_tick(&mut self, _env: &mut Environment) {
         self.updater.run(|| {
             self.pattern = self.pattern.next();
             log::info!("Test pattern: {:?}", self.pattern);
             self.frame = 0;
-            self.surface.set_shader(Box::new(self.pattern.clone()));
+            self.surface.set_shader(self.pattern);
+            //bus.push(Animations.new_property_change( "test.pattern", format!("{:?}", self.pattern)));
         });
         self.stepper.run(|| {
             self.frame = self.frame.wrapping_add(1);
             self.surface.set_opacity(sin8(self.frame));
-            //log::info!("Step {}", self.frame);
-            self.surface.set_rect( &match self.pattern {
+            self.surface.set_rect( match self.pattern {
                 TestShader::SweepX => Rectangle::new(
                     Coordinates::new(self.frame, 0),
                     Coordinates::new(self.frame, 255)
@@ -197,7 +122,7 @@ impl<T: Surface> Task for TestPattern<T> {
         });
     }
 
-    fn stop(&mut self) {
+    fn stop(&mut self, _env: &mut Environment) {
         self.surface.clear_shader();
     }
-}
+}

src/buffers.rs

@@ -2,24 +2,19 @@ use crate::geometry::*;
 use crate::lib8::interpolate::Fract8Ops;
 use crate::power::AsMilliwatts;
 use crate::render::{PixelView, Sample, Shader, Surface, Surfaces, HardwarePixel};
-use crate::task::Task;
 
-use std::fmt::Debug;
-use std::io;
+use std::cell::RefCell;
 use std::ops::IndexMut;
 
 use std::sync::atomic::AtomicBool;
-use std::sync::RwLock;
 use std::sync::{Arc, Mutex};
 
-#[derive(Debug)]
 struct ShaderBinding {
     shader: Option<Box<dyn Shader>>,
     rect: Rectangle<Virtual>,
     opacity: u8
 }
 
-#[derive(Debug)]
 struct SurfaceUpdate {
     shader: Option<Option<Box<dyn Shader>>>,
     rect: Option<Rectangle<Virtual>>,
@@ -41,12 +36,6 @@ impl SurfaceUpdate {
     }
 }
 
-impl Debug for Box<dyn Shader> {
-    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-        f.debug_struct("Shader").finish()
-    }
-}
-
 impl Default for SurfaceUpdate {
     fn default() -> Self {
         SurfaceUpdate {
@@ -58,7 +47,6 @@ impl Default for SurfaceUpdate {
     }
 }
 
-#[derive(Debug)]
 pub struct BufferedSurface {
     updater: Arc<UpdateQueue>,
     slot: usize
@@ -81,24 +69,23 @@ impl Surface for BufferedSurface {
         });
     }
 
-    fn set_rect(&mut self, rect: &Rectangle<Virtual>) {
+    fn set_rect(&mut self, rect: Rectangle<Virtual>) {
         self.updater.push(SurfaceUpdate {
-            rect: Some(rect.clone()),
+            rect: Some(rect),
             slot: self.slot,
             ..Default::default()
         });
     }
 
-    fn set_shader(&mut self, shader: Box<dyn Shader>) {
+    fn set_shader<T: Shader + 'static>(&mut self, shader: T) {
         self.updater.push(SurfaceUpdate {
-            shader: Some(Some(shader)),
+            shader: Some(Some(Box::new(shader))),
             slot: self.slot,
             ..Default::default()
         });
     }
 }
 
-#[derive(Debug)]
 struct UpdateQueue {
     pending: Mutex<Vec<SurfaceUpdate>>,
     damaged: AtomicBool
@@ -135,7 +122,6 @@ impl UpdateQueue {
     }
 }
 
-#[derive(Debug)]
 pub struct ShaderChain {
     bindings: Vec<ShaderBinding>,
     updates: Arc<UpdateQueue>
@@ -154,36 +140,33 @@ impl ShaderChain {
     }
 
     pub fn commit(&mut self) {
-        let mut queue: Vec<SurfaceUpdate> = {
-            let mut updates = self.updates.pending.lock().unwrap();
-            std::mem::take(updates.as_mut())
-        };
-        for update in queue.iter_mut() {
-            let target_slot = &mut self.bindings[update.slot];
-            if let Some(shader) = update.shader.take() {
-                target_slot.shader = shader;
-            }
-            if let Some(opacity) = update.opacity.take() {
-                target_slot.opacity = opacity;
-            }
-            if let Some(rect) = update.rect.take() {
-                target_slot.rect = rect;
+        if self.is_dirty() {
+            let mut queue: Vec<SurfaceUpdate> = {
+                let mut updates = self.updates.pending.lock().unwrap();
+                std::mem::take(updates.as_mut())
+            };
+            for update in queue.iter_mut() {
+                let target_slot = &mut self.bindings[update.slot];
+                if let Some(shader) = update.shader.take() {
+                    target_slot.shader = shader;
+                }
+                if let Some(opacity) = update.opacity.take() {
+                    target_slot.opacity = opacity;
+                }
+                if let Some(rect) = update.rect.take() {
+                    target_slot.rect = rect;
+                }
             }
+            self.updates.damaged.store(false, std::sync::atomic::Ordering::Relaxed);
         }
-        self.updates.damaged.store(false, std::sync::atomic::Ordering::Relaxed);
     }
-}
 
-impl Surfaces for ShaderChain {
-    type Error = ();
-    type Surface = BufferedSurface;
-
-    fn new_surface(&mut self, area: &Rectangle<Virtual>) -> Result<Self::Surface, Self::Error> {
+    fn new_surface(&mut self, area: Rectangle<Virtual>) -> Result<BufferedSurface, ()> {
         let next_slot = self.bindings.len();
         self.bindings.push(ShaderBinding {
             opacity: 255,
             shader: None,
-            rect: area.clone()
+            rect: area
         });
 
         Ok(BufferedSurface {
@@ -193,12 +176,12 @@ impl Surfaces for ShaderChain {
     }
 
     fn render_to<S: Sample>(&self, output: &mut S, frame: usize) {
-        for surface in self.bindings.iter() {
+        for surface in &self.bindings {
             let opacity = surface.opacity;
             if opacity > 0 {
-                let rect = surface.rect;
-                let mut sample = output.sample(&rect);
                 if let Some(ref shader) = surface.shader {
+                    let rect = surface.rect;
+                    let mut sample = output.sample(&rect);
                     while let Some((virt_coords, pixel)) = sample.next() {
                         *pixel = pixel.blend8(shader.draw(&virt_coords, frame).into(), opacity);
                     }
@@ -208,115 +191,29 @@ impl Surfaces for ShaderChain {
     }
 }
 
-#[derive(Clone)]
 pub struct BufferedSurfacePool {
-    pool: Arc<RwLock<ShaderChain>>
+    pool: RefCell<ShaderChain>
 }
 
 impl BufferedSurfacePool {
     pub fn new() -> Self {
         BufferedSurfacePool {
-            pool: Arc::new(RwLock::new(ShaderChain::new()))
+            pool: RefCell::new(ShaderChain::new())
         }
     }
 }
 
 impl Surfaces for BufferedSurfacePool {
     type Error = ();
-    type Surface = <ShaderChain as Surfaces>::Surface;
-    fn new_surface(&mut self, area: &crate::geometry::Rectangle<crate::geometry::Virtual>) -> Result<Self::Surface, Self::Error> {
-        self.pool.write().unwrap().new_surface(area)
+    type Surface = BufferedSurface;
+    fn new_surface(&mut self, area: crate::geometry::Rectangle<crate::geometry::Virtual>) -> Result<Self::Surface, Self::Error> {
+        self.pool.borrow_mut().new_surface(area)
     }
 
     fn render_to<S: crate::render::Sample>(&self, output: &mut S, frame: usize) {
-        self.pool.read().unwrap().render_to(output, frame);
-    }
-}
-
-
-impl Task for BufferedSurfacePool {
-    fn tick(&mut self) {
-        if self.pool.read().unwrap().is_dirty() {
-            self.pool.write().unwrap().commit();
-        }
-    }
-}
-
-#[derive(Clone)]
-pub struct SharedSurface {
-    binding: Arc<Mutex<ShaderBinding>>
-}
-
-
-impl Default for SharedSurface {
-    fn default() -> Self {
-        Self {
-            binding: Arc::new(Mutex::new(ShaderBinding {
-                shader: None,
-                rect: Rectangle::everything(),
-                opacity: 255
-            })),
-        }
-    }
-}
-
-impl Surface for SharedSurface {
-    fn set_shader(&mut self, shader: Box<dyn Shader>) {
-        self.binding.lock().unwrap().shader = Some(shader);
-    }
-
-    fn clear_shader(&mut self) {
-        self.binding.lock().unwrap().shader = None;
-    }
-
-    fn set_rect(&mut self, rect: &Rectangle<Virtual>) {
-        self.binding.lock().unwrap().rect = rect.clone();
-    }
-
-    fn set_opacity(&mut self, opacity: u8) {
-        self.binding.lock().unwrap().opacity = opacity
-    }
-}
-
-
-#[derive(Clone)]
-pub struct SurfacePool {
-    surfaces: Vec<SharedSurface>
-}
-
-impl SurfacePool {
-    pub const fn new() -> Self {
-        Self {
-            surfaces: Vec::new()
-        }
-    }
-}
-
-impl Surfaces for SurfacePool {
-    type Surface = SharedSurface;
-    type Error = io::Error;
-    fn new_surface(&mut self, area: &Rectangle<Virtual>) -> Result<Self::Surface, Self::Error> {
-        let mut surface = SharedSurface::default();
-        surface.set_rect(area);
-        self.surfaces.push(surface.clone());
-        return Ok(surface);
-    }
-
-    fn render_to<Sampler: Sample>(&self, output: &mut Sampler, frame: usize) {
-        for surface in self.surfaces.iter() {
-            let binding = surface.binding.lock().unwrap();
-            let opacity = binding.opacity;
-            if opacity > 0 {
-                let rect = binding.rect;
-                let mut sample = output.sample(&rect);
-
-                if let Some(ref shader) = binding.shader {
-                    while let Some((virt_coords, pixel)) = sample.next() {
-                        *pixel = pixel.blend8(shader.draw(&virt_coords, frame).into(), opacity);
-                    }
-                }
-            }
-        }
+        let mut b = self.pool.borrow_mut();
+        b.commit();
+        b.render_to(output, frame);
     }
 }
 

src/events.rs

@@ -0,0 +1,102 @@
+use core::fmt::Debug;
+use std::{collections::VecDeque, fmt::{Display, Result}, sync::{Arc, Mutex}};
+
+use crate::{properties::{Properties, PropertyID, Variant}, property_namespace};
+
+#[derive(Debug, Clone, PartialEq, Eq)]
+pub enum Event {
+    ReadyToRock,
+    Tick,
+    StartThing(&'static str),
+    StopThing(&'static str),
+    PropertyChange(PropertyID, Variant)
+}
+
+impl Display for Event {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> Result {
+        match self {
+            Self::ReadyToRock => f.write_str("ReadyToRock"),
+            Self::Tick => f.write_str("Tick"),
+            Self::StartThing(name) => write!(f, "Start {}", *name),
+            Self::StopThing(name) => write!(f, "Stop {}", *name),
+            Self::PropertyChange(id, value) => write!(f, "{} -> {:?}", id, value)
+        }
+    }
+}
+
+impl Event {
+    pub const fn new_tick() -> Self {
+        Event::Tick
+    }
+
+    pub fn new_property_change<T>(key: PropertyID, data: T) -> Self where Variant: From<T> {
+        Event::PropertyChange(key, Variant::from(data))
+    }
+
+    pub const fn new_ready_to_rock() -> Self {
+        Event::ReadyToRock
+    }
+
+    pub const fn new_start_thing(name: &'static str) -> Self {
+        Event::StartThing(name)
+    }
+
+    pub const fn new_stop_thing(name: &'static str) -> Self {
+        Event::StopThing(name)
+    }
+}
+
+pub trait Evented {
+    fn on_event(&mut self, event: &Event, bus: &mut EventBus);
+}
+
+pub trait EventSource {
+    fn next(&mut self) -> Event;
+    fn push(&mut self, event: Event);
+}
+
+#[derive(Clone)]
+pub struct EventBus {
+    pending: Arc<Mutex<VecDeque<Event>>>,
+    props: Arc<Mutex<Properties>>
+}
+
+impl EventBus {
+    pub fn new() -> Self {
+        EventBus {
+            pending: Arc::new(Mutex::new(VecDeque::with_capacity(32))),
+            props: Arc::new(Mutex::new(Properties::new()))
+        }
+    }
+}
+
+impl EventSource for EventBus {
+    fn next(&mut self) -> Event {
+        let next = self.pending.lock().unwrap().pop_front().unwrap_or(Event::new_tick());
+
+        match next {
+            Event::PropertyChange(key, value) => {
+                if self.props.lock().unwrap().set(key, value.clone()) {
+                    log::trace!("prop-update key={:?} value={:?}", key, value);
+                    Event::PropertyChange(key, value)
+                } else {
+                    Event::new_tick()
+                }
+            },
+            _ => next
+        }
+    }
+
+    fn push(&mut self, event: Event) {
+        self.pending.lock().unwrap().push_back(event);
+    }
+}
+
+property_namespace!(
+    System,
+    NetworkOnline => false,
+    IP => "",
+    Gateway => "",
+    TimeSync => false,
+    Idle => false
+);

src/geometry.rs

@@ -64,19 +64,19 @@ impl<S: CoordinateSpace> Coordinates<S> {
         }
     }
 
-    fn top_left() -> Self {
+    const fn top_left() -> Self {
         Self::new(S::Data::MIN, S::Data::MIN)
     }
 
-    fn top_right() -> Self {
+    const fn top_right() -> Self {
         Self::new(S::Data::MAX, S::Data::MIN)
     }
 
-    fn bottom_left() -> Self {
+    const fn bottom_left() -> Self {
         Self::new(S::Data::MIN, S::Data::MAX)
     }
 
-    fn bottom_right() -> Self {
+    const fn bottom_right() -> Self {
         Self::new(S::Data::MAX, S::Data::MAX)
     }
 
@@ -101,15 +101,13 @@ pub struct Rectangle<Space: CoordinateSpace> {
 
 impl<Space: CoordinateSpace> Rectangle<Space> {
     pub const fn new(top_left: Coordinates<Space>, bottom_right: Coordinates<Space>) -> Self {
-        //debug_assert!(top_left.x <= bottom_right.x);
-        //debug_assert!(top_left.y <= bottom_right.y);
         Self {
             top_left,
             bottom_right
         }
     }
 
-    pub fn everything() -> Self {
+    pub const fn everything() -> Self {
         Self {
             top_left: Coordinates::<Space>::top_left(),
             bottom_right: Coordinates::<Space>::bottom_right()

src/inputs/circadian.rs

@@ -0,0 +1,102 @@
+use chrono::{DateTime, Timelike, Utc};
+
+use crate::{events::EventBus, lib8::interpolate::lerp8by8, properties::{PropertyID, Variant}, render::props::Output as OutputNS, task::{Environment, Task}, time::Periodically};
+use crate::events::System as SystemNS;
+
+#[derive(Debug, Clone)]
+struct ScheduleEntry {
+    hour: u8,
+    brightness: u8
+}
+
+pub struct CircadianRhythm {
+    time_check: Periodically,
+    schedule: [ScheduleEntry;10]
+}
+
+impl CircadianRhythm {
+    pub fn new() -> Self {
+        CircadianRhythm {
+            time_check: Periodically::new_every_n_seconds(60),
+            schedule: [
+                ScheduleEntry { hour: 0, brightness: 0 },
+                ScheduleEntry { hour: 5, brightness: 0 },
+                ScheduleEntry { hour: 6, brightness: 10 },
+                ScheduleEntry { hour: 7, brightness: 20 },
+                ScheduleEntry { hour: 8, brightness: 80 },
+                ScheduleEntry { hour: 11, brightness: 120 },
+                ScheduleEntry { hour: 18, brightness: 200 },
+                ScheduleEntry { hour: 19, brightness: 255 },
+                ScheduleEntry { hour: 22, brightness: 120 },
+                ScheduleEntry { hour: 23, brightness: 5 }
+            ]
+        }
+    }
+
+    fn update_brightness(&self, env: &mut Environment) {
+        let now: DateTime<Utc> = std::time::SystemTime::now().into();
+        let next_brightness = self.brightness_for_time(now.hour() as u8, now.minute() as u8);
+        env.set_property(OutputNS::Brightness, next_brightness);
+    }
+
+    fn brightness_for_time(&self, hour: u8, minute: u8) -> u8 {
+        let mut start = self.schedule.last().unwrap();
+        let mut end = self.schedule.first().unwrap();
+        for cur in self.schedule.iter() {
+            if cur.hour <= hour {
+                start = cur;
+            } else {
+                end = cur;
+                break;
+            }
+        }
+
+        log::info!("hour={:?} minute={:?} start={:?} end={:?}", hour, minute, start, end);
+
+        let mut adjusted_end = end.clone();
+        if start.hour > end.hour {
+            adjusted_end.hour += 24;
+        }
+
+        let start_time = (start.hour as u16).wrapping_mul(60);
+        let end_time = (end.hour as u16).wrapping_mul(60);
+        let now_time = (hour as u16).wrapping_mul(60).wrapping_add(minute as u16);
+
+        let duration = end_time - start_time;
+        let cur_duration = now_time - start_time;
+
+        let frac = map_range(cur_duration.into(), 0, duration.into(), 0, 255) as u8;
+
+        lerp8by8(start.brightness, end.brightness, frac)
+    }
+}
+
+fn map_range(x: u16, in_min: u16, in_max: u16, out_min: u16, out_max: u16) -> u16 {
+    let run = in_max - in_min;
+    if run == 0 {
+        return 0;
+    }
+    let rise = out_max - out_min;
+    let delta = x - in_min;
+    return (delta.wrapping_mul(rise)) / run + out_min;
+}
+
+
+impl Task for CircadianRhythm {
+    fn on_ready(&mut self, env: &mut Environment) {
+        self.update_brightness(env);
+    }
+
+    fn on_property_change(&mut self, key: PropertyID, value: &Variant, env: &mut Environment) {
+        match (key, value) {
+            (SystemNS::TimeSync, Variant::Boolean(true)) => self.update_brightness(env),
+            _ => ()
+        }
+    }
+
+    fn on_tick(&mut self, env: &mut Environment) {
+        if self.time_check.tick() {
+            self.update_brightness(env);
+        }
+    }
+}

src/inputs/idle.rs

@@ -0,0 +1,38 @@
+use std::time::Instant;
+
+use std::time::Duration;
+
+use crate::task::Environment;
+use crate::{task::Task, time::Periodically};
+use crate::events::System;
+
+pub struct IdleTimer {
+    timeout_start: Instant,
+    timeout_duration: Duration,
+    timeout_check: Periodically
+}
+
+impl IdleTimer {
+    pub fn new() -> Self {
+        IdleTimer {
+            timeout_start: Instant::now(),
+            timeout_duration: Duration::from_secs(60),
+            timeout_check: Periodically::new_every_n_seconds(5)
+        }
+    }
+}
+
+impl Task for IdleTimer {
+    fn start(&mut self, env: &mut Environment) {
+        env.set_property(System::Idle, true);
+    }
+
+    fn on_tick(&mut self, env: &mut Environment) {
+        if self.timeout_check.tick() {
+            let is_idle: bool = env.get_property(System::Idle).unwrap();
+            if !is_idle && self.timeout_start.elapsed() >= self.timeout_duration {
+                env.set_property(System::Idle, true);
+            }
+        }
+    }
+}

src/inputs/mod.rs

@@ -0,0 +1,2 @@
+pub mod circadian;
+pub mod idle;

src/lib8/interpolate.rs

@@ -12,12 +12,21 @@ pub trait Fract8Ops {
 
 impl Fract8Ops for u8 {
     fn scale8(self, scale: Fract8) -> Self {
-        // borrowed from FastLED
-        (self as u16 * (1 + scale as u16)).unsigned_shr(8) as u8
+        match scale {
+            0 => 0,
+            255 => self,
+            _ => 
+                // borrowed from FastLED
+                (self as u16 * (1 + scale as u16)).unsigned_shr(8) as u8
+        }
     }
 
     fn blend8(self, other: Self, scale: Fract8) -> Self {
-        ((((self as u16).unsigned_shl(8).bitor(other as u16)) as u16).wrapping_add(other as u16 * scale as u16).wrapping_sub(self as u16 * scale as u16)).unsigned_shr(8) as u8
+        match scale {
+            0 => self,
+            255 => other,
+            _ => ((((self as u16).unsigned_shl(8).bitor(other as u16)) as u16).wrapping_add(other as u16 * scale as u16).wrapping_sub(self as u16 * scale as u16)).unsigned_shr(8) as u8
+        }
     }
 }
 
@@ -31,13 +40,17 @@ impl Fract8Ops for Rgb<u8> {
     }
 
     fn blend8(self, other: Self, scale: Fract8) -> Self {
-        match (other.r, other.g, other.b) {
-            (0, 0, 0) => self,
-            _ => Rgb::new(
-                self.r.blend8(other.r, scale),
-                self.g.blend8(other.g, scale),
-                self.b.blend8(other.b, scale)
-            )
+        match scale {
+            0 => self,
+            255 => other,
+            _ => match (other.r, other.g, other.b) {
+                (0, 0, 0) => self,
+                _ => Rgb::new(
+                    self.r.blend8(other.r, scale),
+                    self.g.blend8(other.g, scale),
+                    self.b.blend8(other.b, scale)
+                )
+            }
         }
     }
 }
@@ -82,6 +95,25 @@ pub fn lerp7by8(a: i8, b: i8, frac: u8) -> i8 {
     }
 }
 
+pub fn lerp8by8(a: u8, b: u8, frac: u8) -> u8 {
+    if b > a {
+        let delta = b - a;
+        let scaled = scale8(delta, frac);
+        return a + scaled;
+    } else {
+        let delta = a - b;
+        let scaled = scale8(delta, frac);
+        return a - scaled;
+    }
+}
+
+pub fn map8(x: u8, range_start: u8, range_end: u8) -> u8 {
+    let range_width = range_end - range_start;
+    let mut out = scale8(x, range_width);
+    out += range_start;
+    return out;
+}
+
 pub fn ease8InOutQuad(i: u8) -> u8 {
     let j = if i & 0x80 != 0 {
         255 - i

src/lib8/mod.rs

@@ -2,8 +2,6 @@ pub mod interpolate;
 pub mod noise;
 pub mod trig;
 
-use palette::encoding::srgb::Srgb;
-
 use rgb::Rgb;
 
 use crate::lib8::interpolate::scale8;
@@ -18,7 +16,23 @@ impl IntoRgb8 for Rgb<u8> {
     }
 }
 
-impl IntoRgb8 for palette::Hsv<Srgb, u8> {
+pub struct Hsv {
+    pub hue: u8,
+    pub saturation: u8,
+    pub value: u8
+}
+
+impl Hsv {
+    pub fn new(hue: u8, saturation: u8, value: u8) -> Self {
+        Hsv {
+            hue,
+            saturation,
+            value
+        }
+    }
+}
+
+impl IntoRgb8 for Hsv {
     //TODO: Borrowed from FastLED
     fn into_rgb8(self) -> Rgb<u8> {
         const HSV_SECTION_3: u8 = 0x40;
@@ -27,7 +41,7 @@ impl IntoRgb8 for palette::Hsv<Srgb, u8> {
             return Rgb::new(self.value, self.value, self.value)
         }
 
-        let mock_hue = scale8(191, self.hue.into_inner());
+        let mock_hue = scale8(191, self.hue);
         let value: u8 = self.value;
         let saturation: u8 = self.saturation;
         let invsat: u8 = 255 - saturation;
@@ -52,4 +66,4 @@ impl IntoRgb8 for palette::Hsv<Srgb, u8> {
             _ => Rgb::new(rampup_adj_with_floor, brightness_floor, rampdown_adj_with_floor)
         }
     }
-}
+}

src/main.rs

@@ -8,41 +8,98 @@ mod platform;
 mod animations;
 mod mappings;
 mod buffers;
+mod scenes;
+mod inputs;
+mod events;
+mod properties;
 
-use crate::platform::{DefaultBoard, Board};
-use crate::task::{FixedSizeScheduler, Scheduler};
+use events::*;
+use inputs::circadian::CircadianRhythm;
+use inputs::idle::IdleTimer;
+use platform::esp32::mqtt::props::MQTT;
+use render::props::Output;
+use scenes::Sequencer;
+
+use crate::events::{Evented, EventBus};
+use crate::platform::{DefaultBoard, Board, props::Board as BoardNS};
+use crate::task::FixedSizeScheduler;
 use crate::render::{Surfaces, Renderer};
 use crate::geometry::Rectangle;
+use crate::scenes::props::Scenes as SceneNS;
 
 fn main() {
     let mut board: DefaultBoard = Board::take();
 
-    log::info!("Board: {}", core::any::type_name_of_val(&board));
+    log::info!("🐛 Booting Renderbug!");
 
-    log::info!("Creating tasks");
+    log::info!("📡 Board {}", core::any::type_name_of_val(&board));
+
+    log::info!("⚙️ Creating tasks");
     let mut system = board.system_tasks();
-    log::info!("System scheduler: {}", core::any::type_name_of_val(&system));
+    log::info!("⏰ System scheduler: {}", core::any::type_name_of_val(&system));
 
-    log::info!("Creating output");
+    log::info!("💡 Creating output");
     let output = board.output();
     log::info!("Output: {}", core::any::type_name_of_val(&output));
 
-    log::info!("Preparing surfaces");
+    log::info!("🎨 Preparing surfaces");
     let mut surfaces = board.surfaces();
     log::info!("Surface implementation: {}", core::any::type_name_of_val(&output));
 
-    log::info!("Creating animations");
+    log::info!("🌌 Creating animations");
     let mut animations = FixedSizeScheduler::new([
         Box::new(animations::IdleTask::new(&mut surfaces)),
-        Box::new(animations::TestPattern::new(surfaces.new_surface(&Rectangle::everything()).unwrap())),
+        Box::new(animations::test::TestPattern::new(surfaces.new_surface(Rectangle::everything()).unwrap())),
+    ]);
+
+    let mut inputs = FixedSizeScheduler::new([
+        Box::new(CircadianRhythm::new()),
+        Box::new(Sequencer::new()),
+        Box::new(IdleTimer::new())
     ]);
 
     let mut renderer = FixedSizeScheduler::new([Box::new(Renderer::new(output, surfaces))]);
 
-    log::info!("Ready to rock and roll");
+    log::info!("🚌 Starting event bus");
+    let mut bus = EventBus::new();
+
+    {
+        let mut props = bus.properties();
+        props.add_namespace::<System>();
+        props.add_namespace::<BoardNS>();
+        props.add_namespace::<Output>();
+        props.add_namespace::<SceneNS>();
+        props.add_namespace::<MQTT>();
+
+        props.set(BoardNS::ChipID, DefaultBoard::chip_id());
+
+        log::info!("System properties:");
+        log::info!("{}", props);
+    }
+
+    log::info!("Priming events...");
+    let initial_tasks = [
+        "Renderer",
+        "renderbug::scenes::Sequencer",
+        "renderbug::platform::esp32::wifi::WifiTask",
+        "renderbug::platform::esp32::mqtt::MqttTask",
+        "renderbug::inputs::circadian::CircadianRhythm",
+        "renderbug::inputs::idle::IdleTimer"
+    ];
+    for task_name in initial_tasks {
+        bus.push(Event::new_start_thing(task_name));
+    }
+    bus.push(Event::new_ready_to_rock());
+    log::info!("🚀 Ready to rock and roll");
     loop {
-        animations.tick();
-        system.tick();
-        renderer.tick();
+        let next_event = bus.next();
+        match next_event {
+            events::Event::Tick => (),
+            _ => log::info!("⚡ Event: {}", next_event)
+        }
+        inputs.on_event(&next_event, &mut bus);
+        animations.on_event(&next_event, &mut bus);
+        system.on_event(&next_event, &mut bus);
+        renderer.on_event(&next_event, &mut bus);
     }
 }

src/mappings.rs

@@ -20,7 +20,7 @@ pub trait Select<'a> {
 
 #[derive(Debug)]
 pub struct LinearCoordView {
-    rect: Rectangle<Virtual>,
+    max_x: u8,
     idx: usize,
 }
 
@@ -34,7 +34,7 @@ pub type LinearCoords = Coordinates<LinearSpace>;
 impl<'a> CoordinateView<'a> for LinearCoordView {
     type Space = LinearSpace;
     fn next(&mut self) -> Option<(VirtualCoordinates, LinearCoords)> {
-        if self.idx as u8 == self.rect.bottom_right.x {
+        if self.idx as u8 == self.max_x {
             None
         } else {
             let virt = VirtualCoordinates::new(self.idx as u8, 0); // FIXME: scale8
@@ -62,7 +62,7 @@ impl<'a> Select<'a> for LinearPixelMapping {
     type View = LinearCoordView;
     fn select(&'a self, rect: &Rectangle<Virtual>) -> Self::View {
         LinearCoordView {
-            rect: rect.clone(),
+            max_x: rect.bottom_right.x,
             idx: 0,
         }
     }
@@ -101,6 +101,30 @@ impl<const STRIDE_NUM: usize> StrideMapping<STRIDE_NUM> {
         ])
     }
 
+    pub fn new_fairylights() -> Self {
+        Self::from_json(&[
+            (0, 0, 50, false)
+        ])
+    }
+
+    pub fn new_cyberplague() -> Self {
+        Self::from_json(&[
+            (0, 6, 6, false),
+            (1, 6, 6, true),
+            (2, 6, 6, false),
+            (3, 4, 9, true),
+            (4, 4, 14, false),
+            (5, 0, 17, true),
+            (6, 2, 12, false),
+            (7, 0, 18, true),
+            (8, 4, 14, false),
+            (9, 5, 9, true),
+            (10, 4, 7, false),
+            (11, 5, 6, true),
+            (12, 5, 6, false)
+        ])
+    }
+
     pub fn new_jar() -> Self {
         Self::from_json(&[
             (0, 0, 17, false),
@@ -145,6 +169,19 @@ impl<const STRIDE_NUM: usize> StrideMapping<STRIDE_NUM> {
         ])
     }
 
+    pub fn new_albus() -> Self {
+        Self::from_json(&[
+            (0, 0, 29, false),
+            (1, 0, 20, false),
+            (2, 0, 22, false),
+            (3, 0, 19, false),
+            (4, 0, 12, false),
+            (5, 0, 14, false),
+            (6, 0, 16, false),
+            (7, 0, 19, false),
+        ])
+    }
+
     pub fn from_json(stride_json: &[(u8, u8, u8, bool)]) -> Self {
         let mut strides = [Stride::default(); STRIDE_NUM];
         let stride_count = stride_json.len();
@@ -185,8 +222,6 @@ impl<const STRIDE_NUM: usize> StrideMapping<STRIDE_NUM> {
         let s = size.take().unwrap();
         log::info!("size={:?}", s);
 
-        log::info!("strides={:?}", strides);
-
         Self {
             strides,
             pixel_count: physical_idx,

src/platform/embedded_graphics_lib.rs

@@ -98,7 +98,7 @@ impl<T: LedPixelShape, S: Surface> Display<S> for EmbeddedDisplay<Ws2812DrawTarg
                 let mut pixel = RGB8::new(0, 0, 0);
                 for surface in self.surfaces.iter() {
                     surface.with_shader(|shader| {
-                        pixel = pixel.saturating_add(shader.draw(virtCoords.clone()));
+                        pixel = pixel.saturating_add(shader.draw(virtCoords));
                     })
                 }
                 self.total_mw += pixel.as_milliwatts();

src/platform/esp32.rs

@@ -1,312 +0,0 @@
-use core::borrow::BorrowMut;
-use std::sync::Arc;
-use std::fmt::Debug;
-use std::sync::Mutex;
-
-use chrono::DateTime;
-use chrono::Utc;
-
-use esp_idf_svc::eventloop::{EspSubscription, EspSystemEventLoop, System};
-use esp_idf_svc::hal::modem::Modem;
-use esp_idf_svc::hal::prelude::Peripherals;
-use esp_idf_svc::hal::task::thread::ThreadSpawnConfiguration;
-use esp_idf_svc::netif::IpEvent;
-use esp_idf_svc::nvs::{EspDefaultNvsPartition, EspNvsPartition, NvsDefault};
-use esp_idf_svc::sntp::EspSntp;
-use esp_idf_svc::sys::esp_efuse_mac_get_default;
-use esp_idf_svc::wifi::{AuthMethod, ClientConfiguration, Configuration, EspWifi, WifiEvent};
-use rgb::Rgb;
-
-use super::smart_leds_lib::rmt::FastWs2812Esp32Rmt;
-use super::smart_leds_lib::StrideOutput;
-use super::Board;
-
-use crate::buffers::BufferedSurfacePool;
-use crate::buffers::Pixbuf;
-use crate::mappings::StrideMapping;
-use crate::task::FixedSizeScheduler;
-use crate::task::Task;
-use crate::time::Periodically;
-
-pub mod i2s {
-    use esp_idf_svc::hal::i2s::*;
-    use rgb::ComponentBytes;
-    use rgb::Rgb;
-    
-    use crate::mappings::*;
-    use crate::buffers::Pixbuf;
-    use crate::render::Output;
-    use crate::render::Sample;
-
-    pub struct I2SOutput<'d> {
-        driver: I2sDriver<'d, I2sTx>,
-        pixbuf: [Rgb<u8>; 310],
-        pixmap: StrideMapping,
-    }
-
-    impl<'d> I2SOutput<'d> {
-        fn new(driver: I2sDriver<'d, I2sTx>) -> Self {
-            I2SOutput {
-                driver,
-                pixbuf: Pixbuf::new(),
-                pixmap: StrideMapping::new_jar()
-            }
-        }
-    }
-
-    impl<'d> Output for I2SOutput<'d> {
-        fn blank(&mut self) {
-            self.pixbuf.blank();
-        }
-
-        fn commit(&mut self) {
-            let bytes = self.pixbuf.as_bytes();
-            let mut written = self.driver.preload_data(bytes).unwrap();
-            self.driver.tx_enable().unwrap();
-            while written < bytes.len() {
-                let next = &bytes[written..];
-                written += self.driver.write(next, 0).unwrap();
-            }
-            self.driver.tx_disable().unwrap();
-        }
-    }
-
-    impl<'d> Sample for I2SOutput<'d> {
-        type Pixel = Rgb<u8>;
-        fn sample(&mut self, rect: &crate::geometry::Rectangle<crate::geometry::Virtual>) -> impl crate::render::PixelView<Pixel = Self::Pixel> {
-            StrideSampler::new(&mut self.pixbuf, self.pixmap.select(rect))
-        }
-    }
-}
-
-pub struct Esp32Board {
-    sys_loop: EspSystemEventLoop,
-    modem: Option<Modem>,
-    surfaces: BufferedSurfacePool,
-    output: Option<<Self as Board>::Output>
-}
-
-impl Board for Esp32Board {
-    type Output = StrideOutput<[Rgb<u8>; 310], FastWs2812Esp32Rmt<'static>>;
-    type Surfaces = BufferedSurfacePool;
-    type Scheduler = FixedSizeScheduler<2>;
-    
-    fn take() -> Self {
-        // It is necessary to call this function once. Otherwise some patches to the runtime
-        // implemented by esp-idf-sys might not link properly. See https://github.com/esp-rs/esp-idf-template/issues/71
-        esp_idf_svc::sys::link_patches();
-
-        // Bind the log crate to the ESP Logging facilities
-        esp_idf_svc::log::EspLogger::initialize_default();
-
-        let peripherals = Peripherals::take().unwrap();
-        let sys_loop = EspSystemEventLoop::take().unwrap();
-
-        let mut chip_id: [u8; 8] = [0; 8];
-        unsafe {
-            esp_efuse_mac_get_default(&mut chip_id as *mut u8);
-        }
-
-        log::info!("Setting up output for chip ID {:x?}", chip_id);
-
-        const POWER_VOLTS : u32 = 5;
-        const POWER_MA : u32 = 500;
-        const MAX_POWER_MW : u32 = POWER_VOLTS * POWER_MA;
-        let pins = peripherals.pins;
-
-        ThreadSpawnConfiguration {
-            pin_to_core: Some(esp_idf_svc::hal::cpu::Core::Core1),
-            ..Default::default()
-        }.set().unwrap();
-        // Wifi driver creates too many interrupts on core0, so we need to use RMT on core1.
-        // But the implementation spawns a thread based on the core the driver was created in,
-        // so we create the driver in another thread briefly.
-        // Fun stuff.
-        let output = match chip_id {
-            [72, 202, 67, 89, 145, 204, 0, 0] => {
-                StrideOutput::new(
-                    Pixbuf::new(),
-                    StrideMapping::new_panel(), 
-                    std::thread::spawn(move || { FastWs2812Esp32Rmt::new(peripherals.rmt.channel0, pins.gpio5).unwrap() }).join().unwrap(),
-                    MAX_POWER_MW
-                )
-            },
-            [0x8C, 0xAA, 0xB5, 0x83, 0x5f, 0x74, 0x0, 0x0] => {
-                StrideOutput::new(
-                    Pixbuf::new(),
-                    StrideMapping::new_jar(),
-                    std::thread::spawn(move || { FastWs2812Esp32Rmt::new(peripherals.rmt.channel0, pins.gpio14).unwrap() }).join().unwrap(),
-                    MAX_POWER_MW
-                )
-            },
-            _ => {
-                StrideOutput::new(
-                    Pixbuf::new(),
-                    StrideMapping::new(), 
-                    std::thread::spawn(move || { FastWs2812Esp32Rmt::new(peripherals.rmt.channel0, pins.gpio5).unwrap() }).join().unwrap(),
-                    MAX_POWER_MW
-                )
-            }
-        };
-        ThreadSpawnConfiguration {
-            ..Default::default()
-        }.set().unwrap();
-
-        Esp32Board {
-            modem: Some(peripherals.modem),
-            sys_loop: sys_loop.clone(),
-            surfaces: BufferedSurfacePool::new(),
-            output: Some(output)
-        }
-    }
-
-    fn output(&mut self) -> Self::Output {
-        self.output.take().unwrap()
-    }
-
-    fn surfaces(&mut self) -> Self::Surfaces {
-        self.surfaces.clone()
-    }
-
-    fn system_tasks(&mut self) -> Self::Scheduler {
-        let nvs = EspDefaultNvsPartition::take().unwrap();
-        FixedSizeScheduler::new([
-            Box::new(WifiTask::new(self.modem.take().unwrap(), self.sys_loop.clone(), &nvs)),
-            Box::new(self.surfaces.clone())
-        ])
-    }
-}
-
-impl Debug for WifiTask {
-    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-        f.debug_struct("WifiTask").finish()
-    }
-}
-
-#[derive(Debug, PartialEq, Clone, Copy)]
-enum WifiState {
-    Stopped,
-    Disconnected,
-    Connecting,
-    Connected
-}
-
-struct WifiTask {
-    wifi: EspWifi<'static>,
-    ntp: EspSntp<'static>,
-    connection_check: Periodically,
-    state: Arc<Mutex<WifiState>>,
-    last_state: WifiState,
-    sys_loop: EspSystemEventLoop,
-    wifi_sub: Option<EspSubscription<'static, System>>,
-    ip_sub: Option<EspSubscription<'static, System>>
-}
-
-impl WifiTask {
-    fn new(modem: Modem, sys_loop: EspSystemEventLoop, nvs: &EspNvsPartition<NvsDefault>) -> Self {
-        log::info!("Installing wifi driver");
-        let mut wifi = EspWifi::new(
-            modem,
-            sys_loop.clone(),
-            Some(nvs.clone())
-        ).unwrap();
-        let wifi_config = Configuration::Client(ClientConfiguration {
-            ssid: "The Frequency".try_into().unwrap(),
-            bssid: None,
-            auth_method: AuthMethod::WPA2Personal,
-            password: "thepasswordkenneth".try_into().unwrap(),
-            channel: None,
-            ..Default::default()
-        });
-        wifi.set_configuration(&wifi_config).unwrap();
-
-        WifiTask {
-            wifi,
-            ntp: EspSntp::new_default().unwrap(),
-            connection_check: Periodically::new_every_n_seconds(1),
-            state: Arc::new(Mutex::new(WifiState::Stopped)),
-            last_state: WifiState::Stopped,
-            sys_loop,
-            wifi_sub: None,
-            ip_sub: None
-        }
-    }
-
-    fn connect(&mut self) {
-        log::info!("Connecting wifi");
-        self.wifi.start().unwrap();
-        self.wifi.connect().unwrap();
-    }
-
-    fn disconnect(&mut self) {
-        log::info!("Disconnecting wifi");
-        self.wifi.disconnect().unwrap();
-        self.wifi.stop().unwrap();
-    }
-}
-
-impl Task for WifiTask {
-    fn start(&mut self) {
-        log::info!("Starting wifi!");
-
-        let wifi_state = self.state.clone();
-        self.wifi_sub = Some(self.sys_loop.subscribe::<WifiEvent, _>( move |evt| {
-            log::warn!("wifi event {:?}", evt);
-            let next_state = match evt {
-                WifiEvent::StaStopped => Some(WifiState::Disconnected),
-                WifiEvent::StaDisconnected => Some(WifiState::Disconnected),
-                WifiEvent::StaStarted => Some(WifiState::Connecting),
-                WifiEvent::StaConnected => Some(WifiState::Connecting),
-                _ => None
-            };
-
-            match next_state {
-                Some(s) => {
-                    let mut state = wifi_state.lock().unwrap();
-                    *state = s
-                },
-                None => ()
-            }
-        }).unwrap());
-        let ip_state = self.state.clone();
-        self.ip_sub = Some(self.sys_loop.subscribe::<IpEvent, _>(move |evt| {
-            log::warn!("ip event {:?}", evt);
-            match evt {
-                IpEvent::DhcpIpAssigned(_) => {
-                    let mut state = ip_state.lock().unwrap();
-                    *state = WifiState::Connected;
-                },
-                _ => ()
-            }
-        }).unwrap());
-
-        self.connect();
-    }
-
-    fn tick(&mut self ) {
-        if self.connection_check.tick() {
-            let cur_state = *self.state.lock().unwrap();
-
-            if self.last_state != cur_state {
-                match cur_state {
-                    WifiState::Connected => log::info!("Wifi connected!"),
-                    WifiState::Connecting => log::info!("Connecting!"),
-                    WifiState::Stopped => log::info!("Stopped!"),
-                    WifiState::Disconnected => log::info!("Disconnected!")
-                }
-
-                log::info!("online: {:?}", cur_state);
-
-                self.last_state = cur_state;
-            }
-
-            let now: DateTime<Utc> = std::time::SystemTime::now().into();
-            log::info!("Current time: {} status={:?}", now.format("%d/%m/%Y %T"), self.ntp.get_sync_status());
-        }
-    }
-
-    fn stop(&mut self) {
-        log::info!("Stopping wifi");
-        self.disconnect();
-    }
-}

src/platform/esp32/board.rs

@@ -0,0 +1,143 @@
+use esp_idf_svc::{eventloop::EspSystemEventLoop, hal::{gpio::Pins, modem::Modem, prelude::Peripherals, rmt::RMT, task::thread::ThreadSpawnConfiguration}, nvs::EspDefaultNvsPartition, sys::esp_efuse_mac_get_default};
+use rgb::Rgb;
+
+use crate::{buffers::{BufferedSurfacePool, Pixbuf}, mappings::StrideMapping, platform::{smart_leds_lib::{rmt::FastWs2812Esp32Rmt, StrideOutput}, Board}, task::FixedSizeScheduler};
+
+use super::{mqtt::MqttTask, wifi::WifiTask};
+
+pub struct Esp32Board {
+    sys_loop: EspSystemEventLoop,
+    modem: Option<Modem>,
+    pins: Option<Pins>,
+    rmt: Option<RMT>,
+    surfaces: Option<BufferedSurfacePool>,
+}
+
+impl Board for Esp32Board {
+    type Output = StrideOutput<[Rgb<u8>; 310], FastWs2812Esp32Rmt<'static>>;
+    type Surfaces = BufferedSurfacePool;
+    type Scheduler = FixedSizeScheduler<2>;
+
+    fn chip_id() -> u64 {
+        let mut chip_id: [u8; 8] = [0; 8];
+        unsafe {
+            esp_efuse_mac_get_default(&mut chip_id as *mut u8);
+        }
+        return u64::from_be_bytes(chip_id);
+    }
+
+    fn take() -> Self {
+        // It is necessary to call this function once. Otherwise some patches to the runtime
+        // implemented by esp-idf-sys might not link properly. See https://github.com/esp-rs/esp-idf-template/issues/71
+        esp_idf_svc::sys::link_patches();
+
+        // Bind the log crate to the ESP Logging facilities
+        esp_idf_svc::log::EspLogger::initialize_default();
+
+        let peripherals = Peripherals::take().unwrap();
+        let sys_loop = EspSystemEventLoop::take().unwrap();
+
+        Esp32Board {
+            modem: Some(peripherals.modem),
+            sys_loop: sys_loop,
+            surfaces: Some(BufferedSurfacePool::new()),
+            pins: Some(peripherals.pins),
+            rmt: Some(peripherals.rmt)
+        }
+    }
+
+    fn output(&mut self) -> Self::Output {
+
+        log::info!("Setting up output for chip ID {:x?}", Self::chip_id());
+
+        const POWER_VOLTS : u32 = 5;
+        const POWER_MA : u32 = 500;
+        const MAX_POWER_MW : u32 = POWER_VOLTS * POWER_MA;
+
+        let pins = self.pins.take().unwrap();
+        let rmt = self.rmt.take().unwrap();
+        ThreadSpawnConfiguration {
+            pin_to_core: Some(esp_idf_svc::hal::cpu::Core::Core1),
+            ..Default::default()
+        }.set().unwrap();
+        // Wifi driver creates too many interrupts on core0, so we need to use RMT on core1.
+        // But the implementation spawns a thread based on the core the driver was created in,
+        // so we create the driver in another thread briefly.
+        // Fun stuff.
+        let output = match Self::chip_id().to_be_bytes() { // panel test board
+            [72, 202, 67, 89, 145, 204, 0, 0] => {
+                StrideOutput::new(
+                    Pixbuf::new(),
+                    StrideMapping::new_panel(), 
+                    std::thread::spawn(move || { FastWs2812Esp32Rmt::new(rmt.channel0, pins.gpio5).unwrap() }).join().unwrap(),
+                    MAX_POWER_MW
+                )
+            },
+            [0x8C, 0xAA, 0xB5, 0x83, 0x5f, 0x74, 0x0, 0x0] => { //ponderjar
+                StrideOutput::new(
+                    Pixbuf::new(),
+                    StrideMapping::new_jar(),
+                    std::thread::spawn(move || { FastWs2812Esp32Rmt::new(rmt.channel0, pins.gpio14).unwrap() }).join().unwrap(),
+                    MAX_POWER_MW
+                )
+            },
+            [0x4a, 0xca, 0x43, 0x59, 0x85, 0x58, 0x0, 0x0] => { // Albus the tree
+                StrideOutput::new(
+                    Pixbuf::new(),
+                    StrideMapping::new_albus(),
+                    std::thread::spawn(move || { FastWs2812Esp32Rmt::new(rmt.channel0, pins.gpio5).unwrap() }).join().unwrap(),
+                    POWER_VOLTS * 2_400
+                )
+            },
+            [0x48, 0xca, 0x43, 0x59, 0x9d, 0x48, 0x0, 0x0] => { // kitchen cabinets
+                StrideOutput::new(
+                    Pixbuf::new(),
+                    StrideMapping::new_fairylights(),
+                    std::thread::spawn(move || { FastWs2812Esp32Rmt::new(rmt.channel0, pins.gpio5).unwrap() }).join().unwrap(),
+                    MAX_POWER_MW
+                )
+            },
+            [0x48, 0xca, 0x43, 0x59, 0x9e, 0xdc, 0x0, 0x0] => { // front window
+                StrideOutput::new(
+                    Pixbuf::new(),
+                    StrideMapping::new_fairylights(),
+                    std::thread::spawn(move || { FastWs2812Esp32Rmt::new(rmt.channel0, pins.gpio5).unwrap() }).join().unwrap(),
+                    MAX_POWER_MW
+                )
+            },
+            [0xfc, 0xf5, 0xc4, 0x05, 0xb8, 0x30, 0x0, 0x0] => { // cyberplague
+                StrideOutput::new(
+                    Pixbuf::new(),
+                    StrideMapping::new_cyberplague(),
+                    std::thread::spawn(move || { FastWs2812Esp32Rmt::new(rmt.channel0, pins.gpio13).unwrap() }).join().unwrap(),
+                    MAX_POWER_MW
+                )
+            },
+            _ => {
+                StrideOutput::new(
+                    Pixbuf::new(),
+                    StrideMapping::new(), 
+                    std::thread::spawn(move || { FastWs2812Esp32Rmt::new(rmt.channel0, pins.gpio5).unwrap() }).join().unwrap(),
+                    MAX_POWER_MW
+                )
+            }
+        };
+        ThreadSpawnConfiguration {
+            ..Default::default()
+        }.set().unwrap();
+
+        output
+    }
+
+    fn surfaces(&mut self) -> Self::Surfaces {
+        self.surfaces.take().unwrap()
+    }
+
+    fn system_tasks(&mut self) -> Self::Scheduler {
+        let nvs = EspDefaultNvsPartition::take().unwrap();
+        FixedSizeScheduler::new([
+            Box::new(WifiTask::new(self.modem.take().unwrap(), self.sys_loop.clone(), &nvs)),
+            Box::new(MqttTask::new())
+        ])
+    }
+}

src/platform/esp32/mod.rs

@@ -0,0 +1,3 @@
+pub mod board;
+pub mod wifi;
+pub mod mqtt;

src/platform/esp32/mqtt.rs

@@ -0,0 +1,183 @@
+use core::str;
+use std::{borrow::BorrowMut, collections::{HashMap, LinkedList}, thread::JoinHandle};
+
+use esp_idf_svc::mqtt::client::{EspMqttClient, EventPayload, MqttClientConfiguration};
+use log::Log;
+use serde_json::{json, Value};
+
+use crate::{events::{Event, EventBus, System as SystemNS}, properties::{NamespaceKey, PropertyID, PropertyKey, Variant}, render::props::Output as OutputNS, scenes::props::Scenes as SceneNS, task::{Environment, Task}};
+use crate::platform::props::Board as BoardNS;
+
+struct HADevice {
+    device_id: String,
+    scenes: Vec<String>
+}
+
+impl HADevice {
+    fn new(device_id: String, scenes: Vec<String>) -> Self {
+        HADevice {
+            device_id,
+            scenes
+        }
+    }
+
+    fn topic(&self, key: &str) -> String {
+        format!("renderbug/{}/{}", self.device_id, key)
+    }
+
+    fn command_topic(&self) -> String {
+        self.topic("command")
+    }
+
+    fn state_topic(&self) -> String {
+        self.topic("state")
+    }
+
+    fn device_config_topic(&self) -> String {
+        format!("homeassistant/device/renderbug-rs-{}/config", self.device_id)
+    }
+
+    fn registration(&self) -> Value {
+        let components: HashMap<String, Value> = self.scenes.iter().map(|f| {
+            let scene_id = format!("renderbug_{}_scene_{}", self.device_id, f);
+            (scene_id.clone(), json!({
+                "p": "scene",
+                "unique_id": scene_id,
+                "payload_on": json!({"namespace": SceneNS::Namespace, "key": SceneNS::Current.key, "value": f}).to_string(),
+                "name": f
+            }))
+        }).collect();
+        json!({
+            "stat_t": self.state_topic(),
+            "cmd_t": self.command_topic(),
+            "dev": {
+                "name": format!("Renderbug-rs ESP32 {}", self.device_id),
+                "mdl": "Renderbug-rs ESP32",
+                "mf": "Phong Robotics",
+                "identifiers": [self.device_id]
+            },
+            "origin": {
+                "name": "renderbug-rs"
+            },
+            "cmps": components
+        })
+    }
+}
+
+pub struct MqttTask {
+    client: Option<EspMqttClient<'static>>,
+    ha_device: Option<HADevice>
+}
+
+impl MqttTask {
+    pub fn new() -> Self {
+        MqttTask {
+            client: None,
+            ha_device: None
+        }
+    }
+
+    fn start_mqtt(&mut self, env: &mut Environment) {
+        log::info!("Starting MQTT");
+        let chip_id: u64 = env.get_property(BoardNS::ChipID).unwrap();
+
+        let mut client_bus = env.bus.clone();
+        self.client = Some(EspMqttClient::new_cb(
+            "mqtt://10.0.0.2:1883",
+            &MqttClientConfiguration {
+                client_id: Some(&format!("{:X}", chip_id)),
+                ..Default::default()
+            },
+            move |evt| {
+                match evt.payload() {
+                    EventPayload::Received { id, topic, data, details } => {
+                        log::info!("got event id={} topic={:?} data={:?} details={:?}", id, topic, str::from_utf8(data), details);
+                        let props: HashMap<String, Variant> = serde_json::from_slice(data).unwrap();
+                        let ns: String = props.get("namespace").unwrap().to_owned().into();
+                        let key: String = props.get("key").unwrap().to_owned().into();
+                        let value: Variant = props.get("value").unwrap().clone();
+                        let all = client_bus.properties().all_props();
+                        let prop_id = all.iter().find(|f| {
+                            f.key.0 == key && f.namespace.0 == ns
+                        });
+
+                        client_bus.set_property(SystemNS::Idle, false);
+
+                        match prop_id {
+                            None => log::warn!("Could not find property {} in {}!", key, ns),
+                            Some(id) => client_bus.set_property(id.clone(), value),
+                        }
+                    },
+                    EventPayload::Connected(_) => {
+                        log::info!("MQTT is online!");
+                        client_bus.set_property(props::MQTT::Online, true);
+                    },
+                    EventPayload::Disconnected => {
+                        log::info!("MQTT is offline!");
+                        client_bus.set_property(props::MQTT::Online, false);
+                    },
+                    _ => ()
+                }
+            }
+        ).unwrap());
+        log::info!("Connected!");
+    }
+}
+
+impl Task for MqttTask {
+    fn start(&mut self, bus: &mut EventBus) {
+        bus.set_property(props::MQTT::Online, false);
+    }
+
+    fn on_ready(&mut self, bus: &mut EventBus) {
+        let chip_id: u64 = bus.get_property(BoardNS::ChipID).unwrap();
+        let scenes: Vec<String> = bus.get_property(SceneNS::All).unwrap();
+        log::info!("Setting up scenes: {:?}", scenes);
+        self.ha_device = Some(HADevice::new(format!("{:X}", chip_id), scenes));
+    }
+
+    fn on_property_change(&mut self, key: PropertyID, value: &Variant, bus: &mut EventBus) {
+        match (key, value) {
+            (SystemNS::NetworkOnline, Variant::Boolean(true)) => {
+                log::info!("Registering with MQTT");
+                self.start_mqtt(bus);
+            },
+            (props::MQTT::Online, Variant::Boolean(true)) => {
+                if let Some(ref mut client) = self.client {
+                    if let Some(ref device) = self.ha_device {
+                        client.enqueue(
+                            &device.device_config_topic(),
+                            esp_idf_svc::mqtt::client::QoS::AtLeastOnce,
+                            false,
+                            device.registration().to_string().as_bytes()
+                        ).unwrap();
+                        log::info!("Subscribing to {}", device.command_topic());
+                        client.subscribe(&device.command_topic(), esp_idf_svc::mqtt::client::QoS::AtLeastOnce).unwrap();
+                    }
+                }
+            }
+            /*(prop_id!(OutputNS::FPS), Variant::UInt(fps)) => {
+                if let Some(ref mut client) = self.client {
+                    if let Some(ref sensor) = self.ha_device {
+                        client.enqueue(
+                            &sensor.topic("state"),
+                            esp_idf_svc::mqtt::client::QoS::AtLeastOnce,
+                            false,
+                            json!(fps).to_string().as_bytes()
+                        ).unwrap();
+                    }
+                }
+            },*/
+            _ => ()
+        }
+    }
+}
+
+pub mod props {
+    use crate::property_namespace;
+
+    property_namespace!(
+        MQTT,
+        Online => true
+    );
+}

src/platform/esp32/wifi.rs

@@ -0,0 +1,104 @@
+use esp_idf_svc::{eventloop::{EspSubscription, EspSystemEventLoop, System}, hal::modem::Modem, netif::IpEvent, nvs::{EspNvsPartition, NvsDefault}, sntp::{EspSntp, SyncStatus}, wifi::{AuthMethod, ClientConfiguration, Configuration, EspWifi, WifiEvent}};
+
+use crate::{events::{EventBus, System as SystemNS}, properties::Variant, task::{Environment, Task}, time::Periodically};
+
+use std::fmt::Debug;
+
+pub struct WifiTask {
+    wifi: EspWifi<'static>,
+    ntp: EspSntp<'static>,
+    connection_check: Periodically,
+    sys_loop: EspSystemEventLoop,
+    wifi_sub: Option<EspSubscription<'static, System>>,
+    ip_sub: Option<EspSubscription<'static, System>>,
+}
+
+impl Debug for WifiTask {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        f.debug_struct("WifiTask").finish()
+    }
+}
+
+impl WifiTask {
+    pub fn new(modem: Modem, sys_loop: EspSystemEventLoop, nvs: &EspNvsPartition<NvsDefault>) -> Self {
+        log::info!("Installing wifi driver");
+        let wifi = EspWifi::new(
+            modem,
+            sys_loop.clone(),
+            Some(nvs.clone())
+        ).unwrap();
+
+        WifiTask {
+            wifi,
+            ntp: EspSntp::new_default().unwrap(),
+            connection_check: Periodically::new_every_n_seconds(1),
+            sys_loop,
+            wifi_sub: None,
+            ip_sub: None,
+        }
+    }
+
+    fn connect(&mut self) {
+        log::info!("Connecting wifi");
+        let wifi_config = Configuration::Client(ClientConfiguration {
+            ssid: "The Frequency".try_into().unwrap(),
+            bssid: None,
+            auth_method: AuthMethod::WPA2Personal,
+            password: "thepasswordkenneth".try_into().unwrap(),
+            channel: None,
+            ..Default::default()
+        });
+        self.wifi.set_configuration(&wifi_config).unwrap();
+        self.wifi.start().unwrap();
+        self.wifi.connect().unwrap();
+    }
+
+    fn disconnect(&mut self) {
+        log::info!("Disconnecting wifi");
+        self.wifi.disconnect().unwrap();
+        self.wifi.stop().unwrap();
+    }
+}
+
+impl Task for WifiTask {
+    fn start(&mut self, bus: &mut EventBus) {
+        log::info!("Starting wifi!");
+        let mut wifi_bus = bus.clone();
+        self.wifi_sub = Some(self.sys_loop.subscribe::<WifiEvent, _>( move |evt| {
+            log::debug!("wifi event {:?}", evt);
+            wifi_bus.set_property(SystemNS::NetworkOnline,false);
+        }).unwrap());
+        let mut ip_bus = bus.clone();
+        self.ip_sub = Some(self.sys_loop.subscribe::<IpEvent, _>(move |evt| {
+            log::debug!("ip event {:?}", evt);
+            match evt {
+                IpEvent::DhcpIpAssigned(addr) => {
+                    ip_bus.set_property(SystemNS::IP, addr.ip().to_string());
+                    ip_bus.set_property(SystemNS::Gateway, addr.gateway().to_string());
+                    ip_bus.set_property(SystemNS::NetworkOnline, true);
+                },
+                _ => ()
+            }
+        }).unwrap());
+        self.connect();
+    }
+
+    fn on_tick(&mut self, env: &mut Environment) {
+        if self.connection_check.tick() {
+            if env.get_property(SystemNS::NetworkOnline).unwrap() {
+                match self.ntp.get_sync_status() {
+                    SyncStatus::Completed => env.set_property(SystemNS::TimeSync, true),
+                    _ => env.set_property(SystemNS::TimeSync, false)
+                }
+            }
+        }
+    }
+
+    fn stop(&mut self, env: &mut Environment) {
+        log::info!("Stopping wifi");
+        self.wifi_sub.take().unwrap();
+        self.ip_sub.take().unwrap();
+        self.disconnect();
+        env.set_property(SystemNS::NetworkOnline, false);
+    }
+}

src/platform/mod.rs

@@ -6,18 +6,28 @@ pub mod smart_leds_lib;
 
 pub mod esp32;
 
-pub type DefaultBoard = esp32::Esp32Board;
+pub type DefaultBoard = esp32::board::Esp32Board;
 
 use crate::render::{Output, Surfaces};
-use crate::task::Scheduler;
+use crate::events::Evented;
 
 pub trait Board {
     type Output: Output;
     type Surfaces: Surfaces;
-    type Scheduler: Scheduler;
+    type Scheduler: Evented;
 
     fn take() -> Self;
     fn output(&mut self) -> Self::Output;
     fn surfaces(&mut self) -> Self::Surfaces;
     fn system_tasks(&mut self) -> Self::Scheduler;
+    fn chip_id() -> u64;
+}
+
+pub mod props {
+    use crate::property_namespace;
+
+    property_namespace!(
+        Board,
+        ChipID => 0_u64
+    );
 }

src/platform/smart_leds_lib.rs

@@ -1,7 +1,8 @@
 use smart_leds_trait::SmartLedsWrite;
 
 use crate::buffers::Pixbuf;
-use crate::render::{HardwarePixel, Output, PixelView, Sample};
+use crate::properties::Variant;
+use crate::render::{HardwarePixel, Output, PixelView, Sample, props::Output as OutputNS};
 use crate::power::brightness_for_mw;
 use crate::geometry::*;
 use crate::mappings::*;
@@ -18,7 +19,8 @@ pub struct StrideOutput<P: Pixbuf, T: FastWrite> {
     pixbuf: P,
     stride_map: StrideMapping,
     target: T,
-    max_mw: u32
+    max_mw: u32,
+    brightness: u8
 }
 
 impl<P: Pixbuf, T: FastWrite> StrideOutput<P, T> {
@@ -28,7 +30,8 @@ impl<P: Pixbuf, T: FastWrite> StrideOutput<P, T> {
             pixbuf,
             stride_map,
             target,
-            max_mw
+            max_mw,
+            brightness: 255
         }
     }
 }
@@ -46,11 +49,18 @@ impl<P: Pixbuf<Pixel=T::Color>, T: FastWrite> Output for StrideOutput<P, T> {
     }
 
     fn commit(&mut self) {
-        let b = brightness_for_mw(self.pixbuf.as_milliwatts(), 255, self.max_mw);
+        let b = brightness_for_mw(self.pixbuf.as_milliwatts(), self.brightness, self.max_mw);
         if self.target.fast_write(self.pixbuf.iter_with_brightness(b)).is_err() {
             panic!("Could not write frame!");
         };
     }
+
+    fn on_event(&mut self, event: &crate::events::Event) {
+        match event {
+            crate::events::Event::PropertyChange(OutputNS::Brightness, Variant::Byte(new_brightness)) => self.brightness = *new_brightness,
+            _ => ()
+        }
+    }
 }
 
 pub trait FastWrite: Send {

src/properties.rs

@@ -0,0 +1,328 @@
+use std::fmt::Display;
+
+use rgb::Rgb;
+use serde::{de::Visitor, ser::{SerializeSeq, SerializeTuple}, Deserialize, Serialize};
+
+#[derive(Debug, Clone, PartialEq, Eq)]
+pub enum Variant {
+    SignedByte(i8),
+    Byte(u8),
+    UInt(u32),
+    Int(i32),
+    BigUInt(u64),
+    BigInt(i64),
+    Boolean(bool),
+    String(String),
+    RGB(Rgb<u8>),
+    Vec(Vec<Variant>)
+}
+
+impl Serialize for Variant {
+    fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
+        where
+            S: serde::Serializer {
+        match self {
+            Variant::SignedByte(b) => serializer.serialize_i8(*b),
+            Variant::Byte(b) => serializer.serialize_u8(*b),
+            Variant::UInt(b) => serializer.serialize_u32(*b),
+            Variant::Int(b) => serializer.serialize_i32(*b),
+            Variant::BigUInt(b) => serializer.serialize_u64(*b),
+            Variant::BigInt(b) => serializer.serialize_i64(*b),
+            Variant::Boolean(b) => serializer.serialize_bool(*b),
+            Variant::String(b) => serializer.serialize_str(&b),
+            Variant::RGB(rgb) => {
+                let mut seq = serializer.serialize_tuple(3).unwrap();
+                seq.serialize_element(&rgb.r).unwrap();
+                seq.serialize_element(&rgb.g).unwrap();
+                seq.serialize_element(&rgb.b).unwrap();
+                seq.end()
+            },
+            Variant::Vec(v) => {
+                let mut seq = serializer.serialize_seq(Some(v.len())).unwrap();
+                for i in v {
+                    seq.serialize_element(i).unwrap();
+                }
+                seq.end()
+            }
+        }
+    }
+}
+
+struct VariantVisitor {}
+
+impl<'de> Visitor<'de> for VariantVisitor {
+    type Value = Variant;
+
+    fn expecting(&self, formatter: &mut std::fmt::Formatter) -> std::fmt::Result {
+        formatter.write_str("variant")
+    }
+
+    fn visit_bool<E>(self, v: bool) -> Result<Self::Value, E>
+        where
+            E: serde::de::Error, {
+        Ok(Variant::Boolean(v))
+    }
+
+    fn visit_str<E>(self, v: &str) -> Result<Self::Value, E>
+        where
+            E: serde::de::Error, {
+        Ok(Variant::String(v.into()))
+    }
+
+    fn visit_string<E>(self, v: String) -> Result<Self::Value, E>
+        where
+            E: serde::de::Error, {
+        Ok(Variant::String(v))
+    }
+}
+
+impl<'de> Deserialize<'de> for Variant {
+    fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
+        where
+            D: serde::Deserializer<'de> {
+        deserializer.deserialize_any(VariantVisitor {})
+    }
+}
+
+macro_rules! impl_variant_type {
+    ($type:ty, $var_type:tt) => {
+        impl From<$type> for Variant {
+            fn from(value: $type) -> Self {
+                Variant::$var_type(value)
+            }
+        }
+
+        impl Into<$type> for Variant {
+            fn into(self) -> $type {
+                match self {
+                    Variant::$var_type(value) => value,
+                    _ => panic!(concat!("Expected Variant::", stringify!($var_type), "but got {:?}"), self)
+                }
+            }
+        }
+
+        impl Into<Vec<$type>> for Variant {
+            fn into(self) -> Vec<$type> {
+                match self {
+                    Variant::Vec(v) => {
+                        v.into_iter().map(|f| {
+                            f.into()
+                        }).collect()
+                    },
+                    _ => panic!(concat!("Expected Variant::Vec(", stringify!($var_type), "but got {:?}"), self)
+                }
+            }
+        }
+    };
+}
+
+impl_variant_type!(u8, Byte);
+impl_variant_type!(i8, SignedByte);
+impl_variant_type!(u32, UInt);
+impl_variant_type!(i32, Int);
+impl_variant_type!(i64, BigInt);
+impl_variant_type!(u64, BigUInt);
+impl_variant_type!(bool, Boolean);
+impl_variant_type!(String, String);
+impl_variant_type!(Rgb<u8>, RGB);
+impl_variant_type!(Vec<Variant>, Vec);
+
+impl<'a> From<&'a str> for Variant {
+    fn from(value: &'a str) -> Self {
+        Variant::String(value.to_owned())
+    }
+}
+
+#[derive(Debug, Clone, PartialEq, Eq, Serialize)]
+pub struct Property {
+    key: PropertyKey,
+    value: Variant
+}
+
+impl Property {
+    pub const fn new(key: PropertyKey, value: Variant) -> Self {
+        Property {
+            key,
+            value,
+        }
+    }
+}
+
+#[derive(Debug, Clone, Copy, Eq, PartialEq, Serialize, Deserialize)]
+pub struct NamespaceKey(pub &'static str);
+
+#[derive(Debug, Clone, Copy, Eq, PartialEq, Serialize, Deserialize)]
+pub struct PropertyKey(pub &'static str);
+
+#[derive(Debug, Clone, Copy, Eq, PartialEq, Serialize)]
+pub struct PropertyID {
+    pub namespace: NamespaceKey,
+    pub key: PropertyKey
+}
+
+impl<'a> Display for PropertyID {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        write!(f, "{}::{}", self.namespace.0, self.key.0)
+    }
+}
+
+pub trait Namespace {
+    fn nskey() -> NamespaceKey;
+    fn properties() -> Vec<Property>;
+}
+
+#[macro_export]
+macro_rules! property_namespace {
+    ($name:ident, $($prop_name:ident => $default_value:expr),*) => {
+        pub struct $name {}
+
+        impl $name {
+            #[allow(non_upper_case_globals)]
+            pub const Namespace: $crate::properties::NamespaceKey = $crate::properties::NamespaceKey(stringify!($name));
+            $(
+                #[allow(non_upper_case_globals)]
+                pub const $prop_name: $crate::properties::PropertyID = $crate::properties::PropertyID {
+                    namespace: Self::Namespace,
+                    key: $crate::properties::PropertyKey(stringify!($prop_name))
+                };
+            )*
+        }
+        
+        impl crate::properties::Namespace for $name {
+            fn nskey() -> $crate::properties::NamespaceKey {
+                Self::Namespace
+            }
+
+            fn properties() -> Vec<$crate::properties::Property> {
+                vec![
+                    $($crate::properties::Property::new(Self::$prop_name.key, $crate::properties::Variant::from($default_value)),)*
+                ]
+            }
+        }
+    };
+}
+
+#[derive(Debug, Clone)]
+pub struct NamespaceProps {
+    props: Vec<Property>,
+    key: NamespaceKey
+}
+
+impl NamespaceProps {
+    fn new<NS: Namespace>() -> Self {
+        NamespaceProps {
+            props: NS::properties(),
+            key: NS::nskey()
+        }
+    }
+
+    fn get_key(&self, key: &PropertyKey) -> Option<&Property> {
+        for next in self.props.iter() {
+            if next.key == *key {
+                return Some(next);
+            }
+        }
+
+        log::warn!("Unknown key {:?}", key);
+
+        return None;
+    }
+
+    fn get_key_mut(&mut self, key: &PropertyKey) -> Option<&mut Property> {
+        for next in self.props.iter_mut() {
+            if next.key == *key {
+                return Some(next);
+            }
+        }
+
+        log::warn!("Unknown key {:?}", key);
+
+        return None;
+    }
+
+}
+
+#[derive(Debug, Clone)]
+pub struct Properties {
+    contents: Vec<NamespaceProps>
+}
+
+impl Display for Properties {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        for ns in self.contents.iter() {
+            write!(f, "{}\n", ns.key.0).unwrap();
+            for prop in ns.props.iter() {
+                write!(f, "\t{} = {:?}\n", prop.key.0, prop.value).unwrap();
+            }
+        }
+        Result::Ok(())
+    }
+}
+
+impl Properties {
+    pub fn new() -> Self {
+        Properties {
+            contents: Vec::new(),
+        }
+    }
+
+    pub fn add_namespace<T: Namespace>(&mut self) {
+        self.contents.push(NamespaceProps::new::<T>());
+    }
+
+    fn get_namespace(&self, ns: &NamespaceKey) -> Option<&NamespaceProps> {
+        for nsprops in self.contents.iter() {
+            if nsprops.key == *ns {
+                return Some(nsprops)
+            }
+        }
+
+        log::warn!("Unknown namespace {:?}", ns);
+
+        None
+    }
+
+    fn get_namespace_mut(&mut self, ns: &NamespaceKey) -> Option<&mut NamespaceProps> {
+        for nsprops in self.contents.iter_mut() {
+            if nsprops.key == *ns {
+                return Some(nsprops)
+            }
+        }
+
+        log::warn!("Unknown namespace {:?}", ns);
+
+        None
+    }
+
+    pub fn get<T: Into<PropertyID>>(&self, key: T) -> Option<Variant> {
+        let as_id = key.into();
+        match self.get_namespace(&as_id.namespace).unwrap().get_key(&as_id.key) {
+            None => None,
+            Some(v) => Some(v.value.clone())
+        }
+    }
+
+    pub fn set<V, T: Into<PropertyID>>(&mut self, key: T, value: V) -> bool where Variant: From<V> {
+        let as_id = key.into();
+        let as_variant: Variant = value.into();
+        match self.get_namespace_mut(&as_id.namespace).unwrap().get_key_mut(&as_id.key) {
+            None => (),
+            Some(found_key) => {
+                if found_key.value != as_variant {
+                    found_key.value = as_variant;
+                    return true
+                }
+            }
+        }
+
+        return false
+    }
+
+    pub fn all_props(&self) -> Vec<PropertyID> {
+        self.contents.iter().map(|f| {
+            f.props.iter().map(|k| {
+                PropertyID { namespace: f.key, key: k.key }
+            })
+        }).flatten().collect()
+    }
+}

src/render.rs

@@ -1,5 +1,6 @@
 use rgb::Rgb;
-
+use crate::events::*;
+use crate::properties::*;
 use crate::geometry::*;
 use crate::lib8::interpolate::Fract8Ops;
 use crate::power::AsMilliwatts;
@@ -22,26 +23,27 @@ pub trait Sample {
     fn sample(&mut self, rect: &Rectangle<Virtual>) -> impl PixelView<Pixel = Self::Pixel>;
 }
 
-pub trait Shader: Send + Sync {
+pub trait Shader: Send {
     fn draw(&self, surface_coords: &VirtualCoordinates, frame: usize) -> Rgb<u8>;
 }
 
-pub trait Surfaces: Send + Sync {
+pub trait Surfaces: Send {
     type Surface: Surface;
     type Error: Debug;
-    fn new_surface(&mut self, area: &Rectangle<Virtual>) -> Result<Self::Surface, Self::Error>;
+    fn new_surface(&mut self, area: Rectangle<Virtual>) -> Result<Self::Surface, Self::Error>;
     fn render_to<S: Sample>(&self, output: &mut S, frame: usize);
 }
 
-pub trait Surface: Send + Sync {
-    fn set_shader(&mut self, shader: Box<dyn Shader>);
+pub trait Surface: Send {
+    fn set_shader<T: Shader + 'static>(&mut self, shader: T);
     fn clear_shader(&mut self);
 
-    fn set_rect(&mut self, rect: &Rectangle<Virtual>);
+    fn set_rect(&mut self, rect: Rectangle<Virtual>);
     fn set_opacity(&mut self, opacity: u8);
 }
 
 pub trait Output: Sample + Send {
+    fn on_event(&mut self, event: &Event);
     fn blank(&mut self);
     fn commit(&mut self);
 }
@@ -52,7 +54,8 @@ pub struct Renderer<T: Output, S: Surfaces> {
     surfaces: S,
     fps: RealTimeRunningAverage<u32>,
     fps_display: Periodically,
-    frame: usize
+    frame: usize,
+    frame_count: usize
 }
 
 impl<T: Output, S: Surfaces> Renderer<T, S> {
@@ -62,7 +65,8 @@ impl<T: Output, S: Surfaces> Renderer<T, S> {
             surfaces: surfaces,
             fps: RealTimeRunningAverage::default(),
             fps_display: Periodically::new_every_n_seconds(5),
-            frame: 0
+            frame: 0,
+            frame_count: 0
         }
     }
 }
@@ -70,17 +74,33 @@ impl<T: Output, S: Surfaces> Renderer<T, S> {
 impl<T: Output, S: Surfaces> Task for Renderer<T, S> {
     fn name(&self) -> &'static str { "Renderer" }
 
-    fn tick(&mut self) {
+    fn on_property_change(&mut self, key: PropertyID, value: &Variant, _bus: &mut EventBus) {
+        self.output.on_event(&Event::new_property_change(key, value.clone()));
+    }
+
+    fn on_tick(&mut self, bus: &mut EventBus) {
         self.output.blank();
 
         self.surfaces.render_to(&mut self.output, self.frame);
 
         self.output.commit();
 
-        self.fps.insert(1);
         self.frame += 1;
         self.fps_display.run(|| {
-            log::info!("FPS: {}", self.fps.measurement());
+            self.fps.insert((self.frame - self.frame_count) as u32);
+            self.frame_count = self.frame;
+            let fps = self.fps.measurement();
+            bus.set_property(crate::render::props::Output::FPS, fps.rate() as u32);
         });
     }
 }
+
+pub mod props {
+    use crate::property_namespace;
+
+    property_namespace!(
+        Output,
+        FPS => 0,
+        Brightness => 0
+    );
+}

src/scenes.rs

@@ -0,0 +1,144 @@
+use crate::task::Task;
+use crate::events::*;
+use crate::properties::*;
+
+#[derive(Debug, PartialEq, Eq, Clone)]
+struct Scene {
+    name: &'static str,
+    patterns: Vec<&'static str>,
+    trigger: Trigger
+}
+
+#[derive(Debug, PartialEq, Eq, Clone)]
+enum Trigger {
+    Never,
+    Startup,
+    PropertyEquals(PropertyID, Variant)
+}
+
+pub struct Sequencer {
+    scenes: Vec<Scene>,
+    cur_scene: String
+}
+
+impl Sequencer {
+    pub fn new() -> Self {
+        Sequencer {
+            cur_scene: String::new(),
+            scenes: vec![
+                Scene {
+                    name: "Start",
+                    patterns: vec!["Test"],
+                    trigger: Trigger::Startup
+                },
+                Scene {
+                    name: "Test",
+                    patterns: vec!["Test"],
+                    trigger: Trigger::Never
+                },
+                Scene {
+                    name: "Offline",
+                    patterns: vec!["Offline"],
+                    trigger: Trigger::PropertyEquals(System::NetworkOnline.into(), Variant::Boolean(false))
+                },
+                Scene {
+                    name: "Active",
+                    patterns: vec!["Active"],
+                    trigger: Trigger::PropertyEquals(System::Idle.into(), Variant::Boolean(false))
+                },
+                Scene {
+                    name: "Idle",
+                    patterns: vec!["Idle"],
+                    trigger: Trigger::PropertyEquals(System::Idle.into(), Variant::Boolean(true))
+                }
+            ]
+        }
+    }
+
+    fn get_scene(&self, name: &String) -> Option<&Scene> {
+        for scene in self.scenes.iter() {
+            if scene.name == name {
+                return Some(scene);
+            }
+        }
+
+        return None;
+    }
+
+    fn apply_scene(&mut self, name: &String, bus: &mut EventBus) {
+        if let Some(dst_tasks) = self.get_scene(name) {
+            if let Some(src_tasks) = self.get_scene(&self.cur_scene) {
+                let stop_queue = src_tasks.patterns.iter().filter(|i| !dst_tasks.patterns.contains(i));
+                let start_queue = dst_tasks.patterns.iter().filter(|i| !src_tasks.patterns.contains(i));
+
+                log::info!("Switching scene from {} to {}", self.cur_scene, name);
+
+                for stop in stop_queue {
+                    bus.push(Event::new_stop_thing(stop));
+                }
+                for start in start_queue {
+                    bus.push(Event::new_start_thing(start));
+                }
+            } else {
+                log::info!("Starting new scene {}", name);
+                log::info!("start={:?}", dst_tasks.patterns);
+                for start in dst_tasks.patterns.iter() {
+                    bus.push(Event::new_start_thing(start));
+                }
+            }
+            self.cur_scene = name.clone();
+        } else {
+            panic!("Could not apply scene {:?} scenes={:?}", name, self.scenes);
+        }
+    }
+}
+
+pub mod props {
+    use crate::property_namespace;
+
+    property_namespace!(
+        Scenes,
+        Current => "",
+        All => Vec::new()
+    );
+}
+
+use crate::scenes::props::Scenes;
+
+impl Task for Sequencer {
+    fn start(&mut self, bus: &mut EventBus) {
+        log::info!("Starting sequencer!!!");
+
+        let mut scene_list = Vec::new();
+        for scene in self.scenes.iter() {
+            scene_list.push(Variant::String(scene.name.to_string()));
+        }
+
+        bus.set_property(Scenes::All, scene_list);
+
+        let startup_scene = self.scenes.iter().filter(|i| { i.trigger == Trigger::Startup }).next().unwrap();
+        bus.set_property(Scenes::Current, startup_scene.name);
+    }
+
+    fn on_property_change(&mut self, key: PropertyID, value: &Variant, bus: &mut EventBus) {
+        match (key, value) {
+            (Scenes::Current, Variant::String(ref scene_name)) => {
+                log::info!("Applying scene");
+                self.apply_scene(scene_name, bus);
+            },
+            (key, value) => {
+                for scene in self.scenes.iter() {
+                    match scene.trigger {
+                        Trigger::PropertyEquals(trigger_key, ref trigger_value) => {
+                            if trigger_key == key && trigger_value == value {
+                                log::info!("Triggering scene {}", scene.name);
+                                bus.set_property(Scenes::Current, scene.name);
+                            }
+                        },
+                        _ => ()
+                    }
+                }
+            }
+        }
+    }
+}

src/task.rs

@@ -1,15 +1,39 @@
 use core::fmt;
 
+use crate::{events::{Event, EventBus}, properties::{Properties, PropertyID, Variant}, EventSource, Evented};
+
+pub struct Environment<'a> {
+    pub props: &'a mut Properties,
+    pub bus: &'a mut EventBus
+}
+
+impl Environment {
+    
+}
+
+impl<'a> Environment<'a> {
+    pub fn get_property<K: Into<PropertyID>, V>(&mut self, key: K) -> Option<V> where Variant: Into<V> {
+        self.props.get(key.into()).and_then(|f| { Some(f.into()) })
+    }
+
+    pub fn set_property<K: Into<PropertyID>, V: Into<Variant>>(&mut self, key: K, value: V) {
+        self.bus.push(Event::new_property_change(key.into(), value.into()));
+    }
+}
+
 pub trait Task: Send {
-    fn tick(&mut self) {}
-    fn start(&mut self) {}
-    fn stop(&mut self) {}
+    fn on_ready(&mut self, env: &mut Environment) {}
+    fn on_tick(&mut self, env: &mut Environment) {}
+    fn on_property_change(&mut self, key: PropertyID, value: &Variant, env: &mut Environment) {}
+    
+    fn start(&mut self, env: &mut Environment) {}
+    fn stop(&mut self, env: &mut Environment) {}
     fn name(&self) -> &'static str {
         core::any::type_name::<Self>()
     }
 }
 
-#[derive(Debug, Copy, Clone)]
+#[derive(Debug, Clone)]
 enum ScheduledState {
     Stopped,
     Start,
@@ -34,78 +58,105 @@ impl core::fmt::Debug for ScheduledTask {
 impl ScheduledTask {
     fn new(task: Box<dyn Task>) -> Self {
         ScheduledTask {
-            state: ScheduledState::Start,
+            state: ScheduledState::Stopped,
             task: task,
         }
     }
 
     fn start(&mut self) {
-        self.state = match self.state {
-            ScheduledState::Stopped => ScheduledState::Start,
-            ScheduledState::Stop => ScheduledState::Running,
-            _ => self.state
+        match self.state {
+            ScheduledState::Stopped => self.state = ScheduledState::Start,
+            ScheduledState::Stop => self.state = ScheduledState::Running,
+            _ => ()
         }
     }
 
     fn stop(&mut self) {
-        self.state = match self.state {
-            ScheduledState::Running => ScheduledState::Stop,
-            ScheduledState::Start => ScheduledState::Stopped,
-            _ => self.state
+        match self.state {
+            ScheduledState::Running => self.state = ScheduledState::Stop,
+            ScheduledState::Start => self.state = ScheduledState::Stopped,
+            _ => ()
         }
     }
 
-    fn tick(&mut self) {
-        self.state = match self.state {
+    fn on_event(&mut self, event: &Event, env: &mut Environment) {
+        match self.state {
             ScheduledState::Start => {
                 log::info!("Starting task {}", self.task.name());
-                self.task.start();
-                ScheduledState::Running
-            },
-            ScheduledState::Running => {
-                self.task.tick();
-                ScheduledState::Running
+                self.task.start(env);
+                self.state = ScheduledState::Running
             },
             ScheduledState::Stop => {
                 log::info!("Stopping task {}", self.task.name());
-                self.task.stop();
-                ScheduledState::Stopped
+                self.task.stop(env);
+                self.state = ScheduledState::Stopped
             },
-            ScheduledState::Stopped => ScheduledState::Stopped
+            _ => ()
+        };
+
+        match self.state {
+            ScheduledState::Running => {
+                match event {
+                    Event::Tick => self.task.on_tick(env),
+                    Event::ReadyToRock => self.task.on_ready(env),
+                    Event::PropertyChange(key, value) => self.task.on_property_change(key.clone(), value, env),
+                    _ => ()
+                }
+            },
+            _ => ()
         }
     }
 }
 
 #[derive(Debug)]
 pub struct FixedSizeScheduler<const TASK_COUNT: usize> {
-    tasks: [Option<ScheduledTask>; TASK_COUNT],
+    tasks: [ScheduledTask; TASK_COUNT],
 }
 
 impl<const TASK_COUNT: usize> FixedSizeScheduler<TASK_COUNT> {
-    pub fn new(tasks: [Box<dyn Task>; TASK_COUNT]) -> Self {
-        let mut scheduled = [const { None }; TASK_COUNT];
+pub fn new(tasks: [Box<dyn Task>; TASK_COUNT]) -> Self {
+        let mut scheduled: [ScheduledTask; TASK_COUNT] = unsafe { std::mem::MaybeUninit::zeroed().assume_init() };
         let mut idx = 0;
         for task in tasks {
-            scheduled[idx] = Some(ScheduledTask::new(task));
+            log::info!("Scheduling task {}", task.name());
+            let slot = &mut scheduled[idx];
+            unsafe { std::ptr::write(slot, ScheduledTask::new(task)) };
             idx += 1;
         }
         FixedSizeScheduler {
             tasks: scheduled
         }
     }
-}
 
-impl<const TASK_COUNT: usize> Scheduler for FixedSizeScheduler<TASK_COUNT> {
-    fn tick(&mut self) {
+    fn find_task(&mut self, name: &str) -> Option<&mut ScheduledTask> {
         for slot in &mut self.tasks {
-            match slot {
-                Some(task) => task.tick(),
-                _ => ()
+            if slot.task.name() == name {
+                return Some(slot);
+            }
+        }
+
+        None
+    }
+
+    fn on_event(&mut self, event: &Event, env: &mut Environment) {
+        match event {
+            Event::StartThing(task_name) => {
+                if let Some(slot) = self.find_task(task_name) {
+                    log::info!("Queuing start for {}", task_name);
+                    slot.start();
+                }
+            },
+            Event::StopThing(task_name) => {
+                if let Some(slot) = self.find_task(task_name) {
+                    log::info!("Queuing stop for {}", task_name);
+                    slot.stop();
+                }
+            },
+            _ => {
+                for slot in &mut self.tasks {
+                    slot.on_event(event, env);
+                }
             }
         }
     }
-}
-
-pub trait Scheduler {
-    fn tick(&mut self);
 }

src/time.rs

@@ -1,7 +1,7 @@
 use core::time::Duration;
 use std::time::Instant;
 
-#[derive(Debug, Clone, Copy)]
+#[derive(Debug)]
 pub struct Periodically {
     last_run: Instant,
     duration: Duration