Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

corrected cameras for cob4-7 urdf #819

Merged
merged 2 commits into from
Nov 19, 2021
Merged

Conversation

ipa-foj
Copy link
Contributor

@ipa-foj ipa-foj commented Nov 18, 2021

ref https://github.com/mojin-robotics/cob4/issues/1986
ref #818

Copy link
Member

@fmessmer fmessmer left a comment

Choose a reason for hiding this comment

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

the realsense_d435 macro has different xacros args than the realsense_zr300 xacro

$ ( source /root/upstream_ws/install/setup.bash && cd /root/target_ws && catkin_test_results --verbose; )

Full test results for 'build/cob_hardware_config/test_results/cob_hardware_config/roslaunch-check_upload_robot__robot_cob4-7.launch.xml'

-------------------------------------------------

<testsuite errors="0" failures="1" name="roslaunch-check_upload_robot__robot_cob4-7.launch.xml" tests="1" time="1"><testcase classname="roslaunch.RoslaunchCheck" name="cob_hardware_config_upload_robot_launch" status="run" time="1"><failure message="roslaunch check [/root/target_ws/src/cob_robots/cob_hardware_config/upload_robot.launch] failed" type="" /></testcase><system-out>&lt;![CDATA[

[/root/target_ws/src/cob_robots/cob_hardware_config/upload_robot.launch]:

	Invalid &lt;param&gt; tag: Cannot load command parameter [robot_description]: command [/opt/ros/kinetic/lib/xacro/xacro --inorder '/root/target_ws/src/cob_robots/cob_hardware_config/robots/cob4-7/urdf/cob4-7.urdf.xacro'] returned with code [2]. 

Param xml is &lt;param command="$(find xacro)/xacro --inorder '$(arg pkg_hardware_config)/robots/$(arg robot)/urdf/$(arg robot).urdf.xacro'" name="robot_description"/&gt;

]]&gt;</system-out></testsuite>

-------------------------------------------------

Full test results for 'build/cob_hardware_config/test_results/cob_hardware_config/rosunit-cob4-7_test.xml'

-------------------------------------------------

<?xml version="1.0" encoding="utf-8"?>

<testsuite errors="0" failures="1" name="unittest.suite.TestSuite" tests="1" time="1.322"><testcase classname="__main__.TestUrdf" name="test_correct_format" time="1.3219"><failure type="AssertionError">cannot convert xacro. file: /root/target_ws/src/cob_robots/cob_hardware_config/robots/cob4-7/urdf/cob4-7.urdf.xacro

Output: 

Error: error: Invalid parameter "ros_topic" None None

when instantiating macro: realsense_d435 (/root/upstream_ws/install/share/cob_description/urdf/sensors/realsense_d435.urdf.xacro)

in file: /root/target_ws/src/cob_robots/cob_hardware_config/robots/cob4-7/urdf/cob4-7.urdf.xacro

  File "/usr/lib/python2.7/unittest/case.py", line 329, in run

    testMethod()

  File "/root/upstream_ws/install/lib/cob_description/test_urdf.py", line 47, in test_correct_format

    self.fail("cannot convert xacro. file: " + file_to_test + "\nOutput: " + out + "\nError: " + err)

  File "/usr/lib/python2.7/unittest/case.py", line 410, in fail

    raise self.failureException(msg)

</failure></testcase><system-out>&lt;![CDATA[

['/root/upstream_ws/install/lib/cob_description/test_urdf.py', '/root/target_ws/src/cob_robots/cob_hardware_config/robots/cob4-7/urdf/cob4-7.urdf.xacro', '--gtest_output=xml:/root/target_ws/build/cob_hardware_config/test_results/cob_hardware_config/rosunit-cob4-7_test.xml', '__name:=cob4-7_test', '__log:=/root/.ros/log/33287cda-4849-11ec-8355-0242ac110002/cob4-7_test-1.log']

5

testing /root/target_ws/src/cob_robots/cob_hardware_config/robots/cob4-7/urdf/cob4-7.urdf.xacro

]]&gt;</system-out><system-err>&lt;![CDATA[

]]&gt;</system-err></testsuite>

-------------------------------------------------

Full test results for 'build/cob_bringup/test_results/cob_bringup/roslaunch-check_robots_cob4-7.xml.xml'

-------------------------------------------------

<testsuite errors="0" failures="1" name="roslaunch-check_robots_cob4-7.xml.xml" tests="1" time="1"><testcase classname="roslaunch.RoslaunchCheck" name="cob_bringup_robots_cob4-7_xml" status="run" time="1"><failure message="roslaunch check [/root/target_ws/src/cob_robots/cob_bringup/robots/cob4-7.xml] failed" type="" /></testcase><system-out>&lt;![CDATA[

[/root/target_ws/src/cob_robots/cob_bringup/robots/cob4-7.xml]:

	while processing /root/target_ws/install/share/cob_hardware_config/upload_robot.launch:

Invalid &lt;param&gt; tag: Cannot load command parameter [robot_description]: command [/opt/ros/kinetic/lib/xacro/xacro --inorder '/root/target_ws/install/share/cob_hardware_config/robots/cob4-7/urdf/cob4-7.urdf.xacro'] returned with code [2]. 

Param xml is &lt;param command="$(find xacro)/xacro --inorder '$(arg pkg_hardware_config)/robots/$(arg robot)/urdf/$(arg robot).urdf.xacro'" name="robot_description"/&gt;

]]&gt;</system-out></testsuite>

-------------------------------------------------

build/cob_bringup/test_results/cob_bringup/roslaunch-check_robots_cob4-7.xml.xml: 1 tests, 0 errors, 1 failures, 0 skipped

build/cob_hardware_config/test_results/cob_hardware_config/roslaunch-check_upload_robot__robot_cob4-7.launch.xml: 1 tests, 0 errors, 1 failures, 0 skipped

build/cob_hardware_config/test_results/cob_hardware_config/rosunit-cob4-7_test.xml: 1 tests, 0 errors, 1 failures, 0 skipped

Summary: 96 tests, 0 errors, 3 failures, 0 skipped

The command ".industrial_ci/travis.sh" exited with 1.

Copy link
Member

@fmessmer fmessmer left a comment

Choose a reason for hiding this comment

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

also, the mount point of the different realsense are different....please check with roslaunch cob_hardware_config display_robot.launch robot:=cob4-7

and verify the cameras are mounted as they are on the real robot....if they are not you'll have to adjust the calibration_default.xacro of cob4-7 to translate and rotate the cameras accordingly

@ipa-foj
Copy link
Contributor Author

ipa-foj commented Nov 18, 2021

Adapted the args corresponding to the realsense_d435 urdf.

Also checked the mounting of the cameras, they seem to be mounted as in reality.

@fmessmer fmessmer merged commit 78167a9 into ipa320:kinetic_dev Nov 19, 2021
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.

2 participants