SlimeVR / SlimeVR-Tracker-ESP

SlimeVR tracker firmware for ESP32/ESP8266 and different IMUs
Apache License 2.0
840 stars 288 forks source link

Fix 9250 loop #246

Closed castlemccloud closed 1 year ago

castlemccloud commented 1 year ago

Previous refactoring of the code rolled both the assignment of the temp variable and the calibration matrix multiplication into a single loop.

This causes issue because every iteration of the loop for the matrix multiplication uses every value of temp, and it is not completely initialized until the final loop iteration.

The solution is to split the loop into two.

kitlith commented 1 year ago

Yep, this is:

I'll rebase it and apply a style tweak real quick.