-
Notifications
You must be signed in to change notification settings - Fork 55
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
Pose Demo as ROS 2 Controller #172
Comments
Thank you@mhubii. I would like to try to do it on my own at first. If any problems in this process, I will ask for your help. I want to clarify the requirenments in this task, which I need to rewrite pose_control as ros2_controller(e.g https://github.com/lbr-stack/lbr_fri_ros2_stack/blob/dev-humble-cart-vel-demo/lbr_ros2_control/include/lbr_ros2_control/forward_lbr_position_command_controller.hpp) and consider pose limits(such as position limits and velocity limits of the robot) simultaneously. Is it right? |
that is correct. The goal would be for the user to run your ros2 launch lbr_bringup bringup.launch.py ctrl:=pose_control or similar. Writing a |
Hello, @mhubii. I feel sorry for that I did not have enough time to do this task two weeks ago, because I was busy for my course and lab works. When I got ready to process it last week, I found you had done it. Great jobs, thank you. |
hey @BoWangFromMars , thanks for returning to this. No worries at all! So your demo is still a node, but ideally would be a ros2_controller. This hasn't been done yet. |
okay @BoWangFromMars , formatting mostly finished. Unfortunately the intended use of this repository is less documented atm, but you can find an explanation for your demo here:
https://lbr-stack.readthedocs.io/en/latest/lbr_fri_ros2_stack/lbr_demos/lbr_demos_fri_ros2_advanced_cpp/doc/lbr_demos_fri_ros2_advanced_cpp.html#pose-controller
this leaves me with 2 questions:
pose_control
should ideally be aros2_controller
a la https://github.com/lbr-stack/lbr_fri_ros2_stack/blob/dev-humble-cart-vel-demo/lbr_ros2_control/include/lbr_ros2_control/forward_lbr_position_command_controller.hppwould you be interested in turning the
pose_control
into aros2_controller
? I could assist you, otherwise I could do that myself. Let me know. We should open a new PR for that though.Originally posted by @mhubii in #170 (comment)
The text was updated successfully, but these errors were encountered: