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

Commit 6935658 issue with no controllers in list #58

Closed
jodle001 opened this issue Feb 16, 2024 · 5 comments · Fixed by #60
Closed

Commit 6935658 issue with no controllers in list #58

jodle001 opened this issue Feb 16, 2024 · 5 comments · Fixed by #60

Comments

@jodle001
Copy link
Contributor

@Yadunund I am unable to get the controller to be recognized with this change if I do a fresh install. Am I doing something wrong that you can tell?

See below and it says that it says that there are zero controllers in the list. If I revert this commit, it works as normal.

24jsXgS

@jodle001
Copy link
Contributor Author

6935658 this commit

@Yadunund
Copy link
Collaborator

Could you provide more information:

  • OS, version of ROS 2 (whether binary or source install)
  • versions of moveit2, joint_trajectory_controller
  • what launch commands you used
    This would help reproduce the behavior. It would also help if you could paste your entire terminal input into a gist and share the link.

@jodle001
Copy link
Contributor Author

@Yadunund
Copy link
Collaborator

So there are a few breaking changes between humble and iron and rolling binaries of ros2_controllers. I only tested with Iron and Rolling. I plan to fix the CI job in this repo later this week so I can investigate this further then. Maybe the workaround is to create a humble branch off the latest working commit.

@Yadunund
Copy link
Collaborator

Hi @jodle001

I think I found the issue. Could you try this PR #60?

I realized when I tested the commit that "broke" things, I had tested with a moveit config for a different robot which had the moveit_controllers.yaml file properly setup.

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

Successfully merging a pull request may close this issue.

2 participants