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

fix(autoware_traffic_light_visualization): fix to visualize correct color and shapes #8428

Merged

Conversation

ktro2828
Copy link
Contributor

@ktro2828 ktro2828 commented Aug 9, 2024

Description

Related to #8389.

The visualization result of after this PR:

Screencast.from.2024.08.09.17.12.07.webm

Related links

Parent Issue:

  • Link

How was this PR tested?

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

@github-actions github-actions bot added the component:perception Advanced sensor data processing and environment understanding. (auto-assigned) label Aug 9, 2024
Copy link

github-actions bot commented Aug 9, 2024

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@ktro2828 ktro2828 force-pushed the fix/perception/tlr-visualization branch 2 times, most recently from 76422af to 14f2378 Compare August 9, 2024 07:58
…rrect shapes

Signed-off-by: ktro2828 <kotaro.uetake@tier4.jp>
@ktro2828 ktro2828 force-pushed the fix/perception/tlr-visualization branch from 14f2378 to 8640806 Compare August 9, 2024 08:00
Copy link
Contributor

@KhalilSelyan KhalilSelyan left a comment

Choose a reason for hiding this comment

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

LGTM, just tested on awsim

@ktro2828 ktro2828 added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Aug 13, 2024
@ktro2828
Copy link
Contributor Author

@tzhong518 Could you review this PR as a code owner? 🙏

Copy link

codecov bot commented Aug 13, 2024

Codecov Report

Attention: Patch coverage is 0% with 87 lines in your changes missing coverage. Please review.

Project coverage is 23.73%. Comparing base (1bcc0f9) to head (6017c65).
Report is 6 commits behind head on main.

Files Patch % Lines
...on/src/traffic_light_roi_visualizer/shape_draw.cpp 0.00% 45 Missing ⚠️
...on/src/traffic_light_roi_visualizer/shape_draw.hpp 0.00% 23 Missing ⚠️
...lization/src/traffic_light_roi_visualizer/node.hpp 0.00% 16 Missing ⚠️
...lization/src/traffic_light_roi_visualizer/node.cpp 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8428      +/-   ##
==========================================
- Coverage   23.88%   23.73%   -0.16%     
==========================================
  Files        1380     1379       -1     
  Lines      101867   101018     -849     
  Branches    38781    38320     -461     
==========================================
- Hits        24334    23974     -360     
+ Misses      75000    74575     -425     
+ Partials     2533     2469      -64     
Flag Coverage Δ *Carryforward flag
differential 0.00% <0.00%> (?)
total 23.75% <ø> (-0.14%) ⬇️ Carriedforward from 8640806

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ktro2828 ktro2828 merged commit 860eeb7 into autowarefoundation:main Aug 16, 2024
28 of 30 checks passed
@ktro2828 ktro2828 deleted the fix/perception/tlr-visualization branch August 16, 2024 09:01
saka1-s pushed a commit to saka1-s/autoware.universe that referenced this pull request Nov 1, 2024
…olor and shapes (autowarefoundation#8428)

fix(autoware_traffic_light_visualization): fix vialization to draw correct shapes

Signed-off-by: ktro2828 <kotaro.uetake@tier4.jp>
Co-authored-by: Yi-Hsiang Fang (Vivid) <146902905+vividf@users.noreply.github.com>
saka1-s pushed a commit to saka1-s/autoware.universe that referenced this pull request Nov 1, 2024
…olor and shapes (autowarefoundation#8428)

fix(autoware_traffic_light_visualization): fix vialization to draw correct shapes

Signed-off-by: ktro2828 <kotaro.uetake@tier4.jp>
Co-authored-by: Yi-Hsiang Fang (Vivid) <146902905+vividf@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:perception Advanced sensor data processing and environment understanding. (auto-assigned) run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants