increase to a value then decrease

[RavensFan]

---

@LandBelenky wrote:

If you want it to go forever, I'd use a while loop and set the condition to "continue if true" and use a constant false.

---

NO NO NO.

An infinite while loop is just bad programming.  The only way to stop your VI is to hit the abort button.

You should put a stop button control and wire that to the stop terminal of the while loop.

[blacksnow]

This is what I got.

Any suggestion to make it more efficient?

[Darin.K]

Ah 80's TV. Easy way up to 32 LEDs, 64 with a few modifications.

[altenbach]

---

blacksnow wrote:

Any suggestion to make it more efficient?

---

• The "initialize array" certainly does not belong inside the inner loop
• Since you are always writing a TRUE, you don't need to index out the false and invert it. 😉
• Since you never use the value in the shift register, there is no need to write to it. delete the shift register!
• ...

Here is my quick attempt. Do you really need to be able to change the numer of LEDs live? I assume not.

Modify as needed. 😉

[MoReese]

---

@LandBelenky wrote:

Surely we can make it prettier than that.

---

You mean prettier than your first attempt?