-
Notifications
You must be signed in to change notification settings - Fork 1
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
use only Twist not TwistStamped #11
Conversation
moriarty
commented
Jul 19, 2023
- This removes the TwistStamped. The Stamp information was unused.
- This makes it easier to use with teleop_twist_joy
- In the future we plan to add a TwistStamped version which can be configured
- This removes the TwistStamped. The Stamp information was unused. - This makes it easier to use with teleop_twist_joy - In the future we plan to add a TwistStamped version which can be configured Signed-off-by: Alex Moriarty <alex.moriarty@picknik.ai>
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.
Stamping
Codecov Report
@@ Coverage Diff @@
## main #11 +/- ##
=====================================
Coverage 0.00% 0.00%
=====================================
Files 1 1
Lines 56 56
=====================================
Misses 56 56
Flags with carried forward coverage won't be shown. Click here to find out more. 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@abake48 this hasn't been tested on hardware yet. |
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.
I was able to jog the arm on hardware using teleop_twsit_joy
and forwarding the twist commands from /cmd_vel
to /twist_controller/commands/
Perfect, I also tested this today:
I will update the documentation in ros2_kortex and re-release everything |
- Related to PickNikRobotics/picknik_controllers#11 - Use Twist instead of TwistStamped because it works better with existing tools - Adds example to use with teleop_twist_keyboard Signed-off-by: Alex Moriarty <alex.moriarty@picknik.ai>
- Related to PickNikRobotics/picknik_controllers#11 - Use Twist instead of TwistStamped because it works better with existing tools - Adds example to use with teleop_twist_keyboard Signed-off-by: Alex Moriarty <alex.moriarty@picknik.ai>