Codecov Report

Merging #558 (0e82010) into master (e7f9962) will decrease coverage by 4.90%.
Report is 352 commits behind head on master.
The diff coverage is 25.91%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master     #558      +/-   ##
==========================================
- Coverage   35.78%   30.88%   -4.90%     
==========================================
  Files         189        7     -182     
  Lines       17570      832   -16738     
  Branches    11592      505   -11087     
==========================================
- Hits         6287      257    -6030     
+ Misses        994      133     -861     
+ Partials    10289      442    -9847     

Flags with carried forward coverage won't be shown. Click here to find out more.

... and 189 files with indirect coverage changes

I just had the time to test on our FR3 robot. And it still didn't solve the hold position issue. I didn't dive deep in debugging, but basically the robot successfully goes to the goal position and either free falls down or drags slowly sideways. We are only using effort commands. I will test today again.

To ensure that I understand you setup: Are you using effort command with active feedback gains of the PID controller?

I think I've found a possible issue, do you have set a goal_time_tolerance>0? If the time tolerance is exceeded, the PID controller will be deactivated even in hold position mode.

I didn't explicitly set a goal_time tolerance. So I guess this was not checked. I used the panda_moveit_config launch file in our franka_ros2 repo to test the new JTC. Maybe it's been implicitly set there but I doubt it.

I checked that now, false alarm. It could happen only if the PID can't hold the position (wrong tuning, effort limit,..), then it might enter a loop setting a new hold position again and again.

I think the problem is set_hold_position() is never get called in our case. When I add this else condition from motified_jtc, it works.

in our case goal_time is default 0 . And the set_hold_position is never get called when the robot reaches the target.

Do you use the topic or action interface? In case of the latter, do you get a success action result?

@BarisYazici do you have a public repo with which I could try your setup?

yes, here is the launch file I used to try JTC - https://github.com/frankaemika/franka_ros2/blob/humble/franka_moveit_config/launch/moveit.launch.py . All settings are default. I removed the old JTC in our repo and added the one in humble branch from ros2_controllers. Devcontainer in the repo should reflect my setup.

yes, here is the launch file I used to try JTC - https://github.com/frankaemika/franka_ros2/blob/humble/franka_moveit_config/launch/moveit.launch.py . All settings are default. I removed the old JTC in our repo and added the one in humble branch from ros2_controllers. Devcontainer in the repo should reflect my setup.

This PR hasn't been backported to humble yet, which means you were still testing the old version of JTC. Could you give it a try with this manual cherrypick please?

This PR hasn't been backported to humble yet, which means you were still testing the old version of JTC. Could you give it a try with this manual cherrypick please?

You are right, my mistake. I tested it, and it works as expected. We will remove the old JTC from our repo in the next update.

thanks for the feedback