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

feat(aip_launcher): set multi fov for mirror and body pointcloud reduction #335

Merged
merged 20 commits into from
Oct 24, 2024

Conversation

kotaro-hihara
Copy link

@kotaro-hihara kotaro-hihara commented Oct 22, 2024

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を作成する。

@TomohitoAndo
Copy link
Contributor

確認rosbag (data searchに上がっていないデータ)
33381cfa-7d30-4e3e-b97c-fbe134ac2bf0_2024-09-27-13-51-24_p0900

Left upper

若干ノイズあり?
image

Right upper

車体に当たる点のちらつきあり(影響なさそうなレベル)
self cropで一旦は消えそう
image

Front upper

わずかに車体の点ちらつきあり
self cropで一旦は消えそう
image

Rear upper

後方に車体の点ちらつきあり
self cropで一旦は消えそう
image

Left lower

後方に車体の点ちらつきあり
self cropで一旦は消えそう
image

Right lower

空中にノイズ?
image

Front lower

まれに前方にちらつき有り(影響なさそうなレベル)
image

Rear lower

けっこうノイズ出ている
走行中のrosbagなので、前からAi-ringで問題になっている巻き上げか?
image

@TomohitoAndo
Copy link
Contributor

Concatで見ると、outlier filterのおかげか、ノイズは何も出ていないように見える

image

@TomohitoAndo TomohitoAndo self-requested a review October 23, 2024 13:32
@TomohitoAndo
Copy link
Contributor

nebulaの更新はこちらで対応

Copy link
Contributor

@TomohitoAndo TomohitoAndo left a comment

Choose a reason for hiding this comment

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

LGTM

若干のノイズはあるが、影響無いレベルかもしれないので必要に応じて調整しましょう!

@TomohitoAndo
Copy link
Contributor

@kotaro-hihara
あと、点群ring削減は「すべてのOT128を1/3に削減する」ことになったので、***_upperのみpoint_filters_oen_third.param.yamlを使う設定に変更をお願いします 🙏
https://tier4.atlassian.net/wiki/spaces/XFST/pages/3356788875/X2+gen2+MJ20-001+2024+10+4#%E8%A9%95%E4%BE%A1%E7%B5%90%E6%9E%9C%E3%81%BE%E3%81%A8%E3%82%81

@TomohitoAndo
Copy link
Contributor

OT128を1/3にした状態の動作OK
image

@kotaro-hihara
Copy link
Author

@TomohitoAndo OT128(**_upper)のconfigを1/3設定に変更しました。私の手元でも確認できたのでmergeします

@kotaro-hihara kotaro-hihara merged commit b3f4126 into beta/x2_gen2/v0.29.0 Oct 24, 2024
8 of 10 checks passed
@kotaro-hihara kotaro-hihara deleted the faet/x2_gen2/v0.29.0/multiFOV branch October 24, 2024 04:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants