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

chore(aip_x2_gen2_launch): use dual_return_filter #387

Closed
wants to merge 4 commits into from

Conversation

kotaro-hihara
Copy link
Contributor

@kotaro-hihara kotaro-hihara commented Feb 5, 2025

dual_return_filterを使用、visibility diagを有効化。
dual_return_filter起動に必要なparamを追加。
visibilityのerror, warnの判定値は現時点でx2/v3.0.3に合わせる。
JIRA ticket

lsimで動作確認済み(雨天や透明なもののrosbag検証は未実施)

@@ -37,15 +37,14 @@
<arg name="cut_angle" value="180.0" />
<arg name="cloud_min_angle" value="88"/>
<arg name="cloud_max_angle" value="274"/>
<arg name="return_mode" value="LastStrongest"/>
<arg name="return_mode" value="Dual"/>
Copy link
Contributor

Choose a reason for hiding this comment

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

LastStrongestDualに変更しても、nebula側では特に影響ないでしょうか?

Copy link
Contributor Author

@kotaro-hihara kotaro-hihara Feb 5, 2025

Choose a reason for hiding this comment

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

確かにそうですね...ベンチで確認します

Copy link
Contributor

Choose a reason for hiding this comment

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

memo:
パット見では、どちらも同じ扱いになっているよう見える
https://github.com/tier4/nebula/blob/v0.2.3/nebula_common/include/nebula_common/hesai/hesai_common.hpp#L446

Copy link
Contributor Author

Choose a reason for hiding this comment

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

誤解してましたがQT128,OT128にDualというreturn_modeはなくDualのLastStrongestが既に設定している状態です。https://github.com/tier4/nebula/blob/main/scripts/hesai_config.py#L48-L55
return_modeはLastStrongestのままでdual_return_filterを使うように修正します(ドライバ側の修正なくなるのでLSIM確認で十分になると思います)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

LastStrongestのままdual_return_filter使用、visibility diag機能すること確認しました。
ring_outlier_filterに関する記述削除しました。

@kotaro-hihara kotaro-hihara force-pushed the chore/x2/use_dual_return_filter branch 2 times, most recently from ed534d8 to b7ce19c Compare February 5, 2025 10:12
@kotaro-hihara
Copy link
Contributor Author

image
ring_outler_filterで弾いてたであろう点群が出てきたので一旦中止(試したnebulaはv0.2.1とpilot-auto.x2 v4.0.2採用中のdevelop-eval)

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