Super-Simple-RGB-WiFi-Lamp (Clock, Night rider and Bell curve error) about super-simple-rgb-wifi-lamp

[thebigpotatoe]

Hi Mate, thanks for coming over here. Will be much easier to deal with coding issues here than Instructables.

From the get go I think it looks like you have not set up the LED's properly in the sketch. As you state you are not using 66 LED's and have instead a 24 LED neo pixel ring I assume from Adafruit or such. Due to the new geometry and lower number of LED's used you will need to be a bit creative and change some of the code for it to work with all my modes.

As a simple overview, I believe what is happening is the modes are trying to address and LED that does not exist in the locations specified by the topLeds and bottomLeds arrays. This the causes the errors you see repeated as the light boots again and attempts to set the same mode causing it to crash again. If you would like to read a little more on arrays and why this is causing the error have a quick look at this.

If you want to use my code for everything else like the website, rainbow, and colour mode, The results will be the same. But for modes like bell curve and night rider, the specific LED's being addressed will be completely wrong.

So as a fix for your project I hope, try replacing lines 38 to 41 to;

int topLeds[] = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11};
int bottomLeds[] = {12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23};
int leftLeds[] = {};
int rightLeds[] = {};

This should map your LED's correctly and stop the crashing. To be sure there is nothing else wrong with the flash on the chip I recommend also erasing all the memory on the ESP when uploading as i have stated in the Instructable.

Let me know if this works,

[LordVallejo]

Hello Sir.
Thank you so much, the error was solved.
You are a very kind person for taking your time to teach and guide others.

Thanks a lot, it works fantastic

[thebigpotatoe]

Awesome glad it's fixed. Closing the issue.