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

Setup cob4-5 #469

Merged
merged 57 commits into from
Aug 10, 2016
Merged

Setup cob4-5 #469

merged 57 commits into from
Aug 10, 2016

Conversation

ipa-cob4-5
Copy link
Contributor

No description provided.

@ipa-cob4-5
Copy link
Contributor Author

The setup is still not complete, only the base components are tested.

@floweisshardt
Copy link
Member

@ipa-nhg: travis is failing due to roslaunch check failure. please help travis to get happy for this PR.

@ipa-nhg
Copy link
Member

ipa-nhg commented Apr 25, 2016

I think that the _ctest_cob_bringup_roslaunch-check_robots_cob4-5.xml fails because the cob_calibration_data pull request is still not merged: ipa320/cob_calibration_data#100 , and the file cob_calibration_data/cob4-5/calibration/calibration_offset.urdf.xacro is required to load the robot description.

Sorry, my bad, I forgot the comment : This pull request requires ipa320/cob_calibration_data#100

@floweisshardt
Copy link
Member

just restarted the Travis job as ipa320/cob_calibration_data#100 is merged now: see https://travis-ci.org/ipa320/cob_robots/builds/125587032

@floweisshardt
Copy link
Member

looks good now

@ipa-nhg
Copy link
Member

ipa-nhg commented May 13, 2016

if #476 and https://github.com/ipa320/cob_robots/pull/477/files are merged, the pull request ipa-cob4-5#1 have to be accepted

@@ -0,0 +1,20 @@
joint_names: ["torso_2_joint","torso_3_joint"]
Copy link
Member

Choose a reason for hiding this comment

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

cob4-5 hasn't got a torso

@floweisshardt
Copy link
Member

before merging this, we'll have to wait for the sick_3vistort release to be in the public repo, see http://repositories.ros.org/status_page/ros_indigo_default.html?q=sick_3vistort_driver

@floweisshardt
Copy link
Member

sick sensor is release and public now, so I'll retrigger travis

@floweisshardt
Copy link
Member

we need to provide a cob_docker_control substitute in cob_substitute before travis is happy

@floweisshardt
Copy link
Member

substitution for docking merged, retriggered travis: https://travis-ci.org/ipa320/cob_robots/builds/149988406

@floweisshardt
Copy link
Member

travis says that sick_3vistort_driver is not available (install tags missing). Will be available soon with new release. Short term fix is to add it as a rosinstall dependency.
Still we have to wait for https://github.com/SICKAG/sick_3vistort/pull/7 to be merged.

@floweisshardt
Copy link
Member

wow, finally Travis is happy!!!

@floweisshardt floweisshardt merged commit ef6d379 into ipa320:indigo_dev Aug 10, 2016
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.

5 participants