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

ROS2によさげなscan mergerがない件 #2

Closed
1 task done
Alpaca-zip opened this issue Jan 9, 2023 · 1 comment · Fixed by #11
Closed
1 task done

ROS2によさげなscan mergerがない件 #2

Alpaca-zip opened this issue Jan 9, 2023 · 1 comment · Fixed by #11
Assignees
Labels

Comments

@Alpaca-zip
Copy link
Contributor

Alpaca-zip commented Jan 9, 2023

実装現状、ira_laser_toolsに変わる良さげなROS2のscan multi mergerがない。orange_ros2上では地図作成にvelodyne_scanのみを利用していて、hokuyoを使っていない。

ros2_laser_scan_merger なるものを発見したけどよく分からないパラメータが多かったので自作した方がいいかもしれません。

タスク

  • ROS2版のscan multi mergerの実装
@Alpaca-zip Alpaca-zip added the enhancement New feature or request label Jan 9, 2023
@Alpaca-zip Alpaca-zip added task and removed enhancement New feature or request labels Jan 24, 2023
@Alpaca-zip
Copy link
Contributor Author

気が付きませんでしたが、iralabdisco/ira_laser_toolsのPRにROS2 Humbleのサポートを追加したという記述がありました。

Add support for ROS2 humble #47

とりあえず上記PRの内容で動作確認して、必要があればデバッグするのがよいと思います。

orange_ros2/orange_sensor_tools/srcフォルダを作ってira_laser_toolsのソースコードを引っ張ってくるか、Package.xmlにira_laser_toolsを追加して、ソースコード自体は別リポジトリでforkしてくるかで悩んでいますがどうしますか?

@Alpaca-zip Alpaca-zip linked a pull request Apr 8, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants