-
Notifications
You must be signed in to change notification settings - Fork 17
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
feat(aip_launcher): set multi fov for mirror and body pointcloud reduction #335
feat(aip_launcher): set multi fov for mirror and body pointcloud reduction #335
Conversation
Signed-off-by: kotaro-hihara <[email protected]>
Signed-off-by: kotaro-hihara <[email protected]>
0160aa5
to
3200015
Compare
e8b9e32
to
5b4f606
Compare
ab6201e
to
39aebf3
Compare
nebulaの更新はこちらで対応 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
若干のノイズはあるが、影響無いレベルかもしれないので必要に応じて調整しましょう!
@kotaro-hihara |
e364981
to
d64dfd3
Compare
@TomohitoAndo OT128(**_upper)のconfigを1/3設定に変更しました。私の手元でも確認できたのでmergeします |
LiDARのmultisection FOV設定
JIRA ticket
従来crop box filterにより除外していたミラー点群と、LiDAR本体のchannelごとのFOV設定により除外していた車体点群を、
nebulaのpoint_filtersにより除去するためのparamを追加。
ring削減と同param内で設定するため、各LiDARごとにmulti section FOV設定のみ、multiFOV+ring削減のparamファイルを設定してある。
FOV検討link
本PRではnebula側でmultisectionFOVを追加しただけのため、multisectionFOV設定とcrop_box_filter両採用する形になる。
今後の実験データでcrop_box_filterでのミラー点群除去、ボディ点群除去を非採用にした場合の挙動を確認し、それぞれ非採用にできるようだったら別途PRを作成する。