-
Notifications
You must be signed in to change notification settings - Fork 672
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(crosswalk_traffic_light): add detector and classifier for pedestrian traffic light #5871
feat(crosswalk_traffic_light): add detector and classifier for pedestrian traffic light #5871
Conversation
Signed-off-by: tzhong518 <[email protected]>
Signed-off-by: tzhong518 <[email protected]>
Signed-off-by: tzhong518 <[email protected]>
Signed-off-by: tzhong518 <[email protected]>
Signed-off-by: tzhong518 <[email protected]>
Signed-off-by: tzhong518 <[email protected]>
This reverts commit 7a16659.
This reverts commit 47d8cdd.
Signed-off-by: tzhong518 <[email protected]>
Signed-off-by: tzhong518 <[email protected]>
Signed-off-by: tzhong518 <[email protected]>
Signed-off-by: tzhong518 <[email protected]>
Signed-off-by: tzhong518 <[email protected]>
Signed-off-by: tzhong518 <[email protected]>
If possible, could you post a video of the detection results here as well? |
Yes, I attached them in the desription. |
...r4_perception_launch/launch/traffic_light_recognition/traffic_light_node_container.launch.py
Outdated
Show resolved
Hide resolved
...r4_perception_launch/launch/traffic_light_recognition/traffic_light_node_container.launch.py
Outdated
Show resolved
Hide resolved
...r4_perception_launch/launch/traffic_light_recognition/traffic_light_node_container.launch.py
Show resolved
Hide resolved
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
Signed-off-by: tzhong518 <[email protected]>
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!
@tzhong518 Could you fix CI errors?
Signed-off-by: tzhong518 <[email protected]>
Signed-off-by: tzhong518 <[email protected]>
Signed-off-by: tzhong518 <[email protected]>
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #5871 +/- ##
==========================================
- Coverage 15.23% 15.18% -0.06%
==========================================
Files 1750 1754 +4
Lines 120710 121165 +455
Branches 36740 36740
==========================================
Hits 18394 18394
- Misses 81674 82129 +455
Partials 20642 20642
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
CI errors are fixed. |
…rian traffic light (autowarefoundation#5871) * add: crosswalk traffic light recognition Signed-off-by: tzhong518 <[email protected]> * fix: set conf=0 when occluded Signed-off-by: tzhong518 <[email protected]> * fix: clean code Signed-off-by: tzhong518 <[email protected]> * fix: refactor Signed-off-by: tzhong518 <[email protected]> * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: output not detected signals as unknown Signed-off-by: tzhong518 <[email protected]> * Revert "fix: output not detected signals as unknown" This reverts commit 7a16659. * Revert "fix: occlusion predictor" This reverts commit 47d8cdd. * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: clean debug code Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: launch file Signed-off-by: tzhong518 <[email protected]> * fix: set max angle range for different type Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: cancel the judge of flashing for estimated crosswalk traffic light Signed-off-by: tzhong518 <[email protected]> * delete: not necessary judgement on label Signed-off-by: tzhong518 <[email protected]> * Update perception/traffic_light_classifier/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/include/crosswalk_traffic_light_estimator/node.hpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: topic names and message attribute name Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: model names Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * fix: argument position Signed-off-by: tzhong518 <[email protected]> * fix: set classifier type in launch file Signed-off-by: tzhong518 <[email protected]> * fix: function and parameter name Signed-off-by: tzhong518 <[email protected]> * fix: func name Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_map_based_detector/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: move max angle range to config Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * fix: model name Signed-off-by: tzhong518 <[email protected]> * fix: conflict Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: CI test Signed-off-by: tzhong518 <[email protected]> --------- Signed-off-by: tzhong518 <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Yusuke Muramatsu <[email protected]>
…rian traffic light (autowarefoundation#5871) * add: crosswalk traffic light recognition Signed-off-by: tzhong518 <[email protected]> * fix: set conf=0 when occluded Signed-off-by: tzhong518 <[email protected]> * fix: clean code Signed-off-by: tzhong518 <[email protected]> * fix: refactor Signed-off-by: tzhong518 <[email protected]> * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: output not detected signals as unknown Signed-off-by: tzhong518 <[email protected]> * Revert "fix: output not detected signals as unknown" This reverts commit 7a16659. * Revert "fix: occlusion predictor" This reverts commit 47d8cdd. * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: clean debug code Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: launch file Signed-off-by: tzhong518 <[email protected]> * fix: set max angle range for different type Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: cancel the judge of flashing for estimated crosswalk traffic light Signed-off-by: tzhong518 <[email protected]> * delete: not necessary judgement on label Signed-off-by: tzhong518 <[email protected]> * Update perception/traffic_light_classifier/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/include/crosswalk_traffic_light_estimator/node.hpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: topic names and message attribute name Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: model names Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * fix: argument position Signed-off-by: tzhong518 <[email protected]> * fix: set classifier type in launch file Signed-off-by: tzhong518 <[email protected]> * fix: function and parameter name Signed-off-by: tzhong518 <[email protected]> * fix: func name Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_map_based_detector/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: move max angle range to config Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * fix: model name Signed-off-by: tzhong518 <[email protected]> * fix: conflict Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: CI test Signed-off-by: tzhong518 <[email protected]> --------- Signed-off-by: tzhong518 <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Yusuke Muramatsu <[email protected]>
…rian traffic light (autowarefoundation#5871) * add: crosswalk traffic light recognition Signed-off-by: tzhong518 <[email protected]> * fix: set conf=0 when occluded Signed-off-by: tzhong518 <[email protected]> * fix: clean code Signed-off-by: tzhong518 <[email protected]> * fix: refactor Signed-off-by: tzhong518 <[email protected]> * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: output not detected signals as unknown Signed-off-by: tzhong518 <[email protected]> * Revert "fix: output not detected signals as unknown" This reverts commit 7a16659. * Revert "fix: occlusion predictor" This reverts commit 47d8cdd. * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: clean debug code Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: launch file Signed-off-by: tzhong518 <[email protected]> * fix: set max angle range for different type Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: cancel the judge of flashing for estimated crosswalk traffic light Signed-off-by: tzhong518 <[email protected]> * delete: not necessary judgement on label Signed-off-by: tzhong518 <[email protected]> * Update perception/traffic_light_classifier/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/include/crosswalk_traffic_light_estimator/node.hpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: topic names and message attribute name Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: model names Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * fix: argument position Signed-off-by: tzhong518 <[email protected]> * fix: set classifier type in launch file Signed-off-by: tzhong518 <[email protected]> * fix: function and parameter name Signed-off-by: tzhong518 <[email protected]> * fix: func name Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_map_based_detector/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: move max angle range to config Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * fix: model name Signed-off-by: tzhong518 <[email protected]> * fix: conflict Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: CI test Signed-off-by: tzhong518 <[email protected]> --------- Signed-off-by: tzhong518 <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Yusuke Muramatsu <[email protected]>
…rian traffic light (autowarefoundation#5871) * add: crosswalk traffic light recognition Signed-off-by: tzhong518 <[email protected]> * fix: set conf=0 when occluded Signed-off-by: tzhong518 <[email protected]> * fix: clean code Signed-off-by: tzhong518 <[email protected]> * fix: refactor Signed-off-by: tzhong518 <[email protected]> * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: output not detected signals as unknown Signed-off-by: tzhong518 <[email protected]> * Revert "fix: output not detected signals as unknown" This reverts commit 7a16659. * Revert "fix: occlusion predictor" This reverts commit 47d8cdd. * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: clean debug code Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: launch file Signed-off-by: tzhong518 <[email protected]> * fix: set max angle range for different type Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: cancel the judge of flashing for estimated crosswalk traffic light Signed-off-by: tzhong518 <[email protected]> * delete: not necessary judgement on label Signed-off-by: tzhong518 <[email protected]> * Update perception/traffic_light_classifier/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/include/crosswalk_traffic_light_estimator/node.hpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: topic names and message attribute name Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: model names Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * fix: argument position Signed-off-by: tzhong518 <[email protected]> * fix: set classifier type in launch file Signed-off-by: tzhong518 <[email protected]> * fix: function and parameter name Signed-off-by: tzhong518 <[email protected]> * fix: func name Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_map_based_detector/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: move max angle range to config Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * fix: model name Signed-off-by: tzhong518 <[email protected]> * fix: conflict Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: CI test Signed-off-by: tzhong518 <[email protected]> --------- Signed-off-by: tzhong518 <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Yusuke Muramatsu <[email protected]>
…rian traffic light (autowarefoundation#5871) * add: crosswalk traffic light recognition Signed-off-by: tzhong518 <[email protected]> * fix: set conf=0 when occluded Signed-off-by: tzhong518 <[email protected]> * fix: clean code Signed-off-by: tzhong518 <[email protected]> * fix: refactor Signed-off-by: tzhong518 <[email protected]> * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: output not detected signals as unknown Signed-off-by: tzhong518 <[email protected]> * Revert "fix: output not detected signals as unknown" This reverts commit 7a16659. * Revert "fix: occlusion predictor" This reverts commit 47d8cdd. * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: clean debug code Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: launch file Signed-off-by: tzhong518 <[email protected]> * fix: set max angle range for different type Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: cancel the judge of flashing for estimated crosswalk traffic light Signed-off-by: tzhong518 <[email protected]> * delete: not necessary judgement on label Signed-off-by: tzhong518 <[email protected]> * Update perception/traffic_light_classifier/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/include/crosswalk_traffic_light_estimator/node.hpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: topic names and message attribute name Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: model names Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * fix: argument position Signed-off-by: tzhong518 <[email protected]> * fix: set classifier type in launch file Signed-off-by: tzhong518 <[email protected]> * fix: function and parameter name Signed-off-by: tzhong518 <[email protected]> * fix: func name Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_map_based_detector/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: move max angle range to config Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * fix: model name Signed-off-by: tzhong518 <[email protected]> * fix: conflict Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: CI test Signed-off-by: tzhong518 <[email protected]> --------- Signed-off-by: tzhong518 <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Yusuke Muramatsu <[email protected]>
…rian traffic light (autowarefoundation#5871) * add: crosswalk traffic light recognition Signed-off-by: tzhong518 <[email protected]> * fix: set conf=0 when occluded Signed-off-by: tzhong518 <[email protected]> * fix: clean code Signed-off-by: tzhong518 <[email protected]> * fix: refactor Signed-off-by: tzhong518 <[email protected]> * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: output not detected signals as unknown Signed-off-by: tzhong518 <[email protected]> * Revert "fix: output not detected signals as unknown" This reverts commit 7a16659. * Revert "fix: occlusion predictor" This reverts commit 47d8cdd. * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: clean debug code Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: launch file Signed-off-by: tzhong518 <[email protected]> * fix: set max angle range for different type Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: cancel the judge of flashing for estimated crosswalk traffic light Signed-off-by: tzhong518 <[email protected]> * delete: not necessary judgement on label Signed-off-by: tzhong518 <[email protected]> * Update perception/traffic_light_classifier/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/include/crosswalk_traffic_light_estimator/node.hpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: topic names and message attribute name Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: model names Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * fix: argument position Signed-off-by: tzhong518 <[email protected]> * fix: set classifier type in launch file Signed-off-by: tzhong518 <[email protected]> * fix: function and parameter name Signed-off-by: tzhong518 <[email protected]> * fix: func name Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_map_based_detector/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: move max angle range to config Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * fix: model name Signed-off-by: tzhong518 <[email protected]> * fix: conflict Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: CI test Signed-off-by: tzhong518 <[email protected]> --------- Signed-off-by: tzhong518 <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Yusuke Muramatsu <[email protected]>
…rian traffic light (autowarefoundation#5871) * add: crosswalk traffic light recognition Signed-off-by: tzhong518 <[email protected]> * fix: set conf=0 when occluded Signed-off-by: tzhong518 <[email protected]> * fix: clean code Signed-off-by: tzhong518 <[email protected]> * fix: refactor Signed-off-by: tzhong518 <[email protected]> * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: output not detected signals as unknown Signed-off-by: tzhong518 <[email protected]> * Revert "fix: output not detected signals as unknown" This reverts commit 7a16659. * Revert "fix: occlusion predictor" This reverts commit 47d8cdd. * fix: occlusion predictor Signed-off-by: tzhong518 <[email protected]> * fix: clean debug code Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: launch file Signed-off-by: tzhong518 <[email protected]> * fix: set max angle range for different type Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: cancel the judge of flashing for estimated crosswalk traffic light Signed-off-by: tzhong518 <[email protected]> * delete: not necessary judgement on label Signed-off-by: tzhong518 <[email protected]> * Update perception/traffic_light_classifier/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/include/crosswalk_traffic_light_estimator/node.hpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: topic names and message attribute name Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * fix: model names Signed-off-by: tzhong518 <[email protected]> * style(pre-commit): autofix * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/crosswalk_traffic_light_estimator/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_occlusion_predictor/src/nodelet.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * fix: argument position Signed-off-by: tzhong518 <[email protected]> * fix: set classifier type in launch file Signed-off-by: tzhong518 <[email protected]> * fix: function and parameter name Signed-off-by: tzhong518 <[email protected]> * fix: func name Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml Co-authored-by: Yusuke Muramatsu <[email protected]> * Update perception/traffic_light_map_based_detector/src/node.cpp Co-authored-by: Yusuke Muramatsu <[email protected]> * style(pre-commit): autofix * fix: move max angle range to config Signed-off-by: tzhong518 <[email protected]> * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * Update launch/tier4_perception_launch/launch/perception.launch.xml * fix: model name Signed-off-by: tzhong518 <[email protected]> * fix: conflict Signed-off-by: tzhong518 <[email protected]> * fix: precommit Signed-off-by: tzhong518 <[email protected]> * fix: CI test Signed-off-by: tzhong518 <[email protected]> --------- Signed-off-by: tzhong518 <[email protected]> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Yusuke Muramatsu <[email protected]> Signed-off-by: t4-x2 <[email protected]>
Description
Related links
Tests performed
Results: https://tier4.atlassian.net/wiki/spaces/~459732634/pages/2952200606
Please also checkout the related PRs when testing:
tier4/tier4_autoware_msgs#110
autowarefoundation/autoware_launch#752
autowarefoundation/autoware#4087
https://github.com/tier4/jetson_launcher/pull/35
command:
ros2 launch autoware_launch logging_simulator.launch.xml map_path:=~/map/odaiba_pedtlr_edited/ vehicle_model:=jpntaxi sensor_model:=aip_xx1 vehicle_id:=2 map:=true sensing:=false localization:=false perception:=true planning:=false use_external_lane_change:=false control:=false rviz:=true
detector onnx: https://drive.google.com/file/d/1DSBHlWY6RS3wrLLb7K39LXmBwKXvd0zS/view?usp=drive_link
classifier onnx: https://drive.google.com/file/d/1XczUh6wuA7xU_xlwvM9LVLS-v3dXjKqY/view?usp=drive_link
edited map: https://drive.google.com/drive/folders/1jYobyhS_6F03J-60DV7O-YaguXcMaaur?usp=drive_link
tested rosbag: https://drive.google.com/drive/folders/1mylLtw2SBF0JQP03rpBddg8fH_yjhGAK?usp=drive_link
lamp_ped.txt: https://drive.google.com/file/d/1Gn-D0vM9Wf1R3B1kwwrRPpgqJ4O-dniD/view?usp=drive_link (Please put it under traffic_light_classifier/data)
To visualize the final result on the map, please add
/perception/traffic_light_recognition/traffic_signals/markers
to the Rviz.Videos based on odaiba rosbag:
https://github.com/autowarefoundation/autoware.universe/assets/55872497/cf33b2dc-29e8-4b0a-a9ed-d5b84bbab48c
https://github.com/autowarefoundation/autoware.universe/assets/55872497/a3daa42d-500e-4a13-bcf6-ae08521000e9
https://github.com/autowarefoundation/autoware.universe/assets/55872497/e7d7e7cf-7919-4009-98bd-6ebf37689ce9
https://github.com/autowarefoundation/autoware.universe/assets/55872497/e73f3b00-f2f4-4cd0-aa69-72b2783556d8
Notes for reviewers
Interface changes
Effects on system behavior
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.