Skip to content

Commit

Permalink
Don't assume gripper controller for single joint control in MoveIt Se…
Browse files Browse the repository at this point in the history
…tup Assistant (backport #2555) (#2560)

* For single joint controllers which are not gripper controllers, still output joints list

* Use OR

* Only check for GripperActionController

Co-authored-by: Sebastian Jahr <sebastian.jahr@picknik.ai>

---------

Co-authored-by: Sebastian Jahr <sebastian.jahr@picknik.ai>
(cherry picked from commit 81094a6)

Co-authored-by: Forrest Rogers-Marcovitz <39061824+forrest-rm@users.noreply.github.com>
  • Loading branch information
mergify[bot] and forrest-rm committed Dec 15, 2023
1 parent f64b202 commit e3b91cd
Showing 1 changed file with 1 addition and 1 deletion.
Expand Up @@ -192,7 +192,7 @@ bool ROS2ControllersConfig::GeneratedControllersConfig::writeYaml(YAML::Emitter&
emitter << YAML::Value;
emitter << YAML::BeginMap;
{
if (ci.joints_.size() != 1)
if (ci.type_ != "position_controllers/GripperActionController")
{
emitter << YAML::Key << "joints" << YAML::Value << ci.joints_;
}
Expand Down

0 comments on commit e3b91cd

Please sign in to comment.