rainbowcomet: add missing arg initing base class

tlyu · tlyu · commit 58bc776678f8 · 2023-01-22T14:38:02.000-06:00
In RainbowComet, the call to `super().__init__()` was missing the
`background_color` argument. This caused strange effects when
non-default arguments were passed.

For example, setting `reverse=True` resulted a red comet with a
tail length of 1, and setting `bounce=True` resulted in a reversed
rainbow comet.

Signed-off-by: Taylor Yu &lt;code@argon.blue&gt;
diff --git a/adafruit_led_animation/animation/rainbowcomet.py b/adafruit_led_animation/animation/rainbowcomet.py
@@ -65,7 +65,7 @@ def __init__(
             self._colorwheel_step = step
         self._colorwheel_offset = colorwheel_offset
         super().__init__(
-            pixel_object, speed, 0, tail_length, reverse, bounce, name, ring
+            pixel_object, speed, 0, 0, tail_length, reverse, bounce, name, ring
         )
 
     def _set_color(self, color):

Original file line number	Diff line number	Diff line change
`@@ -65,7 +65,7 @@ def __init__(`
`65`	`65`	`self._colorwheel_step = step`
`66`	`66`	`self._colorwheel_offset = colorwheel_offset`
`67`	`67`	`super().__init__(`
`68`		`- pixel_object, speed, 0, tail_length, reverse, bounce, name, ring`
	`68`	`+ pixel_object, speed, 0, 0, tail_length, reverse, bounce, name, ring`
`69`	`69`	`)`
`70`	`70`
`71`	`71`	`def _set_color(self, color):`