Fix color calculation in gridAddPixel for dynamic color shifting

techniccontroller · techniccontroller · commit 078400e3375f · 2025-01-02T18:29:57.000+01:00
diff --git a/ledmatrix.cpp b/ledmatrix.cpp
@@ -138,7 +138,7 @@ void LEDMatrix::setMinIndicator(uint8_t pattern, uint32_t color)
 void LEDMatrix::gridAddPixel(uint8_t x, uint8_t y, uint32_t color)
 {
   if(dynamicColorShiftActivePhase >= 0){
-    color = Wheel((uint16_t(x + y) * 256 * 2 / (WIDTH*HEIGHT) + dynamicColorShiftActivePhase) % 256);
+    color = Wheel((uint16_t(x + y*WIDTH) * 256 * 2 / (WIDTH*HEIGHT) + dynamicColorShiftActivePhase) % 256);
   }
   // limit ranges of x and y
   if(x < WIDTH && y < HEIGHT){

Original file line number	Diff line number	Diff line change
`@@ -138,7 +138,7 @@ void LEDMatrix::setMinIndicator(uint8_t pattern, uint32_t color)`
`138`	`138`	`void LEDMatrix::gridAddPixel(uint8_t x, uint8_t y, uint32_t color)`
`139`	`139`	`{`
`140`	`140`	`if(dynamicColorShiftActivePhase >= 0){`
`141`		`- color = Wheel((uint16_t(x + y) * 256 * 2 / (WIDTH*HEIGHT) + dynamicColorShiftActivePhase) % 256);`
	`141`	`+ color = Wheel((uint16_t(x + yWIDTH) 256 * 2 / (WIDTH*HEIGHT) + dynamicColorShiftActivePhase) % 256);`
`142`	`142`	`}`
`143`	`143`	`// limit ranges of x and y`
`144`	`144`	`if(x < WIDTH && y < HEIGHT){`