Troy's project - Arduino

RE: Troy's project

Hans — Fri, 17 May 2019 06:11:40 +0000

Thanks Troy - it's always good to hear that, and definitely motivates me to keep doing what I'm doing

It reminds me a little bit about my University days. I wrote my thesis so normal folk would understand it as well (with my thesis presentation I had your peers sitting in, but the majority is family and friends). I really think information like that should be accessible by all people. And we should not "shield" our knowledge by keeping others out, either by poor attitude or by intentionally using language only a few really are familiar with. (I can keep ranting about that for another few hours hahah)

I do like what you're building - very cool! And yes I can see switches like that not being practical in that case.
Looking forward to seeing your end-product! And if something comes to mind I'll post it here right away.

Oh and of course; if you have question; you know where to find me.

RE: Troy's project

troy89 — Fri, 17 May 2019 06:03:44 +0000

Well I for one am extremely grateful that you're willing to keep your code open and share it with the rest of us. I wouldn't have been able to finish this project without your help. Come to think of it, I don't think I ever got a single question answered on the arduino forum. Just general statements that were of no help. Anyway, please keep on keeping on, this site is awesome! I'm not sure that a switch like that would work considering how many effects/colors there are. Although its a solid idea but not right for my particular project. The end result is going to be a small circuit board holding the pro trinket, and power boast board with the buttons and on/off switch. That will all be attached to the lipo as one unit and attached to the side of the goggle strap. These goggles are going to be for raves/festivals so I wanted to have a simple three button set up so that someone could easily switch between the sittings without too much hassle. I'm sure they'll be a huge hit in the music scene! Youtube is a great idea actually, i'll be sure to make a video and put in a plug for this site!

RE: Troy's project

Hans — Fri, 17 May 2019 05:30:11 +0000

Thanks Troy!

Yeah, I'm aware of those kind of people. It's a reason why I'm always wondering if I should or should not make code open source.
It's weird how some people read the question, complain and come with an answer nobody asked for.
I'm trying to keep my website clean of those kind of comments. I suppose it's easier to complain than to actually help or publish something people enjoy.
Oh well ... I know my code isn't perfect ... let them show something better ...

I'm still confused why the 2nd interrupt cause this much problems.
Maybe another alternative would be something like a
- multiposition switch to select an effect,
- potentiometer or another multiposition switch for color,
- a potentiometer for brightness,
- and "Apply" button that triggers the interrupt and reads the values of effect, color and brightness.

Anyhoo - sorry I couldn't be more helpful.
I'm curious what your project will look like in the end.
If you make a video of it (on YouTube for example), then please feel free to post the link here.

RE: Troy's project

troy89 — Fri, 17 May 2019 04:55:29 +0000

Actually that's what I did when I first started debugging these issues. The issues with the functions started as soon as I enabled a second interrupt button, and it didn't matter whether they were both external interrupts or pin change interrupts. I tried posting about this on the arduino forum as well but the people there aren't nearly as helpful as you are. Someone even asked if this code was joke code and that it was badly designed because I had interrupt buttons....all they seemed to do is reference 'blink without delay' or nitpick at the code design rather than actually answer any of my questions.

At this point I'm resigned to the fact that whatever is causing these issues is something that is above me. I'll keep attempting to debug but now I'm working on the physical aspect of this project which is PCB prototyping and circuit board design. I'm going to attempt to add some additional functions at a later date but for now this is the final version. I don't know if I ever mentioned but this code is for an 'infinity goggle' project that I think you'll find very cool/interesting. Once I get my prototype complete I'll be sure to share the finished result!

RE: Troy's project

troy89 — Fri, 17 May 2019 04:41:25 +0000

The “CRGB row1; // array used by colorWheel function” is used in the following code

void setColorWheelColors(byte gRed, byte gGreen, byte gBlue) {
  row1 = CRGB(gRed, gGreen, gBlue);
  row1 = CRGB(gRed, gGreen, gBlue);
  row1 = CRGB(gRed, gGreen, gBlue);
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB(gRed, gGreen, gBlue);
  row1 = CRGB(gRed, gGreen, gBlue);
  row1 = CRGB(gRed, gGreen, gBlue);
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
}

I believe you're looking at older code. Here is my current working version as I've made that ColorArray change.

Here is my working code:

