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

joint_effort_trajectory_controllers package is probably obsolete #5

Open
destogl opened this issue May 5, 2022 · 6 comments
Open

Comments

@destogl
Copy link

destogl commented May 5, 2022

Hi,

Friendly ros2_control maintainer here :) Recently we merged support for effort interfaces into joint_trajectory_controller from ros2_controllers repository. I think and (hope) this makes your joint_effort_trajectory_controllers obsolete. Maybe I am missing something, and you have a bit of different functionality of it which I didn't see scoping the code. Fell free to ask any questions about functionality or features on ros2_controllers repository.

P.S. Please don't hesitate to close this issue if you find it irrelevant.

@ntorresalberto
Copy link

I'm not sure if it's related but I just tried to build this against ros2 and I'm getting a lot of is private within this context from joint_effort_trajectory_controller files.

@BarisYazici
Copy link
Collaborator

As soon as we have this ros-controls/ros2_controllers#558 merged, we can migrate to the official JTC.

@VladimirFokow
Copy link

Hi, I've noticed that this issue can proceed - ros-controls/ros2_controllers#558 has been merged,
and backported to ros2_controllers (humble) in 2.30.0 (2023-12-20).

@TomCC7
Copy link

TomCC7 commented Jun 3, 2024

Hi, a stupid question here. How do I use the ros2_controller's joint_trajectory_controller package instead of the one in the repo? They share the same package name and even the controller name. Is there an elegant way to do that without modifying files in this repo?

@VladimirFokow
Copy link

@TomCC7 I'm not advanced myself but here's my suggestion:

  • delete (or add a file COLCON_IGNORE to) the joint_trajectory_controller folder of this repo

  • install ros2-controllers, e.g.:

sudo apt update && sudo apt install ros-${ROS_DISTRO}-ros2-control ros-${ROS_DISTRO}-ros2-controllers 
  • (remember to delete files that previous colcon build created (in build and install directories)), and
    rosdep install and colcon build

@TomCC7
Copy link

TomCC7 commented Jun 3, 2024

Thanks for replying! That certainly works and I'm currently using this method. Just don't want to configure this manually every time lol.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants