wpilibsuite / wpilibsuite/SystemcoreTesting
All yaw values from the onboard IMU are wrapped
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 186
- Forks
- 24
- Avg merge
- 8h 27m
- Merged PRs (30d)
- 9
Description
There are several places that the yaw value from the onboard IMU is published to the system NT server, but all of them currently wrap to a range of -180 to 180. Having access to the unwrapped yaw value would be valuable in many cases (for example, 6328 typically uses this value to calibrate the robot's wheel radius by spinning multiple times in place). I think this would be fully addressed by just publishing the yaw_flat, yaw_landscape, and yaw_portrait topics as unwrapped values. WPILib's API already provides separate methods that return the yaw as a double or Rotation2d depending on whether users want implicit wrapping.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Locate the code that publishes the yaw_flat, yaw_landscape, and yaw_portrait topics, then compare its use of WPILib's yaw APIs with the issue's distinction between double and Rotation2d. Done means these topics expose unwrapped yaw values, including across multiple rotations, with existing tests or verification updated as needed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- embedded-iot
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100