Skip to content

Commit

Permalink
parametric calibrator eth: bug fix due to new parking procedure
Browse files Browse the repository at this point in the history
fix error in loading startup position from HOME group
  • Loading branch information
Alberto Cardellino authored and icub committed Jun 25, 2019
1 parent 1fdfa8a commit 8e20550
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -368,15 +368,15 @@ bool parametricCalibratorEth::open(yarp::os::Searchable& config)
*/
xtmp = homeGroup.findGroup("positionHome");
if (xtmp.size()-1!=n_joints) {yError() << deviceName << ": invalid number of PositionHome params"; return false;}
legacyStartupPosition.positions.resize(n_joints);
legacyParkingPosition.positions.resize(n_joints);
for (i = 1; i < xtmp.size(); i++)
legacyStartupPosition.positions[i-1] = xtmp.get(i).asDouble();
legacyParkingPosition.positions[i-1] = xtmp.get(i).asDouble();

xtmp = homeGroup.findGroup("velocityHome");
if (xtmp.size()-1!=n_joints) {yError() << deviceName << ": invalid number of VelocityHome params"; return false;}
legacyStartupPosition.velocities.resize(n_joints);
legacyParkingPosition.velocities.resize(n_joints);
for (i = 1; i < xtmp.size(); i++)
legacyStartupPosition.velocities[i-1] = xtmp.get(i).asDouble();
legacyParkingPosition.velocities[i-1] = xtmp.get(i).asDouble();
}

// this parameter may be superseded by new park sequence mechanism, probably also for startup.
Expand Down

0 comments on commit 8e20550

Please sign in to comment.