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

Feature/add scan merger #11

Merged
merged 6 commits into from
Apr 9, 2023
Merged

Feature/add scan merger #11

merged 6 commits into from
Apr 9, 2023

Conversation

Alpaca-zip
Copy link
Contributor

概要

  • orange_ros2/orange_sensor_tools/にscan multi mergerを追加。

なぜこのタスクを行うのか

  • slam_toolboxを利用して地図を作成する際に、velodyneのscanしか利用できていなかったため。

やったこと

iralabdisco/ira_laser_toolsの以下のPRにROS2 Humbleのサポートを追加したという記述があったため、該当するソースコードをorange_ros2/orange_sensor_tools/srcフォルダを作って引っ張ってきました。

Add support for ROS2 humble #47

ただし、そのまま実行するとscanのintensitiesの有無が原因で、scanをmergeする際にエラーが出ていたので、応急処置的にdelete_intensityというパラメータを作り、以下の処理を加えました。

if (delete_intensity)
{
pcl::PointCloud<pcl::PointXYZ>::Ptr cloud_without_intensity(new pcl::PointCloud<pcl::PointXYZ>);
pcl::fromPCLPointCloud2(clouds[i], *cloud_without_intensity);
pcl::toPCLPointCloud2(*cloud_without_intensity, clouds[i]);
}

その他

  • package.xmlのversionを毎度、変更するのが面倒になってきたのでREADME.mdだけ変更するようにしました。

@Alpaca-zip Alpaca-zip added the enhancement New feature or request label Apr 8, 2023
@Alpaca-zip Alpaca-zip self-assigned this Apr 8, 2023
@Alpaca-zip Alpaca-zip linked an issue Apr 8, 2023 that may be closed by this pull request
1 task
@Alpaca-zip Alpaca-zip added the task label Apr 8, 2023
Copy link
Contributor

@koki0624 koki0624 left a comment

Choose a reason for hiding this comment

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

問題なく動作したのでマージしちゃいます。

@koki0624 koki0624 merged commit 7dbce71 into main Apr 9, 2023
@Alpaca-zip
Copy link
Contributor Author

問題なく動作したのでマージしちゃいます。

助かります。ありがとう。

@Alpaca-zip Alpaca-zip deleted the feature/add_scan_merger branch April 9, 2023 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

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