WLED icon indicating copy to clipboard operation
WLED copied to clipboard

feat: KITT usermod effect

Open casesolved-co-uk opened this issue 1 year ago • 7 comments

v2 usermod for replicating the effect found on KITT, the car from the 80's TV show Knightrider. The original effect was in Red, however, the usermod also supports palettes

Has 4 controls:

  • Speed
  • Tail length
  • Delay before start
  • Dual

Similar to Sinelon but uses triangular wave not sine.

casesolved-co-uk avatar Feb 10 '24 01:02 casesolved-co-uk

That seems similar to K.I.T.T (Scanner and Scanner dual) existing effects.

blazoncek avatar Feb 10 '24 10:02 blazoncek

That seems similar to K.I.T.T (Scanner and Scanner dual) existing effects.

Yes, but there is no delay feature and this combines two effects into one

casesolved-co-uk avatar Feb 10 '24 16:02 casesolved-co-uk

Yes, but there is no delay feature and this combines two effects into one

I managed to test the effect but apart from the delay on one side of the scan it is no different than the mentioned Scanner. If you do want "delay" why not on both sides of the scan? I admit this might be personal preference.

I was also wondering about modulus (%) used in the effect. It is quite inefficient.

What if we incorporate the delay into original effect?

blazoncek avatar Feb 11 '24 18:02 blazoncek

In the original show it was quite common to see intermittent scans, but the scan always did a full loop not one way. Plus the effect gets quite tiring if it is continuous.

Optimisation vs maintenance. A single division operation per segment once every 23ms? How many microseconds are saved, 0.01us?

I'd have thought changing the behaviour of an existing effect would be the last thing to suggest for backward compatibility.

casesolved-co-uk avatar Feb 13 '24 21:02 casesolved-co-uk

Can we close this PR in favour of #3763 ? Or is this usermod still "sufficiently different" so to keep it open?

softhack007 avatar Apr 20 '24 11:04 softhack007

Can we close this PR in favour of #3763 ? Or is this usermod still "sufficiently different" so to keep it open?

I was going to add support for 2D matrices but since blaz doesn't seem to like collaboration maybe he can add it to his. I don't have time.

casesolved-co-uk avatar Apr 20 '24 14:04 casesolved-co-uk

blaz doesn't seem to like collaboration

I did not know this. Thanks for enlightening me.

blazoncek avatar Apr 20 '24 18:04 blazoncek