I have bookmark folders like that as well hahah.
To later find out that I haven't even touched them years later 😉 

Dear Hans!

I'm so grateful! The code compiles without error. I'll check it out with the strip after lunch.

Thank you very much for the lightspeed response. Amazing! Thanks a lot.

Learning to program an Arduino and finding out things is fun, although my bookmark folder looks like the catalogue of the national library meanwhile 😉 

Have a nice weekend,

No problem! 

Calling Sparkle vs SparkleTimes requires an different number of parameters.

Sparkle(byte red, byte green, byte blue, int SpeedDelay);

SparkleTimed(byte red, byte green, byte blue, int SpeedDelay, int Seconds);

So when calling Sparkle, we need to pass 4 numbers: red, green, blue and the delay in milliseconds.

When calling SparkleTime, we need to pass 5 numbers: red, green, blue, delay in milliseconds, and seconds (how long the effect should run).

So in your code:

  if (analogValue < threshold) {
    digitalWrite(ledPin1, HIGH);
    SparkleTimed(0xff, 0xff, 0xff, 0, 5); // added "5" to make the sparkles run 5 seconds
  } else {
    digitalWrite(ledPin1, LOW);
  } 

This is a good experience though, since you'll get more familiar with possible error messages.
It basically says the same thing:

too few arguments to function 'void SparkleTimed(byte, byte, byte, int, int)'

Or in other words: you're missing at least one value, in this case that last "int" number.

Hope this helps 😊 

As the FIRE effect runs so pretty well, I wanted to use your recommendation about the time function with the SPARKLE effect.

So the LDR is triggered, the Strip sparkles for 10 seconds, the strip shuts off and can be triggered again.

Unfortunately I'm not able to get the code running. The compiler always gives the error message:

"too few arguments to function 'void SparkleTimed(byte, byte, byte, int, int)"

I tried the alternatives, which are still in the code but it didn't work.

Even after reading your tutorial about functions (Which, by the way, is superb!!!) I'm still clueless! 😣 

Your assistance would be appreciated a lot.

Here is my code:

/* LDR triggered LED Strip with WS2812B and LED effect "SPARKLE" (by Hans)
 *  1.03.2021
 */

#define FASTLED_INTERNAL // just used to mute the Pragma messages when compiling
#include "FastLED.h"

#define NUM_LEDS 99
CRGB leds;
#define PIN 6
const int analogPin = A0; // pin that the sensor is attached to
const int ledPin1 = A2; // pin that the green LED is attached to
const int ledPin2 = A1; // pin that the yellow LED is attached to
const int threshold = 800; // an arbitrary threshold level that's in the range of the analog input

void setup()
{
  FastLED.addLeds<WS2811, PIN, GRB>(leds, NUM_LEDS).setCorrection( TypicalLEDStrip );
  // initialize the LED pin as an output:
  pinMode(ledPin1, OUTPUT);
  pinMode(ledPin2, OUTPUT);
}

// *** REPLACE FROM HERE ***

void loop() {
  // read the value of the ldr:
  int analogValue = analogRead(analogPin);

  // if the analog value is high enough, turn on the LED2(yellow):
  if (analogValue > threshold) {
    digitalWrite(ledPin2, HIGH);
  } else {
    digitalWrite(ledPin2, LOW);
  }

  // if the analog value is low enough, turn on the LED1green:
  if (analogValue < threshold) {
    digitalWrite(ledPin1, HIGH);
    SparkleTimed(0xff, 0xff, 0xff, 0);
//     SparkleTimed();
//    SparkleTimed(byte red, byte green, byte blue, int SpeedDelay, int Seconds) //          runs SPARKLE if LDR is triggered
//  SparkleTimed(0xff, 0xff, 0xff, 0, byte red, byte green, byte blue, int SpeedDelay, int Seconds);

  } else {
    digitalWrite(ledPin1, LOW);
  } 

  delay(1); // delay in between reads for stability
}

void SparkleTimed(byte red, byte green, byte blue, int SpeedDelay, int Seconds) {
  unsigned long startTime = millis();
  unsigned long runMillis = 1000 * Seconds;

  while (millis() - startTime < runMillis) {
    Sparkle(red, green, blue, SpeedDelay);
  }
  fill_solid( leds, NUM_LEDS, CRGB(0,0,0));
  // fill_solid( leds, NUM_LEDS, CRGB::Black);
      FastLED.show();
}

void Sparkle(byte red, byte green, byte blue, int SpeedDelay) {
  int Pixel = random(NUM_LEDS);
  setPixel(Pixel,red,green,blue);
  showStrip();
  delay(SpeedDelay);
  setPixel(Pixel,0,0,0);
}
// *** REPLACE TO HERE ***

