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

ROS 2に対応 #28

Merged
merged 24 commits into from
Jun 10, 2022
Merged

ROS 2に対応 #28

merged 24 commits into from
Jun 10, 2022

Conversation

ShotaAk
Copy link
Contributor

@ShotaAk ShotaAk commented Jun 1, 2022

What does this implement/fix?

ライブラリをROS 2向けにリリースするための変更です。

  • ライブラリをROS 2でビルドするためにCMakeLists.txtとpackage.xmlを追加します
    • パスの設定がうまくいかないところがあったので、srcディレクトリのCMakeListsの中身を、トップ階層のCMakeListsに移植しています
    • ユーザがライブラリを直接使うことを想定していないため、ドキュメントは最小限にしています
  • ライブラリが使えることを確認するためrt_manipulators_examplesパッケージを追加します
  • ROS 2環境でのビルド確認用にindustrial_ciをGitHub Workflowに追加します
    • foxy ~ humbleとrollingに対応させます
  • samplesディレクトリを削除します
    • サンプルのROS 2対応は実施しません
    • samplesディレクトリを残すとビルドエラーが発生するため削除しています

Does this close any currently open issues?

いいえ

How has this been tested?

ROS 2 Foxy環境でライブラリがビルドできること、FKサンプルが動作することを確認しました。

Foxy以外でのビルドはCIで確認しています。

Any other comments?

Checklists

@ShotaAk ShotaAk added the Type: Feature New Feature label Jun 1, 2022
rt_manipulators_examples/package.xml Outdated Show resolved Hide resolved
rt_manipulators_lib/package.xml Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
ShotaAk and others added 3 commits June 8, 2022 17:10
Co-authored-by: Daisuke Sato <tiryoh@gmail.com>
Co-authored-by: Daisuke Sato <tiryoh@gmail.com>
Co-authored-by: Daisuke Sato <tiryoh@gmail.com>
@ShotaAk ShotaAk marked this pull request as ready for review June 8, 2022 08:11
Co-authored-by: Daisuke Sato <tiryoh@gmail.com>
Copy link
Contributor

@Tiryoh Tiryoh left a comment

Choose a reason for hiding this comment

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

よさそうです。

@Tiryoh Tiryoh merged commit 719c621 into ros2 Jun 10, 2022
@Tiryoh Tiryoh deleted the feature/support_ros2 branch June 10, 2022 08:21
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.

None yet

2 participants