Set and enforce velocity limits

[PeterBowman]

Currently unimplemented (ref). Once done, remember to call this setter from CanBusControlboard:

yarp-devices/libraries/YarpPlugins/CanBusControlboard/DeviceDriverImpl.cpp

Lines 163 to 170 in ec902e4

	if ( ! iPositionControl2Raw[i]->setRefAccelerationRaw( 0, refAccelerations.get(i).asDouble() ) )
	return false;
	if ( ! iPositionControl2Raw[i]->setRefSpeedRaw( 0, refSpeeds.get(i).asDouble() ) )
	return false;
	if ( ! iControlLimits2Raw[i]->setLimitsRaw( 0, mins.get(i).asDouble(), maxs.get(i).asDouble() ) )
	return false;

[PeterBowman]

Note: consume these values in our ICartesianControl implementations at kin-dyn and no longer support an angular velocity limit configuration parameter (ref, roboticslab-uc3m/kinematics-dynamics#121 (comment)).

[PeterBowman]

Make sure these limits will be taken into account in the IPositionDirect mode. In roboticslab-uc3m/kinematics-dynamics#159, a simple formula is proposed: delta_q/duration (where duration = ptModeMs).

[jgvictores]

Make sure these limits will be taken into account in the IPositionDirect mode. In roboticslab-uc3m/kinematics-dynamics#159, a simple formula is proposed: delta_q/duration (where duration = ptModeMs).

#173, right?

[PeterBowman]

Thanks, my bad, that link was meant to point at roboticslab-uc3m/kinematics-dynamics#173 (comment).

[PeterBowman]

If I'm correct, the iPOS drive does not check whether the motor exceeds a software-imposed speed value (as it does with positions and currents). So, perhaps, this should be done by us - either by introducing another thread in the background, or by listening to incoming event messages (#159 (comment)) - through frequent IEncoders::getEncoderSpeed calls (#189).

[PeterBowman]

We were only allowed to get velocity limits as parsed from the corresponding .ini files. This is useful for user applications, see roboticslab-uc3m/kinematics-dynamics#173. Implementing the set counterpart was trivial and has been done at 89cbdba.

[PeterBowman]

Once done, remember to call this setter from CanBusControlboard

Not doing. Reference speeds&accs as well as position&velocity limits are passed to each TechnosoftIpos instance and stored on device config. Then, CAN messages must be sent to the drives in order to load these values on each slave. There is no max-speed object in the CAN dictionary we can act upon, therefore a setVelLimits instruction as suggested by myself would be useless.

[PeterBowman]

Now, let's actually use these minVel/maxVel values! Should I ensure that reference speeds (refSpeed via .ini) are always equal or lower? Are there any other situations that would merit a velocity check (already dropped a note at #208)?

[PeterBowman]

Note: consume these values in our ICartesianControl implementations at kin-dyn and no longer support an angular velocity limit configuration parameter (ref, roboticslab-uc3m/kinematics-dynamics#121 (comment)).

Done at roboticslab-uc3m/kinematics-dynamics@a3600f7 and roboticslab-uc3m/kinematics-dynamics@89020af.

[PeterBowman]

ASWJ the minVels property would be removed and interpreted on the YARP-side (i.e. via IControlLimits getter and setter) as the negative of maxVels.

[PeterBowman]

Done at d7773a1, velocity limit checks are performed on device configuration, in IPositionControlRaw::setRefSpeedRaw and IVelocityMoveRaw::velocityMoveRaw.

[PeterBowman]

Added in PT mode at 7cac9f4 (experimental branch).