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(tier4_perecption_launch): change radar object clustering parameter for ars408 in xx1 #985

Merged
merged 1 commit into from
Oct 30, 2023

Conversation

YoshiRi
Copy link

@YoshiRi YoshiRi commented Oct 25, 2023

Description

Merge前に要相談 DONE. Ready to merge.

ars408のradarのdetectionを矯正するPRになります。

  • 解決したい課題
    • お台場や柏でradarのdetectionが単一のobjから複数出現することが後段のtrackingに悪影響を及ぼすことがわかっています
    • radar clusteringというノードにはこのクラスタリングの閾値があります。(距離・角度・速度)
    • 現状角度の閾値がかなり厳しく、そのせいでOverlapするほどBBOXが接近していても別Objとして出力されています(BBOXが重なった物体同士の距離と角度差を以下のスライドで可視化しました)
    • この角度の閾値を30deg=0.52付近まであげればBBOXが重なるレベルのdetectionを統合することができます

image

  • 想定される動作とデグレ
    • radarから出てくる速度と位置と姿勢が近いdetectionを統合するようになります
    • デグレの可能性として2台の車が隣り合って走っている場合に1台のObjにまとめられてしまう可能性はあります。

image

上記のデメリットを考慮しても交差点モジュールに導入可能か @soblin に要相談。

(補足)Trackingへの悪影響

  • Trackingにて一つの物体から複数Objが生成された場合、生成されたObjが他レーンや外車線に飛び出たりして見栄えが大変悪いです
  • Pathがどの程度悪さをするのかについては応相談です。

Tests performed

Lsimで確認。
Not applicable.

Effects on system behavior

  • 🙆 radar detectionの分身が抑えられる
  • 🙅 近い位置・姿勢・速度の車両を同一のものとしてしまう懸念有り

Not applicable.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@YoshiRi YoshiRi requested review from soblin and tkimura4 October 25, 2023 08:18
@shmpwk shmpwk self-requested a review October 25, 2023 09:31
@tkimura4 tkimura4 changed the base branch from beta/v0.10.3 to beta/v0.10.4 October 27, 2023 04:10
@tkimura4 tkimura4 changed the title feat(tier4_perecption_launch):change radar object clustering parameter for ars408 in xx1 feat(tier4_perecption_launch): change radar object clustering parameter for ars408 in xx1 Oct 27, 2023
@YoshiRi
Copy link
Author

YoshiRi commented Oct 27, 2023

祖父江さんとのMTG結果:

  • 右折シーンで遠距離の同速度(閾値14.4km/s)同位置(閾値10m)の物体が2つか1つになるのはそこまで問題ではない
  • ほぼ同速度の場合あまりPathの到達位置はそこまで変わらないだろう
  • Merge時の位置ずれでレーンが変わるのだけがややネック(右折レーン → 直進レーンになってしまった場合など)だが、交差点70m以遠から右折レーンを見ている必要は道路構造的にもあまりなさそう
  • 総じて、LiDAR観測により近距離に来たときに正しく分離できれば交差点シーンでは問題ないだろうとのこと

以上を踏まえてMerge Readyなわけですがproject依存の変更ではあるのでそこだけ注意してください。

Copy link
Collaborator

@shmpwk shmpwk left a comment

Choose a reason for hiding this comment

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

想定されるデグレを理解した上でLGTMありがとうございます。

@shmpwk shmpwk merged commit 08b1677 into beta/v0.10.4 Oct 30, 2023
19 of 26 checks passed
@shmpwk shmpwk deleted the feat/change_radar_clustering_parameter_for_xx1 branch October 30, 2023 01:56
@shmpwk
Copy link
Collaborator

shmpwk commented Oct 30, 2023

@YoshiRi
Mergeしてから気づいたのですが、XX1固有の値でしたらautoware_launchにパラメタを外だししていただいた方が、良いと思われます!このままですとv0.11以降のメインバージョンでもcherry-pickをすることになるためです。

@YoshiRi
Copy link
Author

YoshiRi commented Oct 30, 2023

@shmpwk
確かに…! AWF側でパラメタをautoware_launchに出して、その後に各Projに反映させるようにしましょうか。

@shmpwk
Copy link
Collaborator

shmpwk commented Oct 31, 2023

@YoshiRi

確かに…! AWF側でパラメタをautoware_launchに出して、その後に各Projに反映させるようにしましょうか。

こちら作業お願いしたいです、、、!

@YoshiRi
Copy link
Author

YoshiRi commented Oct 31, 2023

@shmpwk
結構修正が大きそうなのでAWFのマージ間に合わないかもですm(_ _)m

@shmpwk
Copy link
Collaborator

shmpwk commented Oct 31, 2023

@YoshiRi
全然急ぎではないです、、、!
次のリリースに漏れなくこの反映がなされていれば少なくとも大丈夫です、、、!

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 this pull request may close these issues.

2 participants