Jan/WLED_MM_Infinity
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ARC mapping optimization

Browse Source 
The biggest optimization was to avoid sin_t / cos_t.
Now let's try to help the compiler optimize the drawing loop.
This commit is contained in:
Frank
2024-02-25 19:46:53 +01:00
parent e0f08864a2
commit 8a0b97e0a8
1 changed files with 10 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
wled00/FX_fcn.cpp
View File

@@ -889,13 +889,19 @@ void IRAM_ATTR_YN Segment::setPixelColor(int i, uint32_t col) //WLEDMM: IRAM_ATT
else { else {
//WLEDMM: drawArc(0, 0, i, col); could work as alternative //WLEDMM: drawArc(0, 0, i, col); could work as alternative
float step = HALF_PI / (2.85f*i); //WLEDMM: some opimizations for the drawing loop
for (float rad = 0.0f; rad <= HALF_PI+step/2; rad += step) { float radius = float(i); // pre-calculate, for some speed
float step = HALF_PI / (2.85f * radius);
unsigned numSteps = 1 + ((HALF_PI + step/2.0f) / step); // pre-calculate, so the compiler can better optimize the for loop
float rad = 0.0f;
for (unsigned count = 0; count < numSteps; count++) {
// may want to try float version as well (with or without antialiasing) // may want to try float version as well (with or without antialiasing)
int x = roundf(sinf(rad) * i); int x = roundf(sinf(rad) * radius);
int y = roundf(cosf(rad) * i); int y = roundf(cosf(rad) * radius);
setPixelColorXY(x, y, col); setPixelColorXY(x, y, col);
rad += step;
} }
// Bresenham’s Algorithm (may not fill every pixel) // Bresenham’s Algorithm (may not fill every pixel)
//int d = 3 - (2*i); //int d = 3 - (2*i);
//int y = i, x = 0; //int y = i, x = 0;