#include 
#include "FastLED.h"
#include "EnableInterrupt.h" // must use this for internal interrupts
#define NUM_LEDS 20
#define PIN 6
#define EFFECT_BUTTON 3
#define COLOR_BUTTON 4
#define BRIGHTNESS_BUTTON 5
CRGB leds;
CRGB row1; // array used by colorWheel function
// Color       (HEX Code)          Array Position
// gGreen       (0x00, 0xFF, 0x00)  0,1,2
// gGreenCyan   (0x00, OxFF, 0x7F)  3,4,5
// Cyan        (0x00, 0xFF, 0xFF)  6,7,8
// gBlueCyan    (0x00, 0x7F, 0xFF)  9,10,11
// gBlue        (0x00, 0x00, 0xFF)  12,13,14
// gBlueMagenta (0x7F, 0x00, 0xFF)  15,16,17
// Magenta     (0xFF, 0x00, 0xFF)  18,19,20
// gRedMagenta  (0xFF, 0x00, 0x7F)  21,22,23
// gRed         (0xFF, 0x00, 0x00)  24,25,26
// Orange      (0xFF, 0x7F, 0x00)  27,28,29
// Yellow      (0xFF, 0xFF, 0x00)  30,31,32
// gGreenYellow (0x7F, 0xFF, 0x00)  33,34,35
// White       (0xFF, 0xFF, 0xFF)  36,37,38
byte ColorArray[] = {0x00, 0xFF, 0x00, 0x00, 0xFF, 0x7F, 0x00, 0xFF, 0xFF, 0x00, 0x7F, 0xFF, 0x00, 0x00, 0xFF, 0x7F, 0x00, 0xFF, 0xFF, 0x00, 0xFF, 0xFF, 0x00, 0x7F, 0xFF, 0x00, 0x00, 0xFF, 0x7F, 0x00, 0xFF, 0xFF, 0x00, 0x7F, 0xFF, 0x00, 0xFF, 0xFF, 0xFF};
byte gRed = ColorArray;
byte gGreen = ColorArray;
byte gBlue = ColorArray;
byte selectedEffect = 0;
byte selectedColor = 0;
byte selectedBrightness = 0;
void setup() {
  FastLED.addLeds(leds, NUM_LEDS).setCorrection( TypicalLEDStrip );
  pinMode (EFFECT_BUTTON, INPUT_PULLUP);  // internal pull-up resistor
  pinMode (COLOR_BUTTON, INPUT_PULLUP);
  pinMode (BRIGHTNESS_BUTTON, INPUT_PULLUP);
  enableInterrupt(EFFECT_BUTTON, changeEffect, CHANGE); // EFFECT_BUTTON pressed
  enableInterrupt(COLOR_BUTTON, changeColor, CHANGE); // COLOR_BUTTON pressed
  enableInterrupt(BRIGHTNESS_BUTTON, changeBrightness, CHANGE); // BRIGHTNESS_BUTTON pressed
  EEPROM.get(0, selectedEffect);
  EEPROM.get(1, selectedColor);
  EEPROM.get(2, selectedBrightness);
  if (selectedEffect > 12) {
    selectedEffect = 0;
    EEPROM.write(0, 0);
  }
  if (selectedColor > 12) {
    selectedColor = 0;
    EEPROM.write(1, 0);
  }
  if (selectedBrightness > 4) {
    selectedBrightness = 0;
    EEPROM.write(2, 0);
  }
}
void loop() {
  switch (selectedBrightness) {
    case 0: {
        LEDS.setBrightness(10);
        break;
      }
    case 1: {
        LEDS.setBrightness(25);
        break;
      }
    case 2: {
        LEDS.setBrightness(50);
        break;
      }
    case 3: {
        LEDS.setBrightness(100);
        break;
      }
    case 4: {
        LEDS.setBrightness(150);
        break;
      }
  }
  switch (selectedColor) {
    case 0: {
        // gGreen
        gRed = ColorArray;
        gGreen = ColorArray;
        gBlue = ColorArray;
        break;
      }
    case 1: {
        // gGreenCyan
        gRed = ColorArray;
        gGreen = ColorArray;
        gBlue = ColorArray;
        break;
      }
    case 2: {
        // Cyan
        gRed = ColorArray;
        gGreen = ColorArray;
        gBlue = ColorArray;
        break;
      }
    case 3: {
        // gBlueCyan
        gRed = ColorArray;
        gGreen = ColorArray;
        gBlue = ColorArray;
        break;
      }
    case 4: {
        // gBlue
        gRed = ColorArray;
        gGreen = ColorArray;
        gBlue = ColorArray;
        break;
      }
    case 5: {
        // gBlueMagenta
        gRed = ColorArray;
        gGreen = ColorArray;
        gBlue = ColorArray;
        break;
      }
    case 6: {
        // Magenta
        gRed = ColorArray;
        gGreen = ColorArray;
        gBlue = ColorArray;
        break;
      }
    case 7: {
        // gRedMagenta
        gRed = ColorArray;
        gGreen = ColorArray;
        gBlue = ColorArray;
        break;
      }
    case 8: {
        // gRed
        gRed = ColorArray;
        gGreen = ColorArray;
        gBlue = ColorArray;
        break;
      }
    case 9: {
        // Orange
        gRed = ColorArray;
        gGreen = ColorArray;
        gBlue = ColorArray;
        break;
      }
    case 10: {
        // Yellow
        gRed = ColorArray;
        gGreen = ColorArray;
        gBlue = ColorArray;
        break;
      }
    case 11: {
        // YellowgGreen
        gRed = ColorArray;
        gGreen = ColorArray;
        gBlue = ColorArray;
        break;
      }
    case 12: {
        // White
        gRed = ColorArray;
        gGreen = ColorArray;
        gBlue = ColorArray;
      }
  }
  switch (selectedEffect) {
    case 0  : { // color change
        fill_solid( leds, NUM_LEDS, CRGB(gRed, gGreen, gBlue) );
        FastLED.show();
        break;
      }
    case 1  : { // color change
        setColorWheelColors(gRed, gGreen, gBlue);
        // speed delay
        colorWheel(50);
        break;
      }
    case 2 : {
        // colorWipe - Color (red, green, blue), speed delay
        colorWipe(gRed, gGreen, gBlue, 35);
        colorWipe(0x00, 0x00, 0x00, 35); //turn leds off
        break;
      }
    case 3  : { // color change
        // CylonBounce - Color (gRed, gGreen, gBlue), eye size, speed delay, end pause
        CylonBounce(gRed, gGreen, gBlue, 3, 20, 0);
        break;
      }
    case 4  : { // color change
        // NewKITT - Color (gRed, gGreen, gBlue), eye size, speed delay, end pause
        NewKITT(gRed, gGreen, gBlue, 3, 20, 50);
        break;
      }
    case 5  : { // no color change
        // meteorRain - Color (gRed, gGreen, gBlue), meteor size, trail decay, random trail decay (true/false), speed delay
        meteorRain(0xff, 0xff, 0xff, 1, 80, true, 25);
        //meteorRain(gRed, gGreen, gBlue, 1, 80, true, 25);
        break;
      }
    case 6  : { // no color change
        // Fire - Cooling rate, Sparking rate, speed delay 55,120,15
        Fire(55, 120, 25);
        break;
      }
    case 7 : { // no color change
        // FadeInOut - Color (Red, Green, Blue)
        FadeInOut(0x00, 0x00, 0xFF); // Blue
        FadeInOut(0xFF, 0x7F, 0x00); // Orange
        FadeInOut(0x00, 0xFF, 0x00); // Green
        break;
      }
    case 8 : { // color change
        // Strobe - Color (gRed, gGreen, gBlue), number of flashes, flash speed, end pause
        Strobe(gRed, gGreen, gBlue, 10, 50, 750);
        break;
      }
    case 9 : { // color change
        // Sparkle - Color (gRed, gGreen, gBlue), speed delay
        Sparkle(gRed, gGreen, gBlue, 0);
        break;
      }
    case 10 : { // color change
        // Twinkle - Color (gRed, gGreen, gBlue), count, speed delay, only one twinkle (true/false)
        Twinkle(gRed, gGreen, gBlue, 10, 100, false);
        break;
      }
    case 11 : { // no color change
        // TwinkleRandom - twinkle count, speed delay, only one (true/false)
        TwinkleRandom(20, 100, false);
        break;
      }
    case 12 : { // no color change
        // rainbowCycle - speed delay
        rainbowCycle(20);
        break;
      }
  }
}
void changeEffect() {
  if (digitalRead (EFFECT_BUTTON) == HIGH) {
    selectedEffect++;
    EEPROM.put(0, selectedEffect);
    asm volatile ("  jmp 0");
  }
}
void changeColor() {
  if (digitalRead (COLOR_BUTTON) == HIGH) {
    selectedColor++;
    EEPROM.put(1, selectedColor);
    asm volatile ("  jmp 0");
  }
}
void changeBrightness() {
  if (digitalRead (BRIGHTNESS_BUTTON) == HIGH) {
    selectedBrightness++;
    EEPROM.put(2, selectedBrightness);
    asm volatile ("  jmp 0");
  }
}
// *************************
// ** LEDEffect Functions **
// *************************
void setColorWheelColors(byte gRed, byte gGreen, byte gBlue) {
  row1 = CRGB(gRed, gGreen, gBlue);
  row1 = CRGB(gRed, gGreen, gBlue);
  row1 = CRGB(gRed, gGreen, gBlue);
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB(gRed, gGreen, gBlue);
  row1 = CRGB(gRed, gGreen, gBlue);
  row1 = CRGB(gRed, gGreen, gBlue);
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
  row1 = CRGB::Black;
}
void colorWheel(int SpeedDelay) {
  for (int j = 0; j < NUM_LEDS; j++) {
    assignTo(j);
    //FastLED.show();
    showStrip();
    delay(SpeedDelay);
  }
}
//used by colorWheel
void assignTo(int j) {
  int i;
  for (i = 0; i < NUM_LEDS; i++)
  {
    leds = row1;
  }
}
void colorWipe(byte gRed, byte gGreen, byte gBlue, int SpeedDelay) {
  for (uint16_t i = 0; i < NUM_LEDS; i++) {
    setPixel(i, gRed, gGreen, gBlue);
    showStrip();
    delay(SpeedDelay);
  }
}
void meteorRain(byte gRed, byte gGreen, byte gBlue, byte meteorSize, byte meteorTrailDecay, boolean meteorRandomDecay, int SpeedDelay) {
  setAll(0, 0, 0);
  for (int i = 0; i < NUM_LEDS + NUM_LEDS; i++) {

    // fade brightness all LEDs one step
    for (int j = 0; j < NUM_LEDS; j++) {
      if ( (!meteorRandomDecay) || (random(10) > 5) ) {
        fadeToBlack(j, meteorTrailDecay );
      }
    }
    // draw meteor
    for (int j = 0; j < meteorSize; j++) {
      if ( ( i - j < NUM_LEDS) && (i - j >= 0) ) {
        setPixel(i - j, gRed, gGreen, gBlue);
      }
    }
    showStrip();
    delay(SpeedDelay);
  }
}
// used by meteorrain
void fadeToBlack(int ledNo, byte fadeValue) {
#ifdef ADAFRUIT_NEOPIXEL_H
  // NeoPixel
  uint32_t oldColor;
  uint8_t R, G, B;
  int value;
  oldColor = strip.getPixelColor(ledNo);
  R = (oldColor & 0x00ff0000UL) >> 16;
  G = (oldColor & 0x0000ff00UL) >> 8;
  B = (oldColor & 0x000000ffUL);
  R = (R <= 10) ? 0 : (int) R - (R * fadeValue / 256);
  G = (G <= 10) ? 0 : (int) G - (G * fadeValue / 256);
  B = (B <= 10) ? 0 : (int) B - (B * fadeValue / 256);
  strip.setPixelColor(ledNo, R, G, B);
#endif
#ifndef ADAFRUIT_NEOPIXEL_H
  // FastLED
  leds.fadeToBlackBy( fadeValue );
#endif
}
void NewKITT(byte gRed, byte gGreen, byte gBlue, int EyeSize, int SpeedDelay, int ReturnDelay) {
  RightToLeft(gRed, gGreen, gBlue, EyeSize, SpeedDelay, ReturnDelay);
  LeftToRight(gRed, gGreen, gBlue, EyeSize, SpeedDelay, ReturnDelay);
  OutsideToCenter(gRed, gGreen, gBlue, EyeSize, SpeedDelay, ReturnDelay);
  CenterToOutside(gRed, gGreen, gBlue, EyeSize, SpeedDelay, ReturnDelay);
  LeftToRight(gRed, gGreen, gBlue, EyeSize, SpeedDelay, ReturnDelay);
  RightToLeft(gRed, gGreen, gBlue, EyeSize, SpeedDelay, ReturnDelay);
  OutsideToCenter(gRed, gGreen, gBlue, EyeSize, SpeedDelay, ReturnDelay);
  CenterToOutside(gRed, gGreen, gBlue, EyeSize, SpeedDelay, ReturnDelay);
}
// used by NewKITT
void CenterToOutside(byte gRed, byte gGreen, byte gBlue, int EyeSize, int SpeedDelay, int ReturnDelay) {
  for (int i = ((NUM_LEDS - EyeSize) / 2); i >= 0; i--) {
    setAll(0, 0, 0);
    setPixel(i, gRed / 10, gGreen / 10, gBlue / 10);
    for (int j = 1; j <= EyeSize; j++) {
      setPixel(i + j, gRed, gGreen, gBlue);
    }
    setPixel(i + EyeSize + 1, gRed / 10, gGreen / 10, gBlue / 10);
    setPixel(NUM_LEDS - i, gRed / 10, gGreen / 10, gBlue / 10);
    for (int j = 1; j <= EyeSize; j++) {
      setPixel(NUM_LEDS - i - j, gRed, gGreen, gBlue);
    }
    setPixel(NUM_LEDS - i - EyeSize - 1, gRed / 10, gGreen / 10, gBlue / 10);
    showStrip();
    delay(SpeedDelay);
  }
  delay(ReturnDelay);
}
// used by NewKITT
void OutsideToCenter(byte gRed, byte gGreen, byte gBlue, int EyeSize, int SpeedDelay, int ReturnDelay) {
  for (int i = 0; i <= ((NUM_LEDS - EyeSize) / 2); i++) {
    setAll(0, 0, 0);
    setPixel(i, gRed / 10, gGreen / 10, gBlue / 10);
    for (int j = 1; j <= EyeSize; j++) {
      setPixel(i + j, gRed, gGreen, gBlue);
    }
    setPixel(i + EyeSize + 1, gRed / 10, gGreen / 10, gBlue / 10);
    setPixel(NUM_LEDS - i, gRed / 10, gGreen / 10, gBlue / 10);
    for (int j = 1; j <= EyeSize; j++) {
      setPixel(NUM_LEDS - i - j, gRed, gGreen, gBlue);
    }
    setPixel(NUM_LEDS - i - EyeSize - 1, gRed / 10, gGreen / 10, gBlue / 10);
    showStrip();
    delay(SpeedDelay);
  }
  delay(ReturnDelay);
}
// used by NewKITT
void LeftToRight(byte gRed, byte gGreen, byte gBlue, int EyeSize, int SpeedDelay, int ReturnDelay) {
  for (int i = 0; i < NUM_LEDS - EyeSize - 2; i++) {
    setAll(0, 0, 0);
    setPixel(i, gRed / 10, gGreen / 10, gBlue / 10);
    for (int j = 1; j <= EyeSize; j++) {
      setPixel(i + j, gRed, gGreen, gBlue);
    }
    setPixel(i + EyeSize + 1, gRed / 10, gGreen / 10, gBlue / 10);
    showStrip();
    delay(SpeedDelay);
  }
  delay(ReturnDelay);
}
// used by NewKITT
void RightToLeft(byte gRed, byte gGreen, byte gBlue, int EyeSize, int SpeedDelay, int ReturnDelay) {
  for (int i = NUM_LEDS - EyeSize - 2; i > 0; i--) {
    setAll(0, 0, 0);
    setPixel(i, gRed / 10, gGreen / 10, gBlue / 10);
    for (int j = 1; j <= EyeSize; j++) {
      setPixel(i + j, gRed, gGreen, gBlue);
    }
    setPixel(i + EyeSize + 1, gRed / 10, gGreen / 10, gBlue / 10);
    showStrip();
    delay(SpeedDelay);
  }
  delay(ReturnDelay);
}
void CylonBounce(byte gRed, byte gGreen, byte gBlue, int EyeSize, int SpeedDelay, int ReturnDelay) {
  for (int i = 0; i < NUM_LEDS - EyeSize - 2; i++) {
    setAll(0, 0, 0);
    setPixel(i, gRed / 10, gGreen / 10, gBlue / 10);
    for (int j = 1; j <= EyeSize; j++) {
      setPixel(i + j, gRed, gGreen, gBlue);
    }
    setPixel(i + EyeSize + 1, gRed / 10, gGreen / 10, gBlue / 10);
    showStrip();
    delay(SpeedDelay);
  }
  delay(ReturnDelay);
  for (int i = NUM_LEDS - EyeSize - 2; i > 0; i--) {
    setAll(0, 0, 0);
    setPixel(i, gRed / 10, gGreen / 10, gBlue / 10);
    for (int j = 1; j <= EyeSize; j++) {
      setPixel(i + j, gRed, gGreen, gBlue);
    }
    setPixel(i + EyeSize + 1, gRed / 10, gGreen / 10, gBlue / 10);
    showStrip();
    delay(SpeedDelay);
  }
  delay(ReturnDelay);
}
void RunningLights(byte gRed, byte gGreen, byte gBlue, int WaveDelay) {
  int Position = 0;
  for (int i = 0; i < NUM_LEDS * 2; i++)
  {
    Position++; // = 0; //Position + Rate;
    for (int i = 0; i < NUM_LEDS; i++) {
      // sine wave, 3 offset waves make a rainbow!
      //float level = sin(i+Position) * 127 + 128;
      //setPixel(i,level,0,0);
      //float level = sin(i+Position) * 127 + 128;
      setPixel(i, ((sin(i + Position) * 127 + 128) / 255)*gRed,
               ((sin(i + Position) * 127 + 128) / 255)*gGreen,
               ((sin(i + Position) * 127 + 128) / 255)*gBlue);
    }
    showStrip();
    delay(WaveDelay);
  }
}
void theaterChase(byte gRed, byte gGreen, byte gBlue, int SpeedDelay) {
  for (int j = 0; j < 10; j++) { //do 10 cycles of chasing
    for (int q = 0; q < 3; q++) {
      for (int i = 0; i < NUM_LEDS; i = i + 3) {
        setPixel(i + q, gRed, gGreen, gBlue);  //turn every third pixel on
      }
      showStrip();
      delay(SpeedDelay);
      for (int i = 0; i < NUM_LEDS; i = i + 3) {
        setPixel(i + q, 0, 0, 0);    //turn every third pixel off
      }
    }
  }
}
void Fire(int Cooling, int Sparking, int SpeedDelay) {
  static byte heat;
  int cooldown;
  // Step 1.  Cool down every cell a little
  for ( int i = 0; i < NUM_LEDS; i++) {
    cooldown = random(0, ((Cooling * 10) / NUM_LEDS) + 2);
    if (cooldown > heat) {
      heat = 0;
    } else {
      heat = heat - cooldown;
    }
  }
  // Step 2.  Heat from each cell drifts 'up' and diffuses a little
  for ( int k = NUM_LEDS - 1; k >= 2; k--) {
    heat = (heat + heat + heat) / 3;
  }
  // Step 3.  Randomly ignite new 'sparks' near the bottom
  if ( random(255) < Sparking ) {
    int y = random(7);
    heat = heat + random(160, 255);
    //heat = random(160,255);
  }
  // Step 4.  Convert heat to LED colors
  for ( int j = 0; j < NUM_LEDS; j++) {
    setPixelHeatColor(j, heat );
  }
  showStrip();
  delay(SpeedDelay);
}
//used by fire
void setPixelHeatColor (int Pixel, byte temperature) {
  // Scale 'heat' down from 0-255 to 0-191
  byte t192 = round((temperature / 255.0) * 191);
  // calculate ramp up from
  byte heatramp = t192 & 0x3F; // 0..63
  heatramp <<= 2; // scale up to 0..252
  // figure out which third of the spectrum we're in:
  if ( t192 > 0x80) {                    // hottest
    setPixel(Pixel, 255, 255, heatramp);
  } else if ( t192 > 0x40 ) {            // middle
    setPixel(Pixel, 255, heatramp, 0);
  } else {                               // coolest
    setPixel(Pixel, heatramp, 0, 0);
  }
}
void FadeInOut(byte gRed, byte gGreen, byte gBlue) {
  float R, G, B;
  for (int k = 0; k < 256; k = k + 1) {
    R = (k / 256.0) * gRed;
    G = (k / 256.0) * gGreen;
    B = (k / 256.0) * gBlue;
    setAll(R, G, B);
    showStrip();
  }
  for (int k = 255; k >= 0; k = k - 2) {
    R = (k / 256.0) * gRed;
    G = (k / 256.0) * gGreen;
    B = (k / 256.0) * gBlue;
    setAll(R, G, B);
    showStrip();
  }
}
void Strobe(byte gRed, byte gGreen, byte gBlue, int StrobeCount, int FlashDelay, int EndPause) {
  for (int j = 0; j < StrobeCount; j++) {
    setAll(gRed, gGreen, gBlue);
    showStrip();
    delay(FlashDelay);
    setAll(0, 0, 0);
    showStrip();
    delay(FlashDelay);
  }
  delay(EndPause);
}
void Sparkle(byte gRed, byte gGreen, byte gBlue, int SpeedDelay) {
  int Pixel = random(NUM_LEDS);
  setPixel(Pixel, gRed, gGreen, gBlue);
  showStrip();
  delay(SpeedDelay);
  setPixel(Pixel, 0, 0, 0);
}
void Twinkle(byte gRed, byte gGreen, byte gBlue, int Count, int SpeedDelay, boolean OnlyOne) {
  setAll(0, 0, 0);
  for (int i = 0; i < Count; i++) {
    setPixel(random(NUM_LEDS), gRed, gGreen, gBlue);
    showStrip();
    delay(SpeedDelay);
    if (OnlyOne) {
      setAll(0, 0, 0);
    }
  }
  delay(SpeedDelay);
}
void TwinkleRandom(int Count, int SpeedDelay, boolean OnlyOne) {
  setAll(0, 0, 0);
  for (int i = 0; i < Count; i++) {
    setPixel(random(NUM_LEDS), random(0, 255), random(0, 255), random(0, 255));
    showStrip();
    delay(SpeedDelay);
    if (OnlyOne) {
      setAll(0, 0, 0);
    }
  }
  delay(SpeedDelay);
}
void rainbowCycle(int SpeedDelay) {
  byte *c;
  uint16_t i, j;
  for (j = 0; j < 256 * 5; j++) { // 5 cycles of all colors on wheel
    for (i = 0; i < NUM_LEDS; i++) {
      c = Wheel(((i * 256 / NUM_LEDS) + j) & 255);
      setPixel(i, *c, *(c + 1), *(c + 2));
    }
    showStrip();
    delay(SpeedDelay);
  }
}
// used by rainbowCycle
byte * Wheel(byte WheelPos) {
  static byte c;
  if (WheelPos < 85) {
    c = WheelPos * 3;
    c = 255 - WheelPos * 3;
    c = 0;
  } else if (WheelPos < 170) {
    WheelPos -= 85;
    c = 255 - WheelPos * 3;
    c = 0;
    c = WheelPos * 3;
  } else {
    WheelPos -= 170;
    c = 0;
    c = WheelPos * 3;
    c = 255 - WheelPos * 3;
  }
  return c;
}



