microsoft / pxt-neopixel

A Neo-Pixel package for pxt-microbit
https://makecode.microbit.org/pkg/microsoft/pxt-neopixel
MIT License
58 stars 140 forks source link

ShowRainbow block is broken - sets all white #26

Closed nbogie closed 6 years ago

nbogie commented 6 years ago

The ShowRainbow block in the neopixel library seems to have broken in the last 3 weeks.

Simplest repeatable failing test: https://makecode.microbit.org/_aupazuXU3HiH

Further testing: If I switch to JS mode, open explorer, and change pxt.json to use “github:microsoft/pxt-neopixel#v0.6.2” instead of the 0.6.4 or 0.6.3 versions, the rainbow function behaves correctly again.

Test Platform: Chrome 67.0.3396.62 and Safari 11.1, both on OSX 10.13.4

nbogie commented 6 years ago

(While this block is not a primitive and arguably not essential, we find it is a very useful quickstart for our younger kids experimenting with neopixels before they progress to their own custom patterns.)

pelikhan commented 6 years ago

Fixed. Thank you for reporting,