Skip to content

TCPリンクを追加 #91

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

TCPリンクを追加 #91

wants to merge 4 commits into from

Conversation

YusukeKato
Copy link
Collaborator

@YusukeKato YusukeKato commented May 21, 2025

What does this implement/fix?

目標とする手先の位置をlink4からグリッパの中心に変更するために新しくTCPリンクを追加します。
また、TCPリンクをデフォルトとしてサンプルも変更します。

Does this close any currently open issues?

#80

How has this been tested?

実機、Gazebo、mock_componentですべてのサンプルが動くことを確認しました。

Any other comments?

私の環境だけかもしれませんが、実機動作中、controller_managerがあらかじめ設定している100Hzの周期に達していないという警告が出ました。
80Hzに設定するとこの警告が出ないことも確認できました。
使用しているPCに依存する可能性が高そうであるため修正は行っていません。

[ros2_control_node-5] [WARN] [1748243047.531917709] [controller_manager]: Overrun detected! The controller manager missed its desired rate of 100 Hz. The loop took 10.451418 ms (missed cycles : 2).

Checklists

@YusukeKato YusukeKato self-assigned this May 21, 2025
@YusukeKato YusukeKato added the Type: Feature New Feature label May 21, 2025
@YusukeKato YusukeKato marked this pull request as ready for review May 26, 2025 07:59
@YusukeKato YusukeKato requested a review from Kuwamai May 26, 2025 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant