PickNikRobotics / ros2_robotiq_gripper

BSD 3-Clause "New" or "Revised" License
50 stars 40 forks source link

Add robotiq f2 140 support #18

Closed JafarAbdi closed 11 months ago

JafarAbdi commented 1 year ago

Copied from git@github.com:ros-industrial/robotiq.git with small modifications to have the same structure as the codebase here

TODOs:

develiberta commented 1 year ago

I'm just waiting for this request to be merged.

JafarAbdi commented 1 year ago

I'm just waiting for this request to be merged.

I'll fix the conflicts and try to get this working this week

skpawar1305 commented 1 year ago

thank you gripper works with gen3

by adding

<?xml version="1.0"?>
<robot xmlns:xacro="http://wiki.ros.org/xacro" name="robotiq_gripper">
    <xacro:macro name="robotiq_gripper" params="
        name
        prefix
        parent
        *origin
        sim_ignition:=false
        sim_isaac:=false
        use_fake_hardware:=false
        fake_sensor_commands:=false
        include_ros2_control:=true
        com_port:=/dev/ttyUSB0">

        <!-- ros2 control include -->
        <xacro:include filename="$(find robotiq_description)/urdf/2f_140.ros2_control.xacro" />
        <!-- if we are simulating or directly communicating with the gripper we need a ros2 control instance -->
        <xacro:if value="${include_ros2_control}">
            <xacro:robotiq_gripper_ros2_control
                name="${name}" prefix="${prefix}"
                sim_ignition="${sim_ignition}"
                sim_isaac="${sim_isaac}"
                use_fake_hardware="${use_fake_hardware}"
                fake_sensor_commands="${fake_sensor_commands}"
                com_port="${com_port}"/>
        </xacro:if>

        <xacro:include filename="$(find kortex_description)/grippers/robotiq_2f_140/urdf/robotiq_arg2f_140_model_macro.xacro" />
        <xacro:robotiq_arg2f_140 parent="${parent}" prefix="${prefix}"/>
    </xacro:macro>
</robot>

to a new file robotiq_2f_140_macro.urdf.xacro and renaming gripper and finger_joint in gen3's launch file

codecov[bot] commented 11 months ago

Codecov Report

Merging #18 (cbed8b2) into main (743d20f) will not change coverage. Report is 5 commits behind head on main. The diff coverage is n/a.

:exclamation: Current head cbed8b2 differs from pull request most recent head cec006d. Consider uploading reports for the commit cec006d to get more accurate results

@@          Coverage Diff          @@
##            main     #18   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files          5       5           
  Lines        352     352           
=====================================
  Misses       352     352           
Flag Coverage Δ
unittests 0.00% <ø> (ø)

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

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more