Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ability to use more than 150 Leds #8

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

aleksandrbeljakov
Copy link

@aleksandrbeljakov aleksandrbeljakov commented Oct 25, 2021

Купил ленту на 5м, а вот включить её всю данной библиотекой не получилось.
Так как вдохновлён был именно этим автором, то вместе с автором #7
решил доделать оптимизацию этой библиотеки.
Оригинал был очень упрощённым и только в одном случае требовал хранение 3 элементов массива - предыдущее значение, текущее и следующее. Решается просто сдвигом значений в массиве.

Всё алгоритмы упрощены до простых итераций и полностью избавился от двумерного массива.

Результат сборки на 300 LED
Global variables use 1571 bytes (76%) of dynamic memory, leaving 477 bytes for local variables. Maximum is 2048 bytes.

- radom30 only algorithm required copy of near elements, solved by 3 elements arrays
@aleksandrbeljakov aleksandrbeljakov changed the title Ability to use more 150 Leds Ability to use more than 150 Leds Oct 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant