Skip to content

Add another moveit dependency#239

Merged
Yadunund merged 2 commits into
masterfrom
additional_moveit_dependency
Mar 24, 2022
Merged

Add another moveit dependency#239
Yadunund merged 2 commits into
masterfrom
additional_moveit_dependency

Conversation

@Yadunund
Copy link
Copy Markdown
Member

I missed another dependency, moveit-planners, in PR #235

Without this, launching the controller manager has some errors displayed.

[ERROR] [1647354067.549891762]: Exception while loading planner 'chomp_interface/CHOMPPlanner': According to the loaded plugin descriptions the class chomp_interface/CHOMPPlanner with base class type planning_interface::PlannerManager does not exist. Declared types are  ompl_interface/OMPLPlanner
Available plugins: ompl_interface/OMPLPlanner
[ERROR] [1647354067.556044888]: Failed to initialize planning pipeline 'chomp'.
[ERROR] [1647354067.615621238, 37.172000000]: Exception while loading planner 'pilz_industrial_motion_planner::CommandPlanner': According to the loaded plugin descriptions the class pilz_industrial_motion_planner::CommandPlanner with base class type planning_interface::PlannerManager does not exist. Declared types are  ompl_interface/OMPLPlanner
Available plugins: ompl_interface/OMPLPlanner
[ERROR] [1647354067.615665542, 37.172000000]: Failed to initialize planning pipeline 'pilz_industrial_motion_planner'.

Signed-off-by: Yadunund yadunund@openrobotics.org

Signed-off-by: Yadunund <yadunund@openrobotics.org>
@Yadunund Yadunund requested a review from crvogt March 16, 2022 09:58
@Yadunund
Copy link
Copy Markdown
Member Author

@crvogt would it be more appropriate to move these moveit dependencies into the package.xml file of the rexrov_oberon7_moveit package?

@crvogt
Copy link
Copy Markdown
Contributor

crvogt commented Mar 17, 2022

@Yadunund Thanks, yes, I think that the rexrov_oberon7_moveit package would be more appropriate.

Signed-off-by: Yadunund <yadunund@openrobotics.org>
@Yadunund
Copy link
Copy Markdown
Member Author

@crvogt done!

Copy link
Copy Markdown
Contributor

@crvogt crvogt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@Yadunund Yadunund merged commit d48e955 into master Mar 24, 2022
@Yadunund Yadunund deleted the additional_moveit_dependency branch March 24, 2022 09:19
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 this pull request may close these issues.

2 participants