Softfusion cleanup #100

New Issue

MrUnknownDE · 2026-04-05T17:51:50+02:00

MrUnknownDE commented

2026-04-05 17:51:50 +02:00

Originally created by @gorbit99 on 4/23/2025

This PR cleans up some code from softfusionsensor.h and the surrounding classes

The three main differences are:

A new IMUConsts struct that handles most of the constexpr stuff instead of having to have that in the sfusion class. This also reduces code duplication.
A new DriverCallbacks struct, that stores the callbacks for the bulkRead method on the individual drivers. This should make it easier to extend this struct in the future and it gets rid of the templates on bulkRead
It moves some of the leftover calibration methods into SoftFusionCalibration, since that's where they were used only

*Originally created by @gorbit99 on 4/23/2025* This PR cleans up some code from softfusionsensor.h and the surrounding classes The three main differences are: - A new IMUConsts struct that handles most of the constexpr stuff instead of having to have that in the sfusion class. This also reduces code duplication. - A new DriverCallbacks struct, that stores the callbacks for the bulkRead method on the individual drivers. This should make it easier to extend this struct in the future and it gets rid of the templates on bulkRead - It moves some of the leftover calibration methods into SoftFusionCalibration, since that's where they were used only

MrUnknownDE closed this issue

2026-04-05 17:51:50 +02:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github/SlimeVR-Tracker-ESP#100