⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 51.40%. Comparing base (77dde32) to head (85beca4).
Report is 1 commits behind head on humble.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##           humble    #3316      +/-   ##
==========================================
+ Coverage   51.40%   51.40%   +0.01%     
==========================================
  Files         382      382              
  Lines       31899    31896       -3     
==========================================
- Hits        16393    16392       -1     
+ Misses      15506    15504       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sea-bass

I think this may be causing an undesirable default behaviour.

Now if you don't specify the action namespace and default in the setup assitant, they won't be generated!

and planning execution won't work out of the box!

moveit_simple_controller_manager:
  controller_names:
    - arm_controller
    - eef_controller

  arm_controller:
    type: FollowJointTrajectory
    joints:
      - right_joint_1
      - right_joint_2
      - right_joint_3
      - right_joint_4
      - right_joint_5
      - right_joint_6
  eef_controller:
    type: FollowJointTrajectory
    joints:
      - right_joint_6

^ no action_ns added by default

[move_group-3] [ERROR] [1744412852.441687230] [move_group.moveit.moveit.plugins.simple_controller_manager]: No action namespace specified for controller `arm_controller` through parameter `moveit_simple_controller_manager.arm_controller.action_ns`
[move_group-3] [INFO] [1744412852.441806768] [move_group.moveit.moveit.plugins.simple_controller_manager]: Returned 0 controllers in list
[move_group-3] [INFO] [1744412852.441815254] [move_group.moveit.moveit.plugins.simple_controller_manager]: Returned 0 controllers in list

I just backported this from #2959, but good to know -- thanks!

Would be great if you could put up a PR to address this in both the unspecified and specified cases (probably needs some if-condition).

Or at least it would be appreciated if you put up a new Git issue so someone can see it and work on it.

@zacharyyamaoka I created #3434

@zacharyyamaoka I created #3434

Thank you!!! sorry for slow response on this. I did some more testing on this just now and I will post there