wip

.gitignore

@@ -1,4 +1,3 @@
-/.vscode
 /.embuild
 /target
-/Cargo.lock
+/Cargo.lock

.vscode/launch.json

@@ -0,0 +1,26 @@
+{
+    // Use IntelliSense to learn about possible attributes.
+    // Hover to view descriptions of existing attributes.
+    // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
+    "version": "0.2.0",
+    "configurations": [
+        {
+            "type": "probe-rs-debug",
+            "request": "launch",
+            "name": "probe-rs Test",
+            "cwd": "${workspaceFolder}",
+            "connectUnderReset": true,
+            "chip": "ESP32S3",
+            "flashingConfig": {
+                "flashingEnabled": true,
+                "haltAfterReset": true
+            },
+            "coreConfigs": [
+                {
+                    "coreIndex": 0,
+                    "programBinary": "./target/xtensa-esp32s3-espidf/debug/${workspaceFolderBasename}"
+                }
+            ]
+        }
+    ]
+}

.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

@@ -11,11 +11,13 @@ name = "renderbug"
 harness = false # do not use the built in cargo test harness -> resolve rust-analyzer errors
 
 [profile.release]
-opt-level = "s"
+opt-level = 3
+lto = true
 
 [profile.dev]
 debug = true    # Symbols are nice and they don't increase the size on Flash
 opt-level = "z"
+lto = true
 
 [features]
 default = ["std", "esp-idf-svc/native", "rmt", "smart-leds"]
@@ -29,8 +31,6 @@ pio = ["esp-idf-svc/pio"]
 std = ["alloc", "esp-idf-svc/binstart", "esp-idf-svc/std"]
 alloc = ["esp-idf-svc/alloc"]
 
-threads = []
-
 [dependencies]
 log = { version = "0.4", default-features = false }
 esp-idf-svc = { version = "0.49", default-features = false }
