Skip to content
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

Merged
merged 48 commits into from
Dec 25, 2024

Conversation

chama1176
Copy link

@chama1176 chama1176 commented Nov 28, 2024

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

@chama1176 chama1176 self-assigned this Nov 28, 2024
@chama1176 chama1176 marked this pull request as ready for review November 29, 2024 01:16
@chama1176 chama1176 requested a review from Kuwamai November 29, 2024 01:17
@Kuwamai Kuwamai added the Type: Feature New Feature label Dec 12, 2024
Copy link
Contributor

@Kuwamai Kuwamai left a 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/README.md Outdated Show resolved Hide resolved
sciurus17_examples_py/README.md Show resolved Hide resolved
sciurus17_examples_py/README.md Outdated Show resolved Hide resolved
sciurus17_examples_py/launch/example.launch.py Outdated Show resolved Hide resolved
sciurus17_examples_py/launch/example.launch.py Outdated Show resolved Hide resolved
chama1176 and others added 14 commits December 17, 2024 12:05
@chama1176 chama1176 requested a review from Kuwamai December 23, 2024 04:44
@chama1176
Copy link
Author

@Kuwamai レビュー指摘事項修正しましたので、再確認お願いします 😺

Copy link
Contributor

@Kuwamai Kuwamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chama1176 実機、シミュレータ両方で動作確認できました。パッケージ情報について少しコメントさせていただきましたのでご確認お願いします。

sciurus17_examples_py/package.xml Show resolved Hide resolved
sciurus17_examples_py/package.xml Outdated Show resolved Hide resolved
sciurus17_examples_py/package.xml Show resolved Hide resolved
@Kuwamai
Copy link
Contributor

Kuwamai commented Dec 24, 2024

@chama1176 あとすみません!こちらのREADMEが更新できていればOKだと思います 🙇

また、READMEのパッケージ一覧部分にsciurus17_examples_pyの記述を追加お願いします。
https://github.com/rt-net/sciurus17_ros/tree/ros2?tab=readme-ov-file#packages

@chama1176
Copy link
Author

@Kuwamai
コメント見逃してました。追加しました

Copy link
Contributor

@Kuwamai Kuwamai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@chama1176 ありがとうございます。LGTMです!

@Kuwamai Kuwamai merged commit 1976541 into ros2 Dec 25, 2024
1 check passed
@Kuwamai Kuwamai deleted the feature/moveit_py branch December 25, 2024 02:37
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.

2 participants