void showStrip() {
 #ifdef ADAFRUIT_NEOPIXEL_H
   // NeoPixel
   strip.show();
 #endif
 #ifndef ADAFRUIT_NEOPIXEL_H
   // FastLED
   FastLED.show();
 #endif
}

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
}

void setAll(byte red, byte green, byte blue) {
  for(int i = 0; i < NUM_LEDS; i++ ) {
    setPixel(i, red, green, blue);
  }
  showStrip();
}

Glad to hear it works as you wanted to! Nice!

And ... you're most welcome! Being able to help someone like this just makes my day 😊 

There's a big HEUREKA!!!

This line was just missing:

FastLED.show();

Now the sketch runs perfectly and I'm so glad!

Thanks to your help. Without you I would never have made it.

Dear Hans!

Again: Thanks for your help!

I have put the codeline in the void FireTimed... exactly at the place you suggested. But the strip still gets stuck.

I double checked if I did it like you suggested.

You're very welcome 😊 

To make the LEDs black after the 10 seconds, you can use

fill_solid( leds, NUM_LEDS, CRGB(0,0,0));

Like so:

void FireTimed(int Cooling, int Sparking, int SpeedDelay, int Seconds) {
  unsigned long startTime = millis();
  unsigned long runMillis = 1000 * Seconds;

  while (millis() - startTime < runMillis) {
    Fire(Cooling, Sparking, SpeedDelay);
  }
  
  fill_solid( leds, NUM_LEDS, CRGB(0,0,0));
}

This will set the entire strip to black (off).

Hope this fixes that little residue 😊 

You are the man!!! I wasn´t expecting an answer that soon. Thank you VERY much!

I just checked the sketch out:

It compiles without errors and runs perfectly after triggering the LDR.

The LDR can be triggered again after the 10 secs and it starts fine again. I´m very impressed!

There is just one small issue:

After the 10secs the Strip gets kind of stuck with some LEDs emitting white at the bottom and a few reds somewhere in the strip.

Maybe it´s neccessary to put all LEDs to low (or black) when the time has elapsed?

Your grateful

Thomas

Since the Arduino is not made to multitask, we'd need to check the LDR status in the while loop as well and act accordingly.
Let's first test this new function and then look at how to work with the LDR's.

Since you're using FastLED, we can cleanup your code a little as well - which includes the new function, but is not yet considering following marbles;

/* LDR triggered LED Strip with WS2812B and LED effect "Fire" (by Hans)
 *  20.02.2021
 */

#define FASTLED_INTERNAL // just used to mute the Pragma messages when compiling
#include "FastLED.h"

#define NUM_LEDS 60
CRGB leds;
#define PIN 6
const int analogPin = A0; // pin that the sensor is attached to
const int ledPin1 = A2; // pin that the green LED is attached to
const int ledPin2 = A1; // pin that the yellow LED is attached to
const int threshold = 800; // an arbitrary threshold level that's in the range of the analog input

void setup() {
  FastLED.addLeds < WS2811, PIN, GRB > (leds, NUM_LEDS).setCorrection(TypicalLEDStrip);

  // initialize the LED pin as an output:
  pinMode(ledPin1, OUTPUT);
  pinMode(ledPin2, OUTPUT);
}

void loop() {
  // read the value of the ldr:
  int analogValue = analogRead(analogPin);

  // if the analog value is high enough, turn on the LED2yellow:
  if (analogValue > threshold) {
    digitalWrite(ledPin2, HIGH);
  } else {
    digitalWrite(ledPin2, LOW);
  }

  // if the analog value is low enough, turn on the LED1green:
  if (analogValue < threshold) {
    digitalWrite(ledPin1, HIGH);
    FireTimed(55, 120, 15, 10); //          ruft die Funktion FIRE auf, wenn LDR unterbrochen

  } else {
    digitalWrite(ledPin1, LOW);
  } //  könnte ich hier die Funktion für FIRE aufrufen?

  delay(1); // delay in between reads for stability
}

void FireTimed(int Cooling, int Sparking, int SpeedDelay, int Seconds) {
  unsigned long startTime = millis();
  unsigned long runMillis = 1000 * Seconds;

  while (millis() - startTime < runMillis) {
    Fire(Cooling, Sparking, SpeedDelay);
  }
}

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);
    }

    FastLED.show();
    delay(SpeedDelay);
  }
}

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
    leds = CRGB(255, 255, heatramp);
  } else if (t192 > 0x40) { // middle
    leds = CRGB(255, heatramp, 0);
  } else { // coolest
    leds = CRGB(heatramp, 0, 0);
  }
}