Unverified Commit 2a5c7657 authored by Florent Lamiraux's avatar Florent Lamiraux Committed by GitHub
Browse files

Merge pull request #159 from Toefinder/devel

fix: detect locked joint with more than 1 DOF
parents 90f41dbc 9c1f0dcf
Pipeline #19218 failed with stage
in 53 seconds
......@@ -304,7 +304,7 @@ ImplicitFunction::dependsOnRelPoseBetween(DeviceConstPtr_t robot) const {
// get the joints involved in the output config
JointConstPtr_t j1;
// check that output interval matches with the config range of one joint
if (outputConfIntervals_.nbRows() != 1) {
if (outputConfIntervals_.rows().size() != 1) {
return std::pair<JointConstPtr_t, JointConstPtr_t>(nullptr, nullptr);
segment_t row = outputConfIntervals_.rows()[0];
......@@ -437,18 +437,80 @@ BOOST_AUTO_TEST_CASE(equality) {
BOOST_CHECK(*functions[0] != *functions[1]); // only the names are equal
/* Create a robot with the following kinematic chain. All joints are
translations along x.
/Px \Px
joint_a0 joint_b0
|Px |Px
joint_a1 joint_b1
DevicePtr_t createRobot() {
std::string urdf(
"<robot name='test'>"
"<link name='base_link'/>"
"<link name='link_test_x'/>"
"<joint name='test_x' type='prismatic'>"
"<parent link='base_link'/>"
"<child link='link_test_x'/>"
"<limit effort='30' velocity='1.0' lower='-4' upper='4'/>"
"<link name='link_a0'/>"
"<link name='link_a1'/>"
"<joint name='joint_a0' type='prismatic'>"
"<parent link='link_test_x'/>"
"<child link='link_a0'/>"
"<limit effort='30' velocity='1.0' lower='-4' upper='4'/>"
"<joint name='joint_a1' type='prismatic'>"
"<parent link='link_a0'/>"
"<child link='link_a1'/>"
"<limit effort='30' velocity='1.0' lower='-4' upper='4'/>"
"<link name='link_b0'/>"
"<link name='link_b1'/>"
"<link name='link_b2'/>"
"<joint name='joint_b0' type='prismatic'>"
"<parent link='link_test_x'/>"
"<child link='link_b0'/>"
"<limit effort='30' velocity='1.0' lower='-4' upper='4'/>"
"<joint name='joint_b1' type='prismatic'>"
"<parent link='link_b0'/>"
"<child link='link_b1'/>"
"<limit effort='30' velocity='1.0' lower='-4' upper='4'/>"
"<joint name='joint_b2' type='floating'>"
"<parent link='link_b1'/>"
"<child link='link_b2'/>"
DevicePtr_t robot = Device::create("test");
loadModelFromString(robot, 0, "", "anchor", urdf, "");
return robot;
BOOST_AUTO_TEST_CASE(dependsOnRelPoseBetween) {
DevicePtr_t device = hpp::pinocchio::unittest::makeDevice(
JointPtr_t ee1 = device->getJointByName("lleg5_joint"),
ee2 = device->getJointByName("rleg5_joint");
DevicePtr_t device = createRobot();
JointPtr_t ee1 = device->getJointByName("joint_a1"),
ee2 = device->getJointByName("joint_b2");
// ensure that the joint indices are as expected
BOOST_REQUIRE(ee1->index() < ee2->index());
BasicConfigurationShooter cs(device);
Transform3f tf1(ee1->currentTransformation());
Transform3f tf2(ee2->currentTransformation());
......@@ -577,30 +639,13 @@ BOOST_AUTO_TEST_CASE(dependsOnRelPoseBetween) {
BOOST_CHECK_EQUAL(jointsConstrained.second->index(), ee1->index());
/// test the locked joint constraint as well
constraint = LockedJoint::create(ee1, ee1->configurationSpace()->neutral());
JointIndex parentIndex = Joint::index(ee1->parentJoint());
JointIndex index1, index2;
// ensure that index1 <= index2
if (parentIndex <= ee1->index()) {
index1 = parentIndex;
index2 = ee1->index();
} else {
index1 = ee1->index();
index2 = parentIndex;
constraint = LockedJoint::create(ee2, ee2->configurationSpace()->neutral());
std::cout << constraint->functionPtr()->name() << std::endl;
joints = constraint->functionPtr()->dependsOnRelPoseBetween(device);
if (index1 == 0) {
} else {
BOOST_CHECK_EQUAL(joints.first->index(), index1);
BOOST_CHECK_EQUAL(joints.second->index(), index2);
BOOST_CHECK_EQUAL(joints.first->index(), ee2->parentJoint()->index());
BOOST_CHECK_EQUAL(joints.second->index(), ee2->index());
// constraint fully locks the joint
jointsConstrained = constraint->doesConstrainRelPoseBetween(device);
if (index1 == 0) {
} else {
BOOST_CHECK_EQUAL(jointsConstrained.second->index(), index2);
BOOST_CHECK_EQUAL(jointsConstrained.second->index(), ee2->index());
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment