Skip to content

BMI323 sfusion implementation - #345

Open
jojos38 wants to merge 12 commits into
SlimeVR:mainfrom
jojos38:main
Open

BMI323 sfusion implementation#345
jojos38 wants to merge 12 commits into
SlimeVR:mainfrom
jojos38:main

Conversation

@jojos38

@jojos38 jojos38 commented Sep 19, 2024

Copy link
Copy Markdown
Contributor

Working BMI323 at 200hz gyro and 200hz accel

Wasn't tested with two sensors as I don't have any but works fine on one sensor

Fix some microcontrollers crashing (Soft WDT reset) during the calibration loop
Fix some microcontrollers crashing (Soft WDT reset) during the calibration loop
Comment thread src/defines.h Outdated
Comment thread src/defines.h Outdated
@jojos38

jojos38 commented Sep 19, 2024

Copy link
Copy Markdown
Contributor Author

Done, I forgot to revert these sorry

@Eirenliel Eirenliel left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The libraries say that it's your copyright, but they look like they were taken from the Bosch's sources. What's the library source?

@jojos38

jojos38 commented Dec 11, 2024

Copy link
Copy Markdown
Contributor Author

The libraries say that it's your copyright, but they look like they were taken from the Bosch's sources. What's the library source?

I rewrote an optimized the code of the library, the library was originally written without OOP, I rewritten (only the required parts) with OOP and also optimized some parts to reduce memory usage and increase speed slightly

I kept the comments and the logic of their lib hence why it looks similar
Also my version is incomplete and only includes what's required for SlimeVR usage

Bosch source: https://github.com/[boschsensortec/BMI323_SensorAPI](https://github.com/boschsensortec/BMI323_SensorAPI)

@jojos38
jojos38 requested a review from loucass003 as a code owner January 25, 2025 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants