-
Notifications
You must be signed in to change notification settings - Fork 614
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
[wpimath] Add overloads for Transform2d and Transform3d #5757
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The following are missing:
- Transform3d constructor prototype and docs in header file
- Transform2d constructor prototype and docs in header file
- Transform2d constructor definition in source file
- Java version of Transform2d constructor overload
- Java version of Transform3d constructor overload
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The changes have been done and pushed.
It seems that PRs created from main branches are causing each other's CI runs to cancel when new changes are pushed. I'm waiting to retrigger this PR's CI until the conflicting CI finishes. (Note that a way to avoid this would be giving PR branches unique names.) |
Adds overloads for Transform2d() constructor to accept x, y, and heading and for Transform3d() to accept x, y, z and rotation as a shorthand for the normal constructors.
Completes issue #5748.