Removed usage of copy array for a couple of effects#7
Open
alekseinovikov wants to merge 1 commit into
Open
Conversation
Author
|
Related issue: #6 |
Author
|
Существенный потребитель памяти: массив ledsX |
|
для качественного экономия памяти есть смысл взглянуть на https://alexgyver.ru/microled/ |
There was a problem hiding this comment.
Просто можно скопировать элемент leds[i] = leds[icw]
There was a problem hiding this comment.
а можно весь массив двинуть с помощью memmove
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Столкнулся с проблемой. Не хватало памяти для всего скетча с размером светодиодов = 300
Выпилил эффекты, которые мне не понравились, но проблему это не решило.
Нашел 2 эффекта, которые используют глобальный двумерный массив, переписал их без его использования.
В проекте все еще есть эффекты, которые используют его, но если их выпилить, тогда можно избавиться от ledsX массива. Это существенно экономит память и появляется возможность управлять намного большим количеством светодиодов (~400-500).
Мой пулл реквест - всего лишь один шаг по выпиливанию зависимости от двумерного массива и снижению потребления памяти. Просто решил поделиться, вдруг кто то тоже столкнется с подобным - будет меньше мароки.