// ***************************************
// ** FastLed/NeoPixel Common Functions **
// ***************************************
// Apply LED color changes
void showStrip() {
#ifdef ADAFRUIT_NEOPIXEL_H
  // NeoPixel
  strip.show();
#endif
#ifndef ADAFRUIT_NEOPIXEL_H
  // FastLED
  FastLED.show();
#endif
}
// Set a LED color (not yet visible)
void setPixel(int Pixel, byte red, byte green, byte blue) {
#ifdef ADAFRUIT_NEOPIXEL_H
  // NeoPixel
  strip.setPixelColor(Pixel, strip.Color(red, green, blue));
#endif
#ifndef ADAFRUIT_NEOPIXEL_H
  // FastLED
  leds.r = red;
  leds.g = green;
  leds.b = blue;
#endif
}
// Set all LEDs to a given color and apply it (visible)
void setAll(byte red, byte green, byte blue) {
  for (int i = 0; i < NUM_LEDS; i++ ) {
    setPixel(i, red, green, blue);
  }
  showStrip();
}

RE: Troy's project

Hans — Fri, 17 May 2019 03:53:34 +0000

Just something I just noticed, not necessarily cause of any problems you're experiencing;
Maybe you overlooked the definition of the "ColorArray". I know I did
You've assigned it a uint32_t type, where it should be byte.
uint32_t ColorArray[] = {0x00, ... 0xFF}; // should be: BYTE
byte Red = ColorArray;
Maybe this was a leftover of earlier attempts where you wanted to store an color set (RGB = 32bit) as one value, versus separate red, green and blue (each a byte).
Since you assigned it an array of an integer type, the byte (for example 0x00) will be stored as a 32 bit integer (0x00000000).
Pulling an element as a byte, (byte Red = ColorArray;) will pull only the first byte of the 4 bytes you've stored.
Since the function theaterChase() is relatively "simple", I can't come up with scenarios where this is going sideways. Something else is interfering.
The odd part is that other, equally "simple" functions appear unaffected.
Another observation; I don't know where "CRGB row1; // array used by colorWheel function" came from ...
Is it a leftover from debugging? I'm not sure there is a need for it.
If I'd be debugging this code (I don't have my Arduino and LED strips handy), then I'd start by going back to the original code.
Sometimes it's good to start over (I do this quite often hahah).
Test if that works OK.
Then add the saving of selectedColor - you could even make a sketch that just toggles the EEPROM values, without any LED functions, just to see and follow what is happening there. Testing functions in small sketches, just for that specific function, is the way to go when things become complex or unexplainable things happen ...

