Does not compile with "valid" header file for other boards

[eppfel]

Hi there,

Saw the video somewhere on the web and now stumbled upon your post on r/FastLED. Awesome project. It already helped me to make changes to my current project. I was curious to try your code and but I have trouble compiling it and found some issues (maybe I should have split reporting them).

1. There seems to be a dependency to an easing library but I could not find out which. Can you illuminate me and maybe at that in the readme? #include <easing.h>

2. I'm pretty sure the header files without ATOMMATRIX flag do not compile at the moment, as some preprocessor variables are not wrapped in a condition. e.g. CRGB matrix_leds[ATOM_NUM_LED];

I might send a pull-request the next days when I made a successful build. But not today, yet.

[costyn]

Hi Felix, Thanks for your interest in my project! I thought the easing lib files would have been included in the source in Git. Weird I guess they were git excluded. In anycase I used this library https://github.com/warrenm/AHEasing/tree/master/AHEasing. I put the .c and .h files in the dir lib/easing/ I don't see it in the Platformio libraries (yet). So for different projects I use different header files with the configuration specific to that project. And yes, since the M5Stack Atom Matrix (https://m5stack.com/products/atom-matrix-esp32-development-kit ) has an WS2812 display, I use it to display the heart. I guess I should surround that code with #ifdef ATOMMATRIX so it isn't included in other projects. A pull request would be great, otherwise I'll update the code one of these days. Hope this helps. Thanks for the feedback! Kind regards, Costyn.

…

On Wed, 15 Jul 2020 at 17:14, Felix A. Epp ***@***.***> wrote: Hi there, Saw the video somewhere on the web and now stumbled upon your post on r/FastLED. Awesome project. It already helped me to make changes to my current project. I was curious to try your code and but I have trouble compiling it and found some issues (maybe I should have split reporting them). 1. There seems to be a dependency to an easing library but I could not find out which. Can you illuminate me and maybe at that in the readme? #include <easing.h> 2. I'm pretty sure the header files without ATOMMATRIX flag do not compile at the moment, as some preprocessor variables are not wrapped in a condition. e.g. CRGB matrix_leds[ATOM_NUM_LED]; I might send a pull-request the next days when I made a successful build. But not today, yet. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#3>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAPSTKHOKCSMTS4XVWVP24DR3XBTTANCNFSM4O2UIS7A> .