Skip to content

Conversation

@dyackzan
Copy link

@dyackzan dyackzan commented Apr 11, 2025

Only remaining thing here is to figure out the Non-recoverable Fault on startup if it's not too big of a lift. Edit: should be resolved now

Screenshot from 2025-04-11 08-28-31

@dyackzan dyackzan force-pushed the add-kinova-gen3-site-config branch from 91c7141 to 01291c0 Compare April 11, 2025 14:04
@dyackzan dyackzan marked this pull request as draft April 11, 2025 14:09
@dyackzan dyackzan force-pushed the add-kinova-gen3-site-config branch 2 times, most recently from 79d8436 to 6a4b898 Compare April 11, 2025 14:44
@dyackzan dyackzan requested a review from marioprats April 11, 2025 14:44
@dyackzan dyackzan marked this pull request as ready for review April 11, 2025 14:45
@dyackzan dyackzan force-pushed the add-kinova-gen3-site-config branch 2 times, most recently from 039cb9c to 722aa86 Compare April 11, 2025 17:10
- use_fake_hardware: "false"
- mock_sensor_commands: "false"
# When set to true, will route gripper comms through kinova and gripper_com_port is unused.
- use_internal_bus_gripper_comm: "false"

Choose a reason for hiding this comment

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

This flag doesn't work for me for some reason. I set it to true to use the internal bus, but then I get this error during bringup:

[ros2_control_node-2] [ERROR] [1744636670.523990026] [resource_manager]: Not acceptable command interfaces combination: 
[ros2_control_node-2] Start interfaces: 
[ros2_control_node-2] [
[ros2_control_node-2]   robotiq_85_left_knuckle_joint/position
[ros2_control_node-2] ]
[ros2_control_node-2] Stop interfaces: 
[ros2_control_node-2] [
[ros2_control_node-2] ]
[ros2_control_node-2] Not existing: 
[ros2_control_node-2] [
[ros2_control_node-2]  robotiq_85_left_knuckle_joint/position
[ros2_control_node-2] ]
[ros2_control_node-2] 
[ros2_control_node-2] [ERROR] [1744636670.524023554] [controller_manager]: Could not switch controllers since prepare command mode switch was rejected.
[ros2 run controller_manager spawner --controller-manager-timeout 180 robotiq_gripper_controller-3] [ERROR] [1744636670.524941085] [spawner_robotiq_gripper_controller]: Failed to activate controller : robotiq_gripper_controller

Copy link
Author

Choose a reason for hiding this comment

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

I was using the kortex_driver and kortex_description binaries before but I've added the updated versions of those packages as a submodule now (which is how they were pulled in for the picknik_ut kinova configs) so could you try with that update?

@marioprats
Copy link

I also see the fault pop up when opening the UI. I think it's because the fault_controller is not running, so this callback is never called, and therefore it doesn't set last_fault_status_update_ to now, and then reports the robot is always in fault. If we run the pstop node, then we need to start and activate the fault_controller as well.

@dyackzan dyackzan force-pushed the add-kinova-gen3-site-config branch 4 times, most recently from 78238d9 to 7e2de0d Compare April 15, 2025 14:32
@dyackzan
Copy link
Author

I also see the fault pop up when opening the UI. I think it's because the fault_controller is not running, so this callback is never called, and therefore it doesn't set last_fault_status_update_ to now, and then reports the robot is always in fault. If we run the pstop node, then we need to start and activate the fault_controller as well.

Gotcha, I added the fault controller back in and set it to activate at startup

@dyackzan dyackzan force-pushed the add-kinova-gen3-site-config branch 3 times, most recently from 56cffe7 to 666b143 Compare April 15, 2025 14:58
@dyackzan dyackzan requested a review from marioprats April 15, 2025 14:59
@dyackzan dyackzan force-pushed the add-kinova-gen3-site-config branch from 666b143 to 44b6020 Compare April 15, 2025 16:40
* Configure JTAC without FTS since by default kinova does not have an
  FTS.
* Add user permissions in Dockerfile to communicate with gripper
* Include support for robotiq gripper communication through kinova or
  external serial port.
* Include support for kinova with embedded wrist camera
* Add ros2_kortex submodule for kinova description & driver dependencies
@dyackzan dyackzan force-pushed the add-kinova-gen3-site-config branch from 44b6020 to 70a3f17 Compare April 15, 2025 16:44
@dyackzan
Copy link
Author

dyackzan commented Apr 15, 2025

Just worked through some incompatibilities between robotiq description and kortex description, should be good to test now
P.S. you will have to update submodules to test

@dyackzan dyackzan added this to the 8.0.0 milestone Apr 15, 2025
@marioprats
Copy link

I still see some issues in the Kinova I have.
The robotiq_gripper_controller still can't be activated for some reason
I'm also seeing faults when executing motion.

But I think we can merge this PR if this works for you. I can spend some time next week when I'm back to debug things on my end and iterate

@dyackzan
Copy link
Author

I'll go ahead and merge so I can add in the docs with links to these files. Planning on reconfiguring the office kinova tomorrow to debug the issues you're seeing @marioprats

@dyackzan dyackzan merged commit abb0b60 into main Apr 16, 2025
5 checks passed
@dyackzan dyackzan deleted the add-kinova-gen3-site-config branch April 16, 2025 21:32
shaur-k pushed a commit that referenced this pull request Sep 30, 2025
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.

3 participants