-
Notifications
You must be signed in to change notification settings - Fork 701
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
refactor(perception-traffic-light-classifier): rework parameters #5797
refactor(perception-traffic-light-classifier): rework parameters #5797
Conversation
4d8fd78
to
8f6a83d
Compare
This pull request has been automatically marked as stale because it has not had recent activity. |
@miursh @yukkysaito please urgently review. |
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.
Sorry for late.
LGTM
@karishma1911 could you rebase this to the latest main? To fix the CI. |
…ort large maps (autowarefoundation#6725) Signed-off-by: Maxime CLEMENT <maxime.clement@tier4.jp>
Signed-off-by: Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>
…cloud_fusion (autowarefoundation#6840) * fix(image_projection_based_fusion): add intensity field for roi_pointcloud_fusion Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * fix: revesve the cluster refinement Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * fix: typo Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> --------- Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp>
…loader (autowarefoundation#6872) Revert "feat(global_parameter_loader): add gtest to global parameter loader (…" This reverts commit dfec62a.
Signed-off-by: Maxime CLEMENT <maxime.clement@tier4.jp> Co-authored-by: Tomas Nagy <tomas@pmc.sk>
…ng flip (autowarefoundation#6881) * feat(perception_online_evaluator): imporve yaw rate metrics considering flip Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> * fix test Signed-off-by: kosuke55 <kosuke.tnp@gmail.com> --------- Signed-off-by: kosuke55 <kosuke.tnp@gmail.com>
…warefoundation#6398) * feat(freespace_planning_algorithms): add Astar search by python Signed-off-by: Takumi Ito <takumi.ito@tier4.jp> * add copywrite and remove unneeded comment Signed-off-by: Takumi Ito <takumi.ito@tier4.jp> * - Add method getWaypoints() into the astar class. - No need to convert to serial when user passes ros2 message to methods. Signed-off-by: Takumi Ito <takumi.ito@tier4.jp> * insert astar python documents into the README Signed-off-by: Takumi Ito <takumi.ito@tier4.jp> * trivial modification Signed-off-by: Takumi Ito <takumi.ito@tier4.jp> * modify typo Signed-off-by: Takumi Ito <takumi.ito@tier4.jp> * Update planning/freespace_planning_algorithms/scripts/bind/astar_search_pybind.cpp Co-authored-by: Kosuke Takeuchi <kosuke.tnp@gmail.com> * ad copyright Signed-off-by: Takumi Ito <takumi.ito@tier4.jp> fix copyright Signed-off-by: Takumi Ito <takumi.ito@tier4.jp> add copyright Signed-off-by: Takumi Ito <takumi.ito@tier4.jp> * Update planning/freespace_planning_algorithms/scripts/bind/astar_search_pybind.cpp --------- Signed-off-by: Takumi Ito <takumi.ito@tier4.jp> Co-authored-by: Takumi Ito <takumi.ito@tier4.jp> Co-authored-by: Kosuke Takeuchi <kosuke.tnp@gmail.com>
…ne (autowarefoundation#6879) style(pre-commit): autofix refactor: topic name Signed-off-by: Mehmet Emin BAŞOĞLU <mehmeteminbasoglu@hotmail.com>
…on#6889) rename ndt diag Signed-off-by: Yamato Ando <yamato.ando@tier4.jp>
…utowarefoundation#6875) * enable aeb, fix topic problem Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * eliminate unused var Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * enable aeb, fix topic problem Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * eliminate unused var Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * add clustering method to eliminate small objects/noise Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * remove comments Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * Crop points outside of EGO predicted path Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * remove offset Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * Update library use Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * add check for empty cloud Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * add extra width for pointcloud cropping Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * Use single PC ptr Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * remove problematic option Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * Revert "Use single PC ptr" This reverts commit b5091fc. Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * refactoring Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * Add back timestamp Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * consider all points in clusters Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * USe object hull to detect collisions Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * Use only closest object point Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * remove unused functions Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * remove debug timer out of code Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * make it so the clustering uses parameters Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * solve type problem Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * update comment Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * update README Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * eliminate member var in favor of local pointcloud ptr Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * remove unused chrono dependency Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> --------- Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
* fix(route_handler): add logger Signed-off-by: Muhammad Zulfaqar Azmi <zulfaqar.azmi@tier4.jp> * fix indent Signed-off-by: Muhammad Zulfaqar Azmi <zulfaqar.azmi@tier4.jp> --------- Signed-off-by: Muhammad Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>
…tion#6743) * add param and schema file Signed-off-by: oguzkaganozt <oguzkaganozt@gmail.com> * style(pre-commit): autofix --------- Signed-off-by: oguzkaganozt <oguzkaganozt@gmail.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Kenzo Lobos Tsunekawa <kenzo.lobos@tier4.jp>
* feat(ndt_scan_matcher): remake diag Signed-off-by: yamato-ando <Yamato ANDO> * style(pre-commit): autofix * add latest_ndt_aling_service_best_score Signed-off-by: yamato-ando <Yamato ANDO> * style(pre-commit): autofix * check nullptr Signed-off-by: yamato-ando <Yamato ANDO> * style(pre-commit): autofix * add validate_distance_from_initial_to_result Signed-off-by: yamato-ando <Yamato ANDO> * style(pre-commit): autofix * rename Signed-off-by: yamato-ando <Yamato ANDO> * style(pre-commit): autofix * style(pre-commit): autofix * [WIP] update Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * [WIP] update Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * [WIP] update Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * [WIP] update Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * fix typo Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * [WIP] update Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * update readme Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * [WIP] update Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * [WIP] update Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * fix typo Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * [WIP] update Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * [WIP] udpate Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * [WIP] udpate Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * add is_need_rebuild Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * fix typo Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * add image Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * remove unused func Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * fix Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * fix typo Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * update image Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * fix typo Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * fix Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * remove unused include Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * move code Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * move code Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * fix FIX ME Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * update Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * remove unused func Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * [WIP] update Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * add diag for trigger node service Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * move code Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * update Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * delete unused code Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * udpate Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * delete RCLCPP message Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * update Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * fix typo Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * rename diag Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * rename func Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * remove Transition condition to OK Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * fix table Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * update readme Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * fix order Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * fix typo Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * remove diag prefix Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * fix readme Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * rename diag Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * remove unused code Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * fix double free Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * fix typo Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * output to terminal Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * fix typo Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> --------- Signed-off-by: yamato-ando <Yamato ANDO> Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> Co-authored-by: yamato-ando <Yamato ANDO> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…els and packages (autowarefoundation#6591) * refactor: lidar_centerpoint Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * refactor: pointpainting Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * chore: fix launch Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * chore: fix launch Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * chore: rearrange params Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * fix: json-schema-check error Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * fix: default param Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * refactor: rename param file Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * chore: typo Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * fix: align centerpoint param namespace with pointpainting Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * fix(centerpoint): add schema json Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * fix(pointpainting): fix schema json typo Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * style(pre-commit): autofix * docs: update pointpainting fusion doc Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * docs: update lidar centerpoint doc Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * fix: change omp param Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * fix:change twist and variance to model params Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * fix: keep build_only in launch Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * fix: schema check Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> * chore: temporary remove schema required Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> --------- Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp> Co-authored-by: Kenzo Lobos Tsunekawa <kenzo.lobos@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…foundation#6901) Signed-off-by: badai-nguyen <dai.nguyen@tier4.jp>
…utowarefoundation#6766) fix(avoidance): check object moving distance Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
…arefoundation#7187) * remove unusing main func Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com> * mod to componentize and use glog Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com> * change exec name and change log output from log to both Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com> * style(pre-commit): autofix --------- Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…ts (autowarefoundation#7177) Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp> Co-authored-by: Takayuki Murooka <takayuki5168@gmail.com>
…warefoundation#7183) * mod to componentize and use glog Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com> * change log output from screen to both Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com> * style(pre-commit): autofix * Update localization/pose_estimator_arbiter/CMakeLists.txt add namespace Co-authored-by: Kento Yabuuchi <moc.liamg.8y8@gmail.com> * remove unusing main func Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com> --------- Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Kento Yabuuchi <moc.liamg.8y8@gmail.com>
…tion#7192) * rename file name and mod to componentize and use glog Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com> * mod to componentize and use glog Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com> * change exec name and add log output both Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com> --------- Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com>
Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
…refoundation#7181) * refactor(start_planner): clarify condition of transition to success Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp> * modify flowchart Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp> * add debug print Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp> --------- Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
* change diag time_stamp Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * fix time now Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> * style(pre-commit): autofix * rename variable Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> --------- Signed-off-by: Yamato Ando <yamato.ando@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
add log output Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com>
…ation#7200) change log output from screen to both Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com>
… departure check (autowarefoundation#7193) * add param for lane departure margin Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * json thing Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * docs Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * make separate param for lane departure margin expansion Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> * update docs Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com> --------- Signed-off-by: Daniel Sanchez <danielsanchezaran@gmail.com>
…ndation#7213) Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
…ion#7208) * fix(yabloc): fix bug in capturing in lambda function Signed-off-by: Ryuta Kambe <ryuta.kambe@tier4.jp> * style(pre-commit): autofix --------- Signed-off-by: Ryuta Kambe <ryuta.kambe@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Signed-off-by: Ryuta Kambe <ryuta.kambe@tier4.jp>
Signed-off-by: Shumpei Wakabayashi <shumpei.wakabayashi@tier4.jp>
…utowarefoundation#7096) * add geometry utility test code * style(pre-commit): autofix * fix scope and declare * change declare name * style(pre-commit): autofix --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…tprint (autowarefoundation#7186) feat: filter unknown objects with its footprint Signed-off-by: yoshiri <yoshiyoshidetteiu@gmail.com>
* feat!: remove autoware_auto_geometry package Signed-off-by: Ryohsuke Mitsudome <ryohsuke.mitsudome@tier4.jp> * docs: remove autoware_auto_geometry package from docs Signed-off-by: Ryohsuke Mitsudome <ryohsuke.mitsudome@tier4.jp> --------- Signed-off-by: Ryohsuke Mitsudome <ryohsuke.mitsudome@tier4.jp>
…ation#7206) reafctor multi_object_tracker Signed-off-by: Ryuta Kambe <ryuta.kambe@tier4.jp>
…#7199) Signed-off-by: Mohammadreza Mirdamadiyan <smrm6789@gmail.com>
…re_perception_rviz_plugin (autowarefoundation#7221) feat(autoware_auto_perception_rviz_plugin): rename package to autoware_perception_rviz_plugin Signed-off-by: Ryohsuke Mitsudome <ryohsuke.mitsudome@tier4.jp> Co-authored-by: Yukihiro Saito <yukky.saito@gmail.com>
…ai/autoware.universe into traffic-light-classifier
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5797 +/- ##
==========================================
- Coverage 15.09% 0.00% -15.10%
==========================================
Files 1967 3 -1964
Lines 135941 258 -135683
Branches 42122 0 -42122
==========================================
- Hits 20520 0 -20520
+ Misses 92700 258 -92442
+ Partials 22721 0 -22721
☔ View full report in Codecov by Sentry. |
Done! |
@karishma1911 DCO checks are failing. Please rebase to main. |
Also please for PRs in Autoware, check this checkbox: So we can do it when necessary. |
@karishma1911 Could you make the change as @xmfcx requested? |
Description
Implement the ROS Node configuration layout described in https://github.com/orgs/autowarefoundation/discussions/3371 for the traffic-light-classifier
2.create schema
Tests performed
Not applicable.
Effects on system behavior
More reliable and faster parameter configuration file creation.
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.
After all checkboxes are checked, anyone who has write access can merge the PR.