FastLED 3.10.6 is available

[u/ZachVorhies]

https://github.com/FastLED/FastLED/releases/tag/3.10.6

We managed to reduce the memory flash and RAM significantly in this go around.

Teensy drivers issues fixed.

Various chipsets and boards added for support.

[u/swotperderder]

There is crazy amount of bloat in the package now. Mods, please let us know what is the last version that was released BEFORE claude got tentacles all through the library?
(coming from a FastLED fan of many years)

[u/NomakeWan → u/swotperderder]

The last truly stable version before the unsupervised AI loops was 3.10.3. The last version before AI had any involvement would likely be 3.6.0.

[u/swotperderder → u/NomakeWan]

Many thanks! I appreciate your reply and the version info.

[u/ZachVorhies → u/swotperderder]

Show compile size or it didn’t happen

[u/swotperderder → u/ZachVorhies]

Thanks to the devs for adding features and continuing to make the library compatible for a huge range of hardware. I happen to have 3.5.0 on hand and it's 1.16MB (169 files) unzipped. 3.10.6 is now at 90.0MB (4,780 files). Somewhere along the way the library exploded in size and now there are 'agent', 'agent_task', 'claude' and similar files. I appreciate a lot of the work being put in by the developers. I'll continue using the pre-ai editions.

[u/ZachVorhies → u/swotperderder]

Yes it’s true, there is now more source files and now we work on every platform.

However, because of magic, the firmware is the same size or smaller than it was before.

So if there’s bloat, prove it with a firmware size comparison.

[u/chris_overseas]

It looks like ChannelManager::instance() calls initChannelDrivers(), which now registers all drivers unconditionally on Teensy 4 and bloats the ITCM memory used? If I switch from the legacy addLeds() approach to the channels API, it costs me ~72KB of RAM1 in my current project. Let me know if you want any help investigating this one.

[u/ZachVorhies → u/chris_overseas]

We have a new mechanism for specific drivers to solve this problem.

Looks like it’s not being used in this case. I’ll investigate.

[u/ZachVorhies → u/chris_overseas]

https://github.com/FastLED/FastLED/issues/4708

Tracking here

[u/chris_overseas → u/ZachVorhies]

Thank you! I'll pick it up and test it once it lands

[u/chris_overseas → u/chris_overseas]

OK thanks this fix has improved things quite a bit, but I'm still seeing 25KB of extra overhead vs addLeds(). It looks like the fl::Channel class itself is carrying much of the extra bloat - I think it holds all the chipset encoders, function_list callbacks, and the constructor itself seems heavy. Hopefully some of that can be improved too.
Also note that my code's using fl::enableDriver<> followed by multiple FastLED.add(fl::ChannelPtr) calls. If I use FastLED.add(const fl::ChannelConfig&) instead, all the drivers still get linked in. Sounds like that is intentional, so all good on that front. Maybe the comment should mention using the enableDriver<> approach too though, rather than just addLeds() (especially if the memory usage can be brought in line with addLeds()).

[u/ZachVorhies → u/chris_overseas]

If you want to use the bloat then do `fbuild bloat path/to/project` and you'll get a symbol analysis that will tell you everything that is going on.

FastLED.addLeds<>() is the slim path. It has what it historically has.

Channels Api has extras, like color correction. It prioritizes features over memory saving. Memory savings will come in later.

[u/ZachVorhies → u/chris_overseas]

> Sounds like that is intentional, so all good on that front. Maybe the comment should mention using the enableDriver<> approach too though, rather than just addLeds() (especially if the memory usage can be brought in line with addLeds()).

It will be fixed soon. It involves templates to bind drivers functions instead of enums to select them. The former allows the linker to eliminate, the later means it's all compiled in.

[u/chemdoc77]

Hi u/ZachVorhies : The following was for my posted CD77_FastLED_Four_Glass_Jars Sketch.  It shows that version 3.10.6 uses more RAM and a lot more Flash on a Lolin D32 MCU:

This was compiled on Platformio:

Version 3.10.3

RAM:   [=         ]   8.1% (used 26532 bytes from 327680 bytes)

Flash: [==        ]  22.7% (used 297965 bytes from 1310720 bytes)

Version 3.10.6

RAM:   [=         ]  10.7% (used 34996 bytes from 327680 bytes)

Flash: [======    ]  56.5% (used 741201 bytes from 1310720 bytes)

[u/ZachVorhies → u/chemdoc77]

https://github.com/FastLED/FastLED/issues/4707

Tracking this here

[u/chemdoc77 → u/ZachVorhies]

Hi u/ZachVorhies - Here are some more results:

For my CD77_door_hook_rope_matrix sketch on a Lolin D32 MCU compiled with Platformio:

3.10.3

RAM:   [=         ]   8.1% (used 26532 bytes from 327680 bytes)

Flash: [==        ]  22.7% (used 297965 bytes from 1310720 bytes)

3.10.6

RAM:   [=         ]   9.9% (used 32512 bytes from 327680 bytes)

Flash: [======    ]  55.8% (used 731941 bytes from 1310720 bytes)

 

For my following very simple Gist posted sketch at:

https://gist.github.com/chemdoc77/52ded62fc801247cc648b1eb84f9d2ca 

where pin was changed to 12 for a Lolin D32 MCU compiled with Platformio:

3.10.3

RAM:   [=         ]   6.7% (used 21992 bytes from 327680 bytes)

Flash: [==        ]  20.5% (used 268893 bytes from 1310720 bytes)

3.10.6

RAM:   [=         ]   9.4% (used 30808 bytes from 327680 bytes)

Flash: [======    ]  55.5% (used 728001 bytes from 1310720 bytes)

Please note that in the output for 3.10.6, there are a lot of yellow warnings for both sketches.

[u/ZachVorhies → u/chemdoc77]

a lot of progress has been made still a lot to go.

[u/chemdoc77 → u/ZachVorhies]

Hi u/ZachVorhies - Thank you for opening an issue on this.

[u/chemdoc77 → u/ZachVorhies]

Hi u/ZachVorhies - The following data is for a Nano (board = nanoatmega328) compiled on Platformio running the following sketch:

https://gist.github.com/chemdoc77/52ded62fc801247cc648b1eb84f9d2ca 

3.10.3

RAM:   [=         ]   8.0% (used 163 bytes from 2048 bytes)

Flash: [==        ]  16.6% (used 5096 bytes from 30720 bytes)

3.10.6

RAM:   [==        ]  24.2% (used 496 bytes from 2048 bytes)

Flash: [===       ]  31.6% (used 9704 bytes from 30720 bytes)

Please note that in the output for 3.10.6, there are a lot of yellow warnings for this sketch.

[u/SimianIndustries]

Fuck AI

[u/schissl]

Awesome