Skip to content
This repository has been archived by the owner on Jan 10, 2025. It is now read-only.

LSM6DSOX Consolidation #167

Open
crsz20 opened this issue Feb 13, 2024 · 0 comments
Open

LSM6DSOX Consolidation #167

crsz20 opened this issue Feb 13, 2024 · 0 comments
Assignees
@TheAditya123
Labels
Accelerometer Gyroscope refactor

Comments

@crsz20
Copy link
Member

crsz20 commented Feb 13, 2024

Description
The LSM6DSOX provides two sensing capabilities: accelerometer and gyroscope. Each of them have been implemented separately as two classes, despite being the same device. This is not ideal, as they both would be sharing access to the same I2C peripheral in hardware.

Consolidate them into one, single class that inherits both IAccelerometer and IGyroscope.

In the Sensor layer, should the source code be kept in the Accelerometer or the Gyroscope folder?
Neither. Instead, make a new IMU folder.

Definition of Done

  • Tested
  • Improvement goals have been met
  • Code reviewed
@crsz20 crsz20 added this to DAQ Feb 13, 2024
@crsz20 crsz20 moved this to 🔖 What's Next in DAQ Feb 13, 2024
@TheAditya123 TheAditya123 self-assigned this Sep 11, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@TheAditya123 TheAditya123

Labels
Accelerometer Gyroscope refactor
Projects
DAQ
Status: 🔖 What's Next
Milestone
No milestone
Development

When branches are created from issues, their pull requests are automatically linked.

167-lsm6dsox-consolidation DallasFormulaRacing/DataAcquisition2.0
2 participants
@TheAditya123 @crsz20