Skip to content

SAMM Add CANdo For IMU, Magnetic Encoder, and ToF Messages#483

Merged
coderask merged 4 commits into
mainfrom
sensor_messages
May 19, 2026
Merged

SAMM Add CANdo For IMU, Magnetic Encoder, and ToF Messages#483
coderask merged 4 commits into
mainfrom
sensor_messages

Conversation

@coderask
Copy link
Copy Markdown
Contributor

@coderask coderask commented May 19, 2026 •
edited by dchansen06
Loading

Add IMU MAG & ToF data messages

Problem and Scope

needed for sensor migration.

Description

needed for sensor migration.

Gotchas and Limitations

Testing

  • HOOTL testing
  • HITL testing
  • Human tested

Testing Details

Larger Impact

Additional Context and Ticket

@dchansen06 dchansen06 requested a review from Shrav816 May 19, 2026 04:16
@dchansen06 dchansen06 added Documentation Improvements or additions to documentation Enhancement New feature or request 2 PRIORITY Important and a priority, but less than URGENT Small Fry Something that is small, could include bug fixes or smaller changes Sensors Something involved with or related to sensors Peripheral Related to or involving a peripheral including abstractions Boards Related to or involving any physical boards labels May 19, 2026
@dchansen06 dchansen06 added this to the GR26 milestone May 19, 2026
@dchansen06 dchansen06 changed the title added new sensor stuff SAMM Add CANdo For IMU, Magnetic Encoder, and ToF Data Messages May 19, 2026
@dchansen06 dchansen06 changed the title SAMM Add CANdo For IMU, Magnetic Encoder, and ToF Data Messages SAMM Add CANdo For IMU, Magnetic Encoder, and ToF Messages May 19, 2026
@dchansen06 dchansen06 self-requested a review May 19, 2026 04:17
@coderask coderask marked this pull request as ready for review May 19, 2026 04:22
dchansen06
dchansen06 approved these changes May 19, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@dchansen06 dchansen06 left a comment

Choose a reason for hiding this comment

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

Lgtm

@coderask coderask merged commit 16bde6a into main May 19, 2026
33 checks passed
@coderask coderask deleted the sensor_messages branch May 19, 2026 04:34
dchansen06 pushed a commit that referenced this pull request May 22, 2026
@coderask @github-actions @dchansen06
SAMM Add CANdo For IMU, Magnetic Encoder, and ToF Messages (#483)
195dd11 
# Add IMU MAG & ToF data messages

## Problem and Scope
needed for sensor migration. 

## Description
needed for sensor migration. 

## Gotchas and Limitations


## Testing

- [ ] HOOTL testing
- [ ] HITL testing
- [X] Human tested

### Testing Details


## Larger Impact


## Additional Context and Ticket

---------

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dchansen06 dchansen06 dchansen06 approved these changes

@Shrav816 Shrav816 Awaiting requested review from Shrav816

Assignees

@coderask coderask

Labels

2 PRIORITY Important and a priority, but less than URGENT Boards Related to or involving any physical boards Documentation Improvements or additions to documentation Enhancement New feature or request Peripheral Related to or involving a peripheral including abstractions Sensors Something involved with or related to sensors Small Fry Something that is small, could include bug fixes or smaller changes

Projects

None yet

Milestone

GR26

Development

Successfully merging this pull request may close these issues.

2 participants

@coderask @dchansen06