search

resibots / libdynamixel

C++ interface to the dynamixel actuators
GNU General Public License v2.0
9 stars 15 forks source link

Get current speed in meaningful unit #14

Closed dogoepp closed 7 years ago

dogoepp commented 8 years ago

Currently, we can only get the current speed as an integer. There should be something similar to get__current_position_angle to do the conversion to an other unit.

Question: Which is the unit we should use ? For information, Robotis's documentation gives rpm values.

costashatz commented 8 years ago

Question: Which is the unit we should use ? For information, Robotis's documentation gives rpm values.

I think we should stick to SI units (rad/s).

jbmouret commented 8 years ago

An angle should be in radians.

-- JBM

On Wed, Mar 16, 2016 at 10:27 AM Dorian Goepp notifications@github.com wrote:

Currently, we can only get the current speed as an integer. There should be something similar to get__current_position_angle to do the conversion to an other unit.

Question: Which is the unit we should use ? For information, Robotis's documentation gives rpm values.

— You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub https://github.com/resibots/libdynamixel/issues/14

costashatz commented 8 years ago

An angle should be in radians.

We are talking about speed/velocity. So it should be rad/s.

dogoepp commented 7 years ago

fixed with #21.