RE: Troy's project

troy89 — Fri, 17 May 2019 00:43:50 +0000
I've been working on debugging theaterChase and i'm completely confused by the function. Regardless of how I call the function it still presents weird behavior. I tried all of the previously mentioned things but its not consistent on how it acts, which is rather odd. I think something in the function is conflicting with the code. Sometimes it will switch between cases, but usually it freezes on the part of the function that lights up every third led. However color changes will work with it, so if I load up the program while selectedEffect equals its case it will run and I can change colors. It breaks down on trying to switch between functions. Sometimes it would work if I pressed the button several times very fast but again, I can't get it to act consistently

RE: Troy's project

troy89 — Fri, 17 May 2019 00:01:32 +0000

Actually I think I just figured it out, I didn't realize that the second call of the function is turning the leds off!!!
colorWipe(0x00, 0x00, 0x00, 50);
so if I leave that line alone then the function works. I'm going to work on debugging the other two functions knowing what I know now, however they were acting different since they were freezing the program or causing the buttons to be unresponsive. I haven't tested those functions or the #define FASTLED_ALLOW_INTERRUPTS 0 with those functions yet so maybe there is still hope!

RE: Troy's project

troy89 — Thu, 16 May 2019 23:56:54 +0000

Ok so here is the weird thing, colorWipe works when I change the call from
case 12 : {
// colorWipe - Color (red, green, blue), speed delay
colorWipe(gRed, gGreen, gBlue, 50);
colorWipe(gRed, gGreen, gBlue, 50);
break;
}
to
case 12 : {
// colorWipe - Color (red, green, blue), speed delay
colorWipe(0x00, 0xff, 0x00, 50);
colorWipe(0x00, 0x00, 0x00, 50);
break;
}

I didn't change the function within the program, so the first call works, here is the function within my code that has it working:
void colorWipe(byte gRed, byte gGreen, byte gBlue, int SpeedDelay) {
for (uint16_t i = 0; i < NUM_LEDS; i++) {
setPixel(i, gRed, gGreen, gBlue);
showStrip();
delay(SpeedDelay);
}
So whatever is causing these functions to fail has to do with the function calls rather then the functions themselves, which would make sense given I haven't changed the functions. However I can't seem to understand the conflict regardless. The other functions work fine....

RE: Troy's project

troy89 — Thu, 16 May 2019 22:50:04 +0000
I changed my variables from volatile byte to byte but it didn't change how the EEPROM.put was overwriting the values. I had tried changing the addresses as well and still the same result. Forwhatever reason EEPROM.write seems to be the only thing that works.

I made the suggested changes using

#define FASTLED_ALLOW_INTERRUPTS 0
before the FastLED library but it didn't change anything. colorWipe still refuses to loop, it only plays once and then the LEDs just sit there full lit....very strange indeed