-
Notifications
You must be signed in to change notification settings - Fork 11
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
moveit_pyでロボットを動かすサンプルを追加します #164
Conversation
This reverts commit 7fdbb47.
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.
@chama1176
PRありがとうございます。
いくつかコメントしたのでご確認お願いします。
また、READMEのパッケージ一覧部分にsciurus17_examples_pyの記述を追加お願いします。
https://github.com/rt-net/sciurus17_ros/tree/ros2?tab=readme-ov-file#packages
sciurus17_examples_py/sciurus17_examples_py/pick_and_place_left_arm.py
Outdated
Show resolved
Hide resolved
sciurus17_examples_py/sciurus17_examples_py/pick_and_place_right_arm_waist.py
Outdated
Show resolved
Hide resolved
sciurus17_examples_py/sciurus17_examples_py/pick_and_place_right_arm_waist.py
Outdated
Show resolved
Hide resolved
sciurus17_examples_py/sciurus17_examples_py/pick_and_place_left_arm.py
Outdated
Show resolved
Hide resolved
Co-authored-by: Kuwamai <[email protected]>
Co-authored-by: Kuwamai <[email protected]>
Co-authored-by: Kuwamai <[email protected]>
Co-authored-by: Kuwamai <[email protected]>
…ht_arm_waist.py Co-authored-by: Kuwamai <[email protected]>
Co-authored-by: Kuwamai <[email protected]>
…t_arm.py Co-authored-by: Kuwamai <[email protected]>
…t_arm.py Co-authored-by: Kuwamai <[email protected]>
…ht_arm_waist.py Co-authored-by: Kuwamai <[email protected]>
…ht_arm_waist.py Co-authored-by: Kuwamai <[email protected]>
…t_arm.py Co-authored-by: Kuwamai <[email protected]>
@Kuwamai レビュー指摘事項修正しましたので、再確認お願いします 😺 |
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.
@chama1176 実機、シミュレータ両方で動作確認できました。パッケージ情報について少しコメントさせていただきましたのでご確認お願いします。
Co-authored-by: Kuwamai <[email protected]>
@chama1176 あとすみません!こちらのREADMEが更新できていればOKだと思います 🙇
|
@Kuwamai |
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.
@chama1176 ありがとうございます。LGTMです!
What does this implement/fix?
moveit_pyでロボットを動かすサンプルを追加します
Does this close any currently open issues?
How has this been tested?
mock componentsを用いてCppサンプルと同様に動くことを確認
Any other comments?
Checklists