From 646d3369fbd6c9c46608fdadca9ed1400d0f2c84 Mon Sep 17 00:00:00 2001
From: Fabian Schlenz <mail@fabianonline.de>
Date: Thu, 30 May 2019 13:08:24 +0200
Subject: [PATCH] Amount of new Pixels per loop for confetti effect is now
 configurable. Also, the color is based on a slowly rotating baseHue.

---
 include/prototypes.h    | 2 ++
 src/effect_confetti.cpp | 5 ++++-
 src/pitrix.cpp          | 5 +++++
 3 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/include/prototypes.h b/include/prototypes.h
index 3f6fe6c..78ea87c 100644
--- a/include/prototypes.h
+++ b/include/prototypes.h
@@ -34,4 +34,6 @@ typedef struct Matrix {
   double a22;
 } Matrix;
 
+extern uint8_t baseHue;
+
 #endif
diff --git a/src/effect_confetti.cpp b/src/effect_confetti.cpp
index 9830ea1..3599d47 100644
--- a/src/effect_confetti.cpp
+++ b/src/effect_confetti.cpp
@@ -1,10 +1,13 @@
 #include "effect_confetti.h"
 #include "config.h"
 #include "functions.h"
+#include "prototypes.h"
 
 void ConfettiEffect::loop() {
 	fadeToBlackBy(leds, LED_COUNT, 1);
-	addPixelColor(random16(LED_COUNT), CHSV(random8(), 200, 255));
+	for (int i=0; i<EFFECT_CONFETTI_PIXELS_PER_LOOP; i++) {
+		addPixelColor(random16(LED_COUNT), CHSV(baseHue + random8(64), 200, 255));
+	}
 }
 
 boolean ConfettiEffect::can_be_shown_with_clock() { return true; };
diff --git a/src/pitrix.cpp b/src/pitrix.cpp
index f379aaf..c51f5b6 100644
--- a/src/pitrix.cpp
+++ b/src/pitrix.cpp
@@ -19,6 +19,7 @@
 uint8_t starting_up = OTA_STARTUP_DELAY;
 int loop_timeouts = 0;
 long loop_started_at = 0;
+uint8_t baseHue = 0; // defined as extern in prototypes.h
 
 void setup() {
   Serial.begin(74880);
@@ -52,6 +53,10 @@ void loop() {
   ntpClient.update();
   mqtt_loop();
 
+  EVERY_N_MILLISECONDS(100) {
+      baseHue++;
+  }
+
   EVERY_N_MILLISECONDS(1000 / FPS) {
     LOGln("Core * loop running");
     current_effect->loop();