Refactor of sensors #470

Closed
opened 2026-04-05 17:53:02 +02:00 by MrUnknownDE · 0 comments
Owner

Originally created by @Vyolex on 12/14/2021

Refactor of the sensors code, tried to keep the substantial changes to a minimum to avoid breaking stuff. Nonetheless, a lot of code has changed and should be tested properly, only managed to test with the 6050s I have.

Added:

  • SensorFactory class responsible for the creation of the sensors
  • Ability to add second BNO055 (untested)
  • Ability to mix different IMU types (code from #48 by @carl-anders), refactored and extended with ability to have BNO's as secondary IMU (untested)

Changed:

  • Refactored sensor classes
  • Fixed some bugs
  • Accidentally auto formatted, sorry

Hopefully this helps a bit and thanks Moof to allow me to re-implement your code ❤️
Future work mentioned on their PR still applies.

*Originally created by @Vyolex on 12/14/2021* Refactor of the sensors code, tried to keep the substantial changes to a minimum to avoid breaking stuff. Nonetheless, a lot of code has changed and should be tested properly, only managed to test with the 6050s I have. **Added:** - `SensorFactory` class responsible for the creation of the sensors - Ability to add second BNO055 (untested) - Ability to mix different IMU types (code from #48 by @carl-anders), refactored and extended with ability to have BNO's as secondary IMU (untested) **Changed:** - Refactored sensor classes - Fixed some bugs - Accidentally auto formatted, sorry Hopefully this helps a bit and thanks Moof to allow me to re-implement your code ❤️ Future work mentioned on their PR still applies.
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github/SlimeVR-Tracker-ESP#470