lukash / refloat

VESC package for self-balancing skateboards.
GNU General Public License v3.0
20 stars 14 forks source link

LEDs: add felony led effect #5

Closed acheronfail closed 3 months ago

acheronfail commented 4 months ago

My kids found this effect amusing from the old float package, so I ported it over to this one, too!

lukash commented 3 months ago

Apologies for the long turnaround.

I wanted to test this, and then I also thought about LED strips with an odd number of LEDs. The way it is one half of the effect will be longer. What about making the center LED always black to make it symmetric?

Also, feel free to squash the second commit to the first and when making changes just amend the outstanding commit, I don't need to see the differences in separate ones.

Lastly, would you add a line like this to your commit message, it's a git trailer and will ensure automatic changelog item (with an attribution to you):

Feature: Add the Felony LED effect.

Thank you!

acheronfail commented 3 months ago

I wanted to test this, and then I also thought about LED strips with an odd number of LEDs. The way it is one half of the effect will be longer. What about making the center LED always black to make it symmetric?

I have some test footage I can upload here if you'd like. Unfortunately I only have LED strips with even numbers of LEDs, so I can't test out odd variations. But I did like your suggestion, so I implemented it.

Also, feel free to squash the second commit to the first and when making changes just amend the outstanding commit, I don't need to see the differences in separate ones.

Lastly, would you add a line like this to your commit message [...]

Squashed, amended commit message with trailer and rebased on the latest trunk. :slightly_smiling_face:

acheronfail commented 3 months ago

You can test it easily, just configure the strip with one less LED 😀

Of course :man_facepalming:

I tested it (at a slow speed 0.2) with the strip set to 20 and 19 leds, as well as with different colours. You can see the results in these videos here:

https://github.com/user-attachments/assets/4eb50c5e-eefe-4444-9a79-f3df6f32af25

https://github.com/user-attachments/assets/9476b68c-815f-4f6f-ba9a-e00bfe6fe08f

lukash commented 3 months ago

Thank you!