@@ -50,6 +50,7 @@ embedded-graphics = { version = "0.8.1", optional = true, features = ["fixed_poi
 ansi_term = "0.12.1"
 num = "0.4.3"
 chrono = "0.4.38"
+fugit = "0.3.7"
 
 [build-dependencies]
 embuild = "0.32.0"

src/animations.rs

@@ -2,6 +2,7 @@ use palette::Hsv;
 
 use rgb::RGB8;
 
+use crate::events::{Event, EventBus};
 use crate::time::Periodically;
 use crate::geometry::*;
 use crate::render::{Shader, Surface, Surfaces};
@@ -54,15 +55,6 @@ impl Shader for ThinkingShader {
     }
 }
 
-#[derive(Debug)]
-struct GayBarberShader {}
-
-impl Shader for GayBarberShader {
-    fn draw(&self, coords: &VirtualCoordinates, frame: usize) -> RGB8 {
-        Hsv::new_srgb((frame as u8).wrapping_add(coords.x).wrapping_add(coords.y), 255, sin8((frame % 128) as u8)).into_rgb8()
-    }
-}
-
 impl<T: Surface> IdleTask<T> {
     pub fn new<S: Surfaces<Surface = T>>(surfaces: &mut S) -> Self {
         IdleTask {
@@ -84,7 +76,7 @@ impl<T: Surface> Task for IdleTask<T> {
         self.shimmer.set_opacity(64);
     }
 
-    fn tick(&mut self) {}
+    fn tick(&mut self, event: &Event, bus: &mut EventBus) {}
 
     fn stop(&mut self) {
         self.solid.clear_shader();
@@ -123,7 +115,7 @@ impl TestShader {
             Self::SweepY => Self::SinX,
             Self::SinX => Self::SinY,
             Self::SinY => Self::Metaballs,
-            Self::Metaballs => Self::RGB
+            Self::Metaballs => Self::Red
         }
     }
 }
@@ -181,7 +173,7 @@ impl<T: Surface> Task for TestPattern<T> {
         self.surface.set_shader(Box::new(self.pattern.clone()));
     }
 
-    fn tick(&mut self) {
+    fn tick(&mut self, event: &Event, bus: &mut EventBus) {
         self.updater.run(|| {
             self.pattern = self.pattern.next();
             log::info!("Test pattern: {:?}", self.pattern);

src/buffers.rs

@@ -1,89 +1,255 @@
+use crate::events::{Event, EventBus};
 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::rc::Rc;
-use std::cell::RefCell;
 use std::io;
 use std::ops::IndexMut;
 
-#[cfg(feature="threads")]
+use std::sync::atomic::AtomicBool;
+use std::sync::RwLock;
 use std::sync::{Arc, Mutex};
 
 #[derive(Debug)]
-pub struct ShaderBinding {
+struct ShaderBinding {
     shader: Option<Box<dyn Shader>>,
     rect: Rectangle<Virtual>,
     opacity: u8
 }
 
-#[derive(Clone)]
-pub struct BoundSurface<T> {
-    pub binding: T
+#[derive(Debug)]
+struct SurfaceUpdate {
+    shader: Option<Option<Box<dyn Shader>>>,
+    rect: Option<Rectangle<Virtual>>,
+    opacity: Option<u8>,
+    slot: usize
 }
 
-impl Debug for BoundSurface<Rc<RefCell<ShaderBinding>>> {
+impl SurfaceUpdate {
+    fn merge(&mut self, mut other: Self) {
+        if other.shader.is_some() {
+            self.shader = other.shader.take()
+        }
+        if other.rect.is_some() {
+            self.rect = other.rect.take()
+        }
+        if other.opacity.is_some() {
+            self.opacity = other.opacity.take()
+        }
+    }
+}
+
+impl Debug for Box<dyn Shader> {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-        f.debug_struct("BoundSurface")
-        .field("shader", &self.binding.borrow().shader)
-        .field("opacity", &self.binding.borrow().opacity)
-        .finish()
+        f.debug_struct("Shader").finish()
     }
 }
 
-pub type SimpleSurface = BoundSurface<Rc<RefCell<ShaderBinding>>>;
-
-impl Default for BoundSurface<Rc<RefCell<ShaderBinding>>>{
+impl Default for SurfaceUpdate {
     fn default() -> Self {
-        Self {
-            binding: Rc::new(RefCell::new(ShaderBinding {
-                shader: None,
-                rect: Rectangle::everything(),
-                opacity: 255
-            })),
+        SurfaceUpdate {
+            shader: None,
+            rect: None,
+            opacity: None,
+            slot: usize::MAX
         }
     }
 }
 
-impl Surface for BoundSurface<Rc<RefCell<ShaderBinding>>> {
-    fn rect(&self) -> Rectangle<Virtual> {
-        self.binding.borrow().rect
-    }
-
-    fn with_shader<F: FnMut(&dyn Shader)>(&self, mut f: F) {
-        if let Some(ref shader) = self.binding.borrow().shader {
-            f(shader.as_ref());
-        }
-    }
-
-    fn set_shader(&mut self, shader: Box<dyn Shader>) {
-        self.binding.borrow_mut().shader = Some(shader);
-    }
+#[derive(Debug)]
+pub struct BufferedSurface {
+    updater: Arc<UpdateQueue>,
+    slot: usize
+}
 
+impl Surface for BufferedSurface {
     fn clear_shader(&mut self) {
-        self.binding.borrow_mut().shader = None;
-    }
-    
-    fn set_rect(&mut self, rect: &Rectangle<Virtual>) {
-        self.binding.borrow_mut().rect = *rect;
-    }
-
-    fn opacity(&self) -> u8 {
-        self.binding.borrow().opacity
+        self.updater.push(SurfaceUpdate {
+            shader: None,
+            slot: self.slot,
+            ..Default::default()
+        });
     }
 
     fn set_opacity(&mut self, opacity: u8) {
-        self.binding.borrow_mut().opacity = opacity
+        self.updater.push(SurfaceUpdate {
+            opacity: Some(opacity),
+            slot: self.slot,
+            ..Default::default()
+        });
+    }
+
+    fn set_rect(&mut self, rect: &Rectangle<Virtual>) {
+        self.updater.push(SurfaceUpdate {
+            rect: Some(rect.clone()),
+            slot: self.slot,
+            ..Default::default()
+        });
+    }
+
+    fn set_shader(&mut self, shader: Box<dyn Shader>) {
+        self.updater.push(SurfaceUpdate {
+            shader: Some(Some(shader)),
+            slot: self.slot,
+            ..Default::default()
+        });
     }
 }
 
-#[cfg(feature="threads")]
-pub type SharedSurface = BoundSurface<Arc<Mutex<ShaderBinding>>>;
+#[derive(Debug)]
+struct UpdateQueue {
+    pending: Mutex<Vec<SurfaceUpdate>>,
+    damaged: AtomicBool
+}
 
-#[cfg(feature="threads")]
-impl Default for BoundSurface<Arc<Mutex<ShaderBinding>>> {
+impl UpdateQueue {
+    fn new() -> Self {
+        UpdateQueue {
+            pending: Mutex::new(Vec::new()),
+            damaged: AtomicBool::new(false)
+        }
+    }
+
+    fn push(&self, update: SurfaceUpdate) {
+        let mut locked = self.pending.lock().unwrap();
+        let mut existing_slot = None;
+        for existing in locked.iter_mut() {
+            if existing.slot == update.slot {
+                existing_slot = Some(existing);
+                break
+            }
+        }
+        match existing_slot {
+            Some(tgt) => {
+                log::debug!("Updating existing shader update");
+                tgt.merge(update);
+            }
+            _ => {
+                log::debug!("Pushing new shader update");
+                locked.push(update);
+                self.damaged.store(true, std::sync::atomic::Ordering::Relaxed);
+            }
+        }
+    }
+}
+
+#[derive(Debug)]
+pub struct ShaderChain {
+    bindings: Vec<ShaderBinding>,
+    updates: Arc<UpdateQueue>
+}
+
+impl ShaderChain {
+    pub fn new() -> Self {
+        ShaderChain {
+            bindings: Vec::new(),
+            updates: Arc::new(UpdateQueue::new())
+        }
+    }
+
+    pub fn is_dirty(&self) -> bool {
+        self.updates.damaged.load(std::sync::atomic::Ordering::Relaxed)
+    }
+
+    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;
+            }
+        }
+        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> {
+        let next_slot = self.bindings.len();
+        self.bindings.push(ShaderBinding {
+            opacity: 255,
+            shader: None,
+            rect: area.clone()
+        });
+
+        Ok(BufferedSurface {
+            updater: Arc::clone(&self.updates),
+            slot: next_slot
+        })
+    }
+
+    fn render_to<S: Sample>(&self, output: &mut S, frame: usize) {
+        for surface in self.bindings.iter() {
+            let opacity = surface.opacity;
+            if opacity > 0 {
+                let rect = surface.rect;
+                let mut sample = output.sample(&rect);
+                if let Some(ref shader) = surface.shader {
+                    while let Some((virt_coords, pixel)) = sample.next() {
+                        *pixel = pixel.blend8(shader.draw(&virt_coords, frame).into(), opacity);
+                    }
+                }
+            }
+        }
+    }
+}
+
+#[derive(Clone)]
+pub struct BufferedSurfacePool {
+    pool: Arc<RwLock<ShaderChain>>
+}
+
+impl BufferedSurfacePool {
+    pub fn new() -> Self {
+        BufferedSurfacePool {
+            pool: Arc::new(RwLock::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)
+    }
+
+    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, event: &Event, bus: &mut EventBus) {
+        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 {
@@ -95,19 +261,7 @@ impl Default for BoundSurface<Arc<Mutex<ShaderBinding>>> {
     }
 }
 
-#[cfg(feature="threads")]
-impl Surface for BoundSurface<Arc<Mutex<ShaderBinding>>> {
-    fn rect(&self) -> Rectangle<Virtual> {
-        let r = self.binding.lock().unwrap();
-        r.rect.clone()
-    }
-
-    fn with_shader<F: FnMut(&dyn Shader)>(&self, mut f: F) {
-        if let Some(ref shader) = self.binding.lock().unwrap().shader {
-            f(shader.as_ref());
-        }
-    }
-
+impl Surface for SharedSurface {
     fn set_shader(&mut self, shader: Box<dyn Shader>) {
         self.binding.lock().unwrap().shader = Some(shader);
     }
@@ -120,67 +274,48 @@ impl Surface for BoundSurface<Arc<Mutex<ShaderBinding>>> {
         self.binding.lock().unwrap().rect = rect.clone();
     }
 
-    fn opacity(&self) -> u8 {
-        self.binding.lock().unwrap().opacity
-    }
-
     fn set_opacity(&mut self, opacity: u8) {
         self.binding.lock().unwrap().opacity = opacity
     }
 }
 
-#[cfg(feature="threads")]
-impl Debug for BoundSurface<Arc<Mutex<ShaderBinding>>> {
-    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-        f.debug_struct("BoundSurface")
-        .field("shader", &self.binding.lock().unwrap().shader)
-        .finish()
-    }
-}
 
 #[derive(Clone)]
-pub struct SurfacePool<S: Surface + Default> {
-    surfaces: Vec<S>
+pub struct SurfacePool {
+    surfaces: Vec<SharedSurface>
 }
 
-impl<S: Surface + Debug> Debug for SurfacePool<S> {
-    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-        self.surfaces.fmt(f)
-    }
-}
-
-impl<S: Surface + Default> SurfacePool<S> {
+impl SurfacePool {
     pub const fn new() -> Self {
         Self {
             surfaces: Vec::new()
         }
     }
-
-    pub fn iter(&self) -> std::slice::Iter<S> {
-        self.surfaces.iter()
-    }
 }
 
-impl<S: Surface + Default> Surfaces for SurfacePool<S> {
-    type Surface = S;
-    fn new_surface(&mut self, area: &Rectangle<Virtual>) -> Result<S, io::Error> {
-        let mut surface = S::default();
+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.iter() {
-            let opacity = surface.opacity();
+        for surface in self.surfaces.iter() {
+            let binding = surface.binding.lock().unwrap();
+            let opacity = binding.opacity;
             if opacity > 0 {
-                let rect = surface.rect();
+                let rect = binding.rect;
                 let mut sample = output.sample(&rect);
-                surface.with_shader(|shader| {
+
+                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);
                     }
-                })
+                }
             }
         }
     }

src/events.rs

@@ -0,0 +1,163 @@
+use core::fmt::Debug;
+
+#[derive(Debug, Clone, Copy, PartialEq, Eq)]
+pub enum InputEvent {
+    PowerOn,
+    PowerOff,
+    NetworkActivity,
+    NetworkOnline,
+    NetworkOffline
+}
+
+#[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord)]
+pub enum Variant {
+    Byte(u8),
+    UInt(u32),
+    Int(i32),
+    BigUInt(u64),
+    BigInt(i64),
+    Boolean(bool)
+}
+
+#[derive(Debug, Clone, Copy, PartialEq, Eq)]
+pub enum Event {
+    ReadyToRock,
+    Tick,
+    StartThing(&'static str),
+    StopThing(&'static str),
+    Input(InputEvent),
+    PropertyChange(&'static str, Variant)
+}
+
+pub struct SystemState {
+    key: &'static str,
+    value: Variant,
+    values: Vec::<Box<SystemState>>
+}
+
+impl SystemState {
+    pub fn new() -> Self {
+        SystemState {
+            key: "",
+            value: Variant::Byte(0),
+            values: Vec::new()
+        }
+    }
+
+    fn get_key(&self, key: &'static str) -> Option<&Self> {
+        if key == self.key {
+            Some(self)
+        } else {
+            for next in self.values.iter() {
+                match next.get_key(key) {
+                    None => (),
+                    Some(next_val) => return Some(next_val)
+                }
+            }
+            return None
+        }
+    }
+
+    fn get_key_mut(&mut self, key: &'static str) -> Option<&mut Self> {
+        if key == self.key {
+            Some(self)
+        } else {
+            for next in self.values.iter_mut() {
+                match next.get_key_mut(key) {
+                    None => (),
+                    Some(next_val) => return Some(next_val)
+                }
+            }
+            return None
+        }
+    }
+
+    pub fn get(&self, key: &'static str) -> Option<Variant> {
+        match self.get_key(key) {
+            None => None,
+            Some(v) => Some(v.value)
+        }
+    }
+
+    pub fn set<V>(&mut self, key: &'static str, value: V) where Variant: From<V> {
+        match self.get_key_mut(key) {
+            None => self.values.push(Box::new(SystemState {
+                value: value.into(),
+                key: key,
+                values: Vec::new()
+            })),
+            Some(found_key) => {
+                found_key.value = value.into()
+            }
+        }
+    }
+}
+
+impl Event {
+    pub fn new_tick() -> Self {
+        Event::Tick
+    }
+
+    pub fn new_property_change<T>(key: &'static str, data: T) -> Self where Variant: From<T> {
+        Event::PropertyChange(key, Variant::from(data))
+    }
+
+    pub fn new_ready_to_rock() -> Self {
+        Event::ReadyToRock
+    }
+
+    pub fn new_input_event(event: InputEvent) -> Self {
+        Event::Input(event)
+    }
+}
+
+impl Into<u8> for Variant {
+    fn into(self) -> u8 {
+        match self {
+            Variant::Byte(b) => b,
+            _ => 0
+        }
+    }
+}
+
+impl From<bool> for Variant {
+    fn from(value: bool) -> Self {
+        Variant::Boolean(value)
+    }
+}
+
+impl From<i64> for Variant {
+    fn from(value: i64) -> Self {
+        Variant::BigInt(value)
+    }
+}
+
+impl From<u8> for Variant {
+    fn from(value: u8) -> Self {
+        Variant::Byte(value)
+    }
+}
+
+pub struct EventBus {
+    pending: Vec<Event>
+}
+
+impl EventBus {
+    pub fn new() -> Self {
+        EventBus {
+            pending: Vec::new()
+        }
+    }
+
+    pub fn next(&mut self) -> Event {
+        if self.pending.len() == 0 {
+            Event::new_tick()
+        } else {
+            self.pending.pop().unwrap()
+        }
+    }
+
+    pub fn push(&mut self, event: Event) {
+        self.pending.push(event);
+    }
+}

src/lib8/interpolate.rs

@@ -82,6 +82,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/main.rs

@@ -8,9 +8,13 @@ mod platform;
 mod animations;
 mod mappings;
 mod buffers;
+mod events;
 
+use events::Event;
+
+use crate::events::EventBus;
 use crate::platform::{DefaultBoard, Board};
-use crate::task::{Task, Scheduler};
+use crate::task::{FixedSizeScheduler, Scheduler};
 use crate::render::{Surfaces, Renderer};
 use crate::geometry::Rectangle;
 
@@ -19,27 +23,39 @@ fn main() {
 
     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!("Creating output");
     let output = board.output();
     log::info!("Output: {}", core::any::type_name_of_val(&output));
 
+    log::info!("Preparing surfaces");
     let mut surfaces = board.surfaces();
     log::info!("Surface implementation: {}", core::any::type_name_of_val(&output));
 
     log::info!("Creating animations");
-    let mut tasks: Vec<Box<dyn Task + 'static>> = vec![
+    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::TestPattern::new(surfaces.new_surface(&Rectangle::everything()).unwrap())),
+    ]);
 
-    tasks.append(&mut board.system_tasks());
-    tasks.push(Box::new(Renderer::new(output, surfaces)));
+    let mut renderer = FixedSizeScheduler::new([Box::new(Renderer::new(output, surfaces))]);
 
-    let mut runner = Scheduler::new(tasks);
-
-    log::info!("Runner ready: {:?}", runner);
+    log::info!("Starting event bus");
+    let mut bus = EventBus::new();
 
     log::info!("Ready to rock and roll");
+    bus.push(Event::new_ready_to_rock());
     loop {
-        runner.tick();
+        let next_event = bus.next();
+        match next_event {
+            events::Event::Tick => (),
+            _ => log::info!("Event: {:?}", next_event)
+        }
+        animations.tick(&next_event, &mut bus);
+        system.tick(&next_event, &mut bus);
+        renderer.tick(&next_event, &mut bus);
     }
 }

src/mappings.rs

@@ -1,6 +1,8 @@
+use crate::buffers::Pixbuf;
 use crate::geometry::*;
 
 use crate::lib8::interpolate::scale8;
+use crate::render::PixelView;
 
 use core::cmp::{max, min};
 use core::fmt::{Formatter, Debug};
@@ -99,6 +101,12 @@ impl<const STRIDE_NUM: usize> StrideMapping<STRIDE_NUM> {
         ])
     }
 
+    pub fn new_fairylights() -> Self {
+        Self::from_json(&[
+            (0, 0, 50, false)
+        ])
+    }
+
     pub fn new_jar() -> Self {
         Self::from_json(&[
             (0, 0, 17, false),
@@ -301,3 +309,29 @@ impl<'a> CoordinateView<'a> for StrideView<'a> {
         None
     }
 }
+
+pub struct StrideSampler<'a, P: Pixbuf> {
+    pixbuf: &'a mut P,
+    selection: StrideView<'a>
+}
+
+impl<'a, P: Pixbuf> StrideSampler<'a, P> {
+    pub fn new(pixbuf: &'a mut P, selection: StrideView<'a>) -> Self {
+        StrideSampler {
+            pixbuf,
+            selection
+        }
+    }
+}
+
+impl<'a, P: Pixbuf> PixelView for StrideSampler<'a, P> {
+    type Pixel = P::Pixel;
+    fn next(&mut self) -> Option<(Coordinates<Virtual>, &mut Self::Pixel)> {
+        if let Some((virt, coords)) = self.selection.next() {
+            let idx = self.selection.map.strides[coords.x as usize].pixel_idx_for_offset(coords.y);
+            Some((virt, &mut self.pixbuf[idx]))
+        } else {
+            None
+        }
+    }
+}

src/platform/esp32.rs

@@ -1,45 +1,108 @@
+use core::borrow::BorrowMut;
 use std::sync::Arc;
 use std::fmt::Debug;
 use std::sync::Mutex;
+use std::thread::JoinHandle;
+use std::thread::ScopedJoinHandle;
 
 use chrono::DateTime;
+use chrono::Timelike;
 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::mqtt::client::EspMqttClient;
+use esp_idf_svc::mqtt::client::EspMqttConnection;
+use esp_idf_svc::mqtt::client::MqttClientConfiguration;
 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::task::Task;
-use crate::buffers::{Pixbuf, SurfacePool};
+use crate::buffers::BufferedSurfacePool;
+use crate::buffers::Pixbuf;
+use crate::events::Event;
+use crate::events::EventBus;
+use crate::lib8::interpolate::lerp8by8;
 use crate::mappings::StrideMapping;
-use crate::platform::smart_leds_lib::StrideOutput;
-use crate::platform::smart_leds_lib::rmt::FastWs2812Esp32Rmt;
+use crate::task::FixedSizeScheduler;
+use crate::task::Task;
 use crate::time::Periodically;
 
-#[cfg(feature="threads")]
-use crate::buffers::SharedSurface as SurfaceType;
+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;
 
-#[cfg(not(feature="threads"))]
-use crate::buffers::SimpleSurface as SurfaceType;
+    pub struct I2SOutput<'d> {
+        driver: I2sDriver<'d, I2sTx>,
+        pixbuf: [Rgb<u8>; 310],
+        pixmap: StrideMapping,
+    }
 
-pub struct Esp32Board<'a> {
-    output: Option<<Self as Board>::Output>,
-    surfaces: Option<SurfacePool<SurfaceType>>,
-    tasks: Option<Vec<Box<dyn Task>>>
+    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 on_event(&mut self, event: &crate::events::Event) {
+            
+        }
+        
+        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))
+        }
+    }
 }
 
-impl<'a> Board for Esp32Board<'a> {
-    type Output = StrideOutput<[Rgb<u8>; 310], FastWs2812Esp32Rmt<'a>>;
-    type Surfaces = SurfacePool<SurfaceType>;
+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<4>;
     
     fn take() -> Self {
         // It is necessary to call this function once. Otherwise some patches to the runtime
@@ -51,48 +114,86 @@ impl<'a> Board for Esp32Board<'a> {
 
         let peripherals = Peripherals::take().unwrap();
         let sys_loop = EspSystemEventLoop::take().unwrap();
-        let nvs = EspDefaultNvsPartition::take().unwrap();
 
-        let channel = peripherals.rmt.channel0;
-        let pins = peripherals.pins;
         let mut chip_id: [u8; 8] = [0; 8];
         unsafe {
             esp_efuse_mac_get_default(&mut chip_id as *mut u8);
         }
 
-        log::info!("Setting up for chip ID {:?}", chip_id);
-
-        let (pixmap, target) = match chip_id {
-            [72, 202, 67, 89, 145, 204, 0, 0] => {
-                (StrideMapping::new_panel(), FastWs2812Esp32Rmt::new(channel, pins.gpio5).unwrap())
-            },
-            [140, 170, 181, 131, 95, 116, 0, 0] => {
-                (StrideMapping::new_jar(), FastWs2812Esp32Rmt::new(channel, pins.gpio14).unwrap())
-            },
-            _ => {
-                (StrideMapping::new(), FastWs2812Esp32Rmt::new(channel, pins.gpio5).unwrap())
-            }
-        };
-
-        let tasks: Vec<Box<dyn Task>> = vec![
-            Box::new(WifiTask::new(peripherals.modem, sys_loop.clone(), &nvs)),
-        ];
+        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;
 
-        let output = StrideOutput::new(
-            Pixbuf::new(),
-            pixmap,
-            target,
-            MAX_POWER_MW
-        );
+        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 { // panel test board
+            [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] => { //ponderjar
+                StrideOutput::new(
+                    Pixbuf::new(),
+                    StrideMapping::new_jar(),
+                    std::thread::spawn(move || { FastWs2812Esp32Rmt::new(peripherals.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_jar(),
+                    std::thread::spawn(move || { FastWs2812Esp32Rmt::new(peripherals.rmt.channel0, pins.gpio5).unwrap() }).join().unwrap(),
+                    MAX_POWER_MW
+                )
+            },
+            [0x48, 0xca, 0x43, 0x59, 0x9d, 0x48, 0x0, 0x0] => { // kitchen cabinets
+                StrideOutput::new(
+                    Pixbuf::new(),
+                    StrideMapping::new_fairylights(),
+                    std::thread::spawn(move || { FastWs2812Esp32Rmt::new(peripherals.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(peripherals.rmt.channel0, pins.gpio5).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 {
-            surfaces: Some(SurfacePool::new()),
-            output: Some(output),
-            tasks: Some(tasks),
+            modem: Some(peripherals.modem),
+            sys_loop: sys_loop.clone(),
+            surfaces: BufferedSurfacePool::new(),
+            output: Some(output)
         }
     }
 
@@ -101,11 +202,168 @@ impl<'a> Board for Esp32Board<'a> {
     }
 
     fn surfaces(&mut self) -> Self::Surfaces {
-        self.surfaces.take().unwrap()
+        self.surfaces.clone()
     }
 
-    fn system_tasks(&mut self) -> Vec<Box<dyn Task>> {
-        self.tasks.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(CircadianRhythm::new()),
+            Box::new(MqttTask::new()),
+            Box::new(self.surfaces.clone())
+        ])
+    }
+}
+
+#[derive(Debug, Clone, Copy)]
+struct ScheduleEntry {
+    hour: u8,
+    brightness: u8
+}
+
+struct CircadianRhythm {
+    time_check: Periodically,
+    schedule: [ScheduleEntry;10]
+}
+
+impl CircadianRhythm {
+    fn new() -> Self {
+        CircadianRhythm {
+            time_check: Periodically::new_every_n_seconds(5),
+            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 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 * 60;
+        let end_time = end.hour * 60;
+        let now_time = hour * 60 + minute;
+
+        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 * rise) / run + out_min;
+}
+
+
+impl Task for CircadianRhythm {
+    fn tick(&mut self, event: &Event, bus: &mut EventBus) {
+        if self.time_check.tick() || event.eq(&Event::ReadyToRock) {
+            let now: DateTime<Utc> = std::time::SystemTime::now().into();
+            let next_brightness = self.brightness_for_time(now.hour() as u8, now.minute() as u8);
+            bus.push(Event::new_property_change("output.brightness", next_brightness));
+        }
+    }
+}
+
+struct MqttTask {
+    client: Option<EspMqttClient<'static>>,
+    conn_thread: Option<JoinHandle<()>>,
+}
+
+impl MqttTask {
+    fn new() -> Self {
+        MqttTask {
+            conn_thread: None,
+            client: None
+        }
+    }
+
+    fn start_mqtt(&mut self) {
+        log::info!("Starting MQTT");
+        let (client, mut conn) = EspMqttClient::new(
+            "mqtt://10.0.0.2:1883",
+            &MqttClientConfiguration {
+                client_id: Some("renderbug-rs"),
+                ..Default::default()
+            }
+        ).unwrap();
+        log::info!("Connected!");
+
+        self.conn_thread = Some(std::thread::Builder::new()
+            .stack_size(6000)
+            .spawn(move || {
+                conn.next().unwrap();
+            }).unwrap());
+        self.client = Some(client);
+    }
+}
+
+impl Task for MqttTask {
+    fn tick(&mut self, event: &Event, bus: &mut EventBus) {
+        match event {
+            Event::Input(crate::events::InputEvent::NetworkOnline) => {
+                log::info!("Registering with MQTT");
+
+                self.start_mqtt();
+
+                if let Some(ref mut client) = self.client {
+                    client.enqueue(
+                        "homeassistant-test/renderbug/rust",
+                        esp_idf_svc::mqtt::client::QoS::AtLeastOnce,
+                        false,
+                        "hello, world".as_bytes()
+                    ).unwrap();
+                    log::info!("MQTT should be online!");
+                }
+            },
+            Event::PropertyChange(name, value) => {
+                if let Some(ref mut client) = self.client {
+                    let payload = format!("name={} value={:?}", name, value);
+                    client.enqueue(
+                        "homeassistant-test/renderbug/rust/property-change",
+                        esp_idf_svc::mqtt::client::QoS::AtLeastOnce,
+                        false,
+                        payload.as_bytes()
+                    ).unwrap();
+                    log::info!("property change bump: {}", payload);
+                }
+            }
+            _ => ()
+        }
     }
 }
 
@@ -136,6 +394,7 @@ struct WifiTask {
 
 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(),
@@ -180,8 +439,6 @@ impl Task for WifiTask {
     fn start(&mut self) {
         log::info!("Starting wifi!");
 
-        log::info!("waiting for loop");
-
         let wifi_state = self.state.clone();
         self.wifi_sub = Some(self.sys_loop.subscribe::<WifiEvent, _>( move |evt| {
             log::warn!("wifi event {:?}", evt);
@@ -216,7 +473,7 @@ impl Task for WifiTask {
         self.connect();
     }
 
-    fn tick(&mut self ) {
+    fn tick(&mut self, event: &Event, bus: &mut EventBus) {
         if self.connection_check.tick() {
             let cur_state = *self.state.lock().unwrap();
 
@@ -231,10 +488,12 @@ impl Task for WifiTask {
                 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());
+                match cur_state {
+                    WifiState::Connected => bus.push(Event::new_input_event(crate::events::InputEvent::NetworkOnline)),
+                    _ => bus.push(Event::new_input_event(crate::events::InputEvent::NetworkOffline))
+                }
+            }
         }
     }
 

src/platform/esp32_nostd.rs

@@ -0,0 +1,44 @@
+use rgb::Rgb;
+use super::smart_leds_lib::rmt::FastWs2812Esp32Rmt;
+use super::smart_leds_lib::StrideOutput;
+
+use crate::task::{FixedSizeScheduler, Task};
+use crate::buffers::StaticSurfacePool;
+
+use super::Board;
+
+pub struct Esp32Board<'a> {
+    output: Option<<Self as Board>::Output>,
+    surfaces: Option<StaticSurfacePool>,
+    tasks: Option<[&'a mut dyn Task; 1]>
+}
+
+impl<'a> Board for Esp32Board<'a> {
+    type Output = StrideOutput<[Rgb<u8>; 310], FastWs2812Esp32Rmt<'a>>;
+    type Surfaces = StaticSurfacePool;
+    type Scheduler = FixedSizeScheduler<0>;
+    fn take() -> Self {
+        let peripherals = esp_hal::init(esp_hal::Config::default());
+        //esp_alloc::heap_allocator!(72 * 1024);
+
+        const POWER_VOLTS : u32 = 5;
+        const POWER_MA : u32 = 500;
+        const MAX_POWER_MW : u32 = POWER_VOLTS * POWER_MA;
+        let pins = peripherals.pins;
+
+
+        Esp32Board { output: None, surfaces: None, tasks: None }
+    }
+
+    fn output(&mut self) -> Self::Output {
+        self.output.take().unwrap()
+    }
+
+    fn surfaces(&mut self) -> Self::Surfaces {
+        self.surfaces.take().unwrap()
+    }
+
+    fn system_tasks(&mut self) -> Self::Scheduler {
+        FixedSizeScheduler::new([])
+    }
+}

src/platform/mod.rs

@@ -9,14 +9,15 @@ pub mod esp32;
 pub type DefaultBoard = esp32::Esp32Board;
 
 use crate::render::{Output, Surfaces};
-use crate::task::Task;
+use crate::task::Scheduler;
 
 pub trait Board {
     type Output: Output;
     type Surfaces: Surfaces;
+    type Scheduler: Scheduler;
 
     fn take() -> Self;
     fn output(&mut self) -> Self::Output;
     fn surfaces(&mut self) -> Self::Surfaces;
-    fn system_tasks(&mut self) -> Vec<Box<dyn Task>>;
+    fn system_tasks(&mut self) -> Self::Scheduler;
 }

src/platform/smart_leds_lib.rs

@@ -1,6 +1,7 @@
 use smart_leds_trait::SmartLedsWrite;
 
 use crate::buffers::Pixbuf;
+use crate::events::Variant;
 use crate::render::{HardwarePixel, Output, PixelView, Sample};
 use crate::power::brightness_for_mw;
 use crate::geometry::*;
@@ -8,23 +9,6 @@ use crate::mappings::*;
 
 use core::fmt::Debug;
 
-struct StrideSampler<'a, P: Pixbuf> {
-    pixbuf: &'a mut P,
-    selection: StrideView<'a>
-}
-
-impl<'a, P: Pixbuf> PixelView for StrideSampler<'a, P> {
-    type Pixel = P::Pixel;
-    fn next(&mut self) -> Option<(Coordinates<Virtual>, &mut Self::Pixel)> {
-        if let Some((virt, coords)) = self.selection.next() {
-            let idx = self.selection.map.strides[coords.x as usize].pixel_idx_for_offset(coords.y);
-            Some((virt, &mut self.pixbuf[idx]))
-        } else {
-            None
-        }
-    }
-}
-
 impl<P: Pixbuf, T: FastWrite> Debug for StrideOutput<P, T> {
     fn fmt(&self, f: &mut core::fmt::Formatter<'_>) -> core::fmt::Result {
         f.debug_struct("StrideOutput").finish()
@@ -35,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> {
@@ -45,7 +30,8 @@ impl<P: Pixbuf, T: FastWrite> StrideOutput<P, T> {
             pixbuf,
             stride_map,
             target,
-            max_mw
+            max_mw,
+            brightness: 255
         }
     }
 }
@@ -53,10 +39,7 @@ impl<P: Pixbuf, T: FastWrite> StrideOutput<P, T> {
 impl<P: Pixbuf, T: FastWrite> Sample for StrideOutput<P, T> {
     type Pixel = P::Pixel;
     fn sample(&mut self, rect: &Rectangle<Virtual>) -> impl PixelView<Pixel = Self::Pixel> {
-        StrideSampler {
-            pixbuf: &mut self.pixbuf,
-            selection: self.stride_map.select(rect)
-        }
+        StrideSampler::new(&mut self.pixbuf,self.stride_map.select(rect))
     }
 }
 
@@ -66,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("output.brightness", new_brightness) => self.brightness = new_brightness.clone().into(),
+            _ => ()
+        }
+    }
 }
 
 pub trait FastWrite: Send {

src/render.rs

@@ -1,6 +1,6 @@
-use std::io;
 use rgb::Rgb;
 
+use crate::events::{Event, EventBus};
 use crate::geometry::*;
 use crate::lib8::interpolate::Fract8Ops;
 use crate::power::AsMilliwatts;
@@ -29,22 +29,21 @@ pub trait Shader: Send + Sync {
 
 pub trait Surfaces: Send + Sync {
     type Surface: Surface;
-    fn new_surface(&mut self, area: &Rectangle<Virtual>) -> Result<Self::Surface, io::Error>;
+    type Error: Debug;
+    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 with_shader<F: FnMut(&dyn Shader)>(&self, f: F);
     fn set_shader(&mut self, shader: Box<dyn Shader>);
     fn clear_shader(&mut self);
 
-    fn rect(&self) -> Rectangle<Virtual>;
     fn set_rect(&mut self, rect: &Rectangle<Virtual>);
-    fn opacity(&self) -> u8;
     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);
 }
@@ -73,17 +72,22 @@ 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) {
-        self.output.blank();
+    fn tick(&mut self, event: &Event, _bus: &mut EventBus) {
+        match event {
+            crate::events::Event::Tick => {
+                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.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.output.on_event(event)
+        }
     }
 }

src/task.rs

@@ -1,7 +1,9 @@
 use core::fmt;
 
-pub trait Task: core::fmt::Debug {
-    fn tick(&mut self) {}
+use crate::events::{Event, EventBus};
+
+pub trait Task: Send {
+    fn tick(&mut self, event: &Event, bus: &mut EventBus) {}
     fn start(&mut self) {}
     fn stop(&mut self) {}
     fn name(&self) -> &'static str {
@@ -55,7 +57,7 @@ impl ScheduledTask {
         }
     }
 
-    fn tick(&mut self) {
+    fn tick(&mut self, event: &Event, bus: &mut EventBus) {
         self.state = match self.state {
             ScheduledState::Start => {
                 log::info!("Starting task {}", self.task.name());
@@ -63,7 +65,7 @@ impl ScheduledTask {
                 ScheduledState::Running
             },
             ScheduledState::Running => {
-                self.task.tick();
+                self.task.tick(event, bus);
                 ScheduledState::Running
             },
             ScheduledState::Stop => {
@@ -77,25 +79,35 @@ impl ScheduledTask {
 }
 
 #[derive(Debug)]
-pub struct Scheduler {
-    tasks: Vec<ScheduledTask>,
+pub struct FixedSizeScheduler<const TASK_COUNT: usize> {
+    tasks: [Option<ScheduledTask>; TASK_COUNT],
 }
 
-impl Scheduler {
-    pub fn new(tasks: Vec<Box<dyn Task>>) -> Self {
-        let mut scheduled = Vec::new();
+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];
+        let mut idx = 0;
         for task in tasks {
-            log::info!("Scheduling task {} {:?}", task.name(), task);
-            scheduled.push(ScheduledTask::new(task));
+            scheduled[idx] = Some(ScheduledTask::new(task));
+            idx += 1;
         }
-        Scheduler {
+        FixedSizeScheduler {
             tasks: scheduled
         }
     }
+}
 
-    pub fn tick(&mut self) {
-        for task in &mut self.tasks {
-            task.tick();
+impl<const TASK_COUNT: usize> Scheduler for FixedSizeScheduler<TASK_COUNT> {
+    fn tick(&mut self, event: &Event, bus: &mut EventBus) {
+        for slot in &mut self.tasks {
+            match slot {
+                Some(task) => task.tick(event, bus),
+                _ => ()
+            }
         }
     }
 }
+
+pub trait Scheduler {
+    fn tick(&mut self, event: &Event, bus: &mut EventBus);
+}