-
Notifications
You must be signed in to change notification settings - Fork 96
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サンプルパッケージ追加 #203
moveit_pyサンプルパッケージ追加 #203
Conversation
Gazebo環境で |
Mock Componentsを用いて |
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.
コードレビューを行いました。コメントの確認をお願いいたします。
また、crane_x7_ros/README.md
とcrane_x7_ros/README.en.md
のPackages
の項目にcrane_x7_examples_py
パッケージの説明を追加する必要がありそうです。
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.
修正ありがとうございます。
LGTMです!
What does this implement/fix?
moveit_pyを使用したサンプルパッケージを追加します。
Sciurus17とできるだけ同じ実装になるようにしています。
製品間で異なる実装になっていないことも軽くご確認お願いします。
rt-net/sciurus17_ros#164
Does this close any currently open issues?
しません
How has this been tested?
パッケージ内のREADME記載の手順で各サンプルが動作することを確認します。
example.launch.pyを使用したサンプルは実機、mock components、Gazebo環境で動作することを確認します。
camera_example.launch.pyを使用したサンプルは実機のみで動作確認します。
Any other comments?
ないです。
Checklists