-
Notifications
You must be signed in to change notification settings - Fork 1
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
Fix to use dynamixel servo motors as head joints of SOAR #5
base: XM
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Offsetについてだけど、確か dynamixel側で homing offsetを設定すると0〜2piから-pi〜piに変更できると思う。hardware interface 側はどう扱っているの?
https://emanual.robotis.com/docs/en/software/dynamixel/dynamixel_wizard2/#modifying-control-values
後 ベースブランチは XMではなかったっけ? |
hardware_interface側では、homing offsetを考慮していないように見えます。具体的には、サーボ内部でhoming offsetによって補正された値をそのまま読み取る形になっており、値を書き込む際もhoming offsetの補正処理はサーボ側で行われているように見えます。実機で試してみます |
ベースブランチがXMである旨承知いたしました。マージ先を変更します |
実機で試したところ、homing offsetはサーボ側で補正されているようでした。ただ、-90deg~90degの範囲でしか設定できないため -pi ~ piにできなさそうです https://forum.robotis.com/t/dynamixel-set-zero-position/5317/4 |
https://forum.robotis.com/t/setting-homing-offset-in-dynamixel-wizard-2-0/1674 のリンクは何のリンク? -90~90について何も書いていない? |
すみません、リンクを間違っていました…正しくはこちらです |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- libdynamixelのレポジトリ・ブランチ何使っているか分からないので,READMEにメモするか,submoduleにしてもいい,CMakeLists.txtと共に楽にインストールしてもいい気がする.
-π to π→0 to 2π の部分は必要だっけ?サンプルの用にoffsetを設定すると-π to πにしても大丈夫だけど?
反応が遅くなり申し訳ございません.READMEに追加しつつCMakeLists.txtでインストールできるように修正したいと思います.来週までに対応いたします. |
@jsupratman13 |
ebeec5a
to
3506931
Compare
3506931
to
fc85dd5
Compare
libdynamixelは以下のブランチで動作可能です。
rebootをtorque enableに失敗したときのみ(失敗したらstatus validがfalseになるのかはわからないが)にしたので、 |
Summary
SOARの首パン軸、チルト軸でDynamixelを使用するための変更。
Detail
CMakeLists.txt
修正urdf_param_name
ros paramの初期値を/robot_description
に変更gear_ratio
をros paramに追加urdf
中のrising
を参照してオフセットするよう修正Impact
Test
Attention