Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[example] Mecanum controller command example has incorrect wheel order #7493

Open
KangarooKoala opened this issue Dec 6, 2024 · 0 comments
Labels
type: bug Something isn't working.

Comments

@KangarooKoala
Copy link
Contributor

Describe the bug
In the drive subsystem of the mecanum controller command example, the order that the wheels are passed to the MecanumDriveWheelSpeeds and MecanumDriveWheelPositions constructors is wrong. It uses frontLeft, rearLeft, frontRight, rearRight, when the actual constructors use frontLeft, frontRight, rearLeft, rearRight.

Links to code

Expected behavior
The order in the example should be correct. (That is, the example should use frontLeft, frontRight, rearLeft, and rearRight)

Additional context
This would be a relatively low effort issue- Just swapping the order of two arguments in 4 places and double checking the other examples for any other bugs.

@KangarooKoala KangarooKoala added the type: bug Something isn't working. label Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working.
Projects
None yet
Development

No branches or pull requests

1 participant