Skip to content

ROS 2 Lyrical対応 - #87

Merged
KuraZuzu merged 9 commits into
ros2from
feature/support-lyrical
Sep 15, 2026
Merged

KuraZuzu merged 9 commits into
ros2from
feature/support-lyrical

Conversation

@KuraZuzu

@KuraZuzu KuraZuzu commented Aug 28, 2026 •

Copy link
Copy Markdown
Contributor

What does this implement/fix?

ROS 2 Lyricalに対応します。

主な変更点は以下のとおりです。

  • Lyricalが推奨するModern Cmakeへの対応
  • CIの更新
  • README.mdの更新

Does this close any currently open issues?

しません。

How has this been tested?

Gazebo上で速度指令が反映されることを確認しました。

# Gazebo起動
ros2 launch raspimouse_gazebo raspimouse_with_emptyworld.launch.py

# 速度指令
ros2 topic pub --rate 10  /cmd_vel  geometry_msgs/msg/TwistStamped   "{header: auto, twist: {linear: {x: 0.0}, angular: {z: 0.0}}}"

本パッケージをビルドするには、下記の依存パッケージもソースビルドをする必要があります。

Any other comments?

  • 本PRがマージされた後、CHANGELOG.rstやpackage.xmlのバージョン情報を更新するPRを別途出します。

Checklists

@KuraZuzu
KuraZuzu requested a review from YusukeKato August 28, 2026 09:12
@YusukeKato

Copy link
Copy Markdown
Contributor

raspimouse_ros2_examplesのサンプルが動かせないようです。

Gazebo起動時に一緒に起動させているraspimouseの模擬ノードがエラーで落ちてしまっているようです。
確認をお願いいたします。

[component_container_mt-8] /opt/ros/lyrical/lib/rclcpp_components/component_container_mt: symbol lookup error: /home/kato/ROS2/rpm_ws/install/raspimouse_fake/lib/libfake_raspimouse_component.so: undefined symbol: _ZN22rosidl_typesupport_cpp31get_service_type_support_handleIN8std_srvs3srv7SetBoolEEEPK29rosidl_service_type_support_tv
[ERROR] [component_container_mt-8]: process has died [pid 24087, exit code 127, cmd '/opt/ros/lyrical/lib/rclcpp_components/component_container_mt --ros-args -r __node:=fake_raspimouse_container -r __ns:=/ -p use_sim_time:=True -p use_sim_time:=True'].

@KuraZuzu

KuraZuzu commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

動作確認ありがとうございます。
上記の不具合は、Modern CMake化の際にstd_srvsの型サポートがリンク対象から漏れていたことが原因だったため修正しました。
raspimouse_ros2_examplesのサンプルの動作も確認済みです。

@YusukeKato YusukeKato left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

サンプルの動作確認できました。
一箇所対応いただければ、その他問題ないかと思います。

Comment thread README.en.md Outdated

```bash
sudo apt install ros-jazzy-raspimouse-sim
sudo apt install ros-$ROS_DISTRO-raspimouse

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
sudo apt install ros-$ROS_DISTRO-raspimouse
sudo apt install ros-$ROS_DISTRO-raspimouse-sim

英語のREADMEも-simが抜けていそうです。

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

修正しました。

@KuraZuzu

Copy link
Copy Markdown
Contributor Author

@YusukeKato
ご指摘箇所を修正しましたのでご確認をお願いします。

@KuraZuzu
KuraZuzu requested a review from YusukeKato September 14, 2026 05:03
@KuraZuzu KuraZuzu self-assigned this Sep 14, 2026
@KuraZuzu KuraZuzu added the Type: Feature New Feature label Sep 14, 2026

@YusukeKato YusukeKato left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTMです。

@KuraZuzu
KuraZuzu merged commit 43db6fe into ros2 Sep 15, 2026
1 check passed
@KuraZuzu
KuraZuzu deleted the feature/support-lyrical branch September 15, 2026 06:18
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