sbgisen / dynamic_slam_tool

Source Code for the DATMO algorithm I developed during my Summer Internship @ University of Calgary
https://doi.org/10.1002/eng2.12275
GNU General Public License v3.0
2 stars 0 forks source link

Feature/import wyw0 #1

Closed nyxrobotics closed 1 year ago

nyxrobotics commented 1 year ago

Summary

Setup

TODO

Multiple locations need to be edited when changing settings. This problem will be fixed in PR using rosparam.

nyxrobotics commented 1 year ago

reformatがPRに混ざってるせいでどこまでがwyw0の成果でどう統合したのかwyw0のrepoじゃだめだったのか全然わかりません

説明のためissue立てました #3 wywoはファイルを本家からコピーして最新版を上書きしたようなコミットになっています。 v1.0の時点で本家と大きく差異がありました https://github.com/WYW0/Dynamic-object-detection-for-unstructured-road/commits/main 本家を本流にして必要な変更のみ取り込もうとした結果が本PRです。 Revierwe目線で「何が取り込まれたかわからない」状態であればどのみちwyw0と同じ状況なので、wyw0をfork元にしたほうが良いという判断もできます。

nyxrobotics commented 1 year ago

理解しました。MovingObjectDetectionCloud::groundPlaneRemovalについて 下記2種類が実装されており、後者はコメントアウトされているので書き換えないといけませんでした。 取り急ぎ共有します。(確認しておきます→そのままでは機能しませんでした)

Tacha-S commented 1 year ago

黒点がもともとのvelodyneの点群で、intensityで色がついているのがこのノードの出力ですが、近距離でフィルターされていそうですが、用途的にいいんですか? trimのパラメータで解決しました Screenshot from 2023-01-19 14-12-20

市役所から出ていく直線脇の低木?草?が全部消えます。 Screenshot from 2023-01-19 14-16-23

マーカーの消し忘れなのかわからないがBoundingBox出ているのに点群が残っているときがある。 Screenshot from 2023-01-19 14-18-46

このパッケージがこの程度の性能って言う事ならこのままなんか動きはしているのでいいと思います。

nyxrobotics commented 1 year ago

遅れて申し訳けございません.

動作確認は取れましたが,どうしますか?使いやすいや性能評価は別としてこのままマージしますか?

お願いします