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: add handle angle scale property to signal display #7774

Merged

Conversation

KhalilSelyan
Copy link
Contributor

Description

Move a hard coded scale value to editable rviz property

Parent Issue:

How was this PR tested?

Checked both old and new overlay for confirming values

Effects on system behavior

None.

@KhalilSelyan KhalilSelyan added the type:ui-ux User interface, user experience, graphical user interfaces. label Jul 1, 2024
@KhalilSelyan KhalilSelyan requested a review from MertClk July 1, 2024 13:59
@KhalilSelyan KhalilSelyan self-assigned this Jul 1, 2024
@github-actions github-actions bot added the component:common Common packages from the autoware-common repository. (auto-assigned) label Jul 1, 2024
Copy link

github-actions bot commented Jul 1, 2024

Thank you for contributing to the Autoware project!

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

Please ensure:

@KhalilSelyan KhalilSelyan linked an issue Jul 1, 2024 that may be closed by this pull request
3 tasks
KhalilSelyan and others added 4 commits July 1, 2024 17:03
Signed-off-by: KhalilSelyan <khalil@leodrive.ai>
…r msg_ptr instead of float value

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>
Signed-off-by: KhalilSelyan <khalil@leodrive.ai>
@github-actions github-actions bot added the type:documentation Creating or refining documentation. (auto-assigned) label Jul 1, 2024
KhalilSelyan added 2 commits July 1, 2024 19:17
Signed-off-by: KhalilSelyan <khalil@leodrive.ai>
Signed-off-by: KhalilSelyan <khalil@leodrive.ai>
@MertClk
Copy link
Contributor

MertClk commented Jul 1, 2024

LGTM

@MertClk MertClk added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Jul 1, 2024
Copy link

codecov bot commented Jul 1, 2024

Codecov Report

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

Project coverage is 28.40%. Comparing base (c1dbd5b) to head (3d7dabe).

Files Patch % Lines
...overlay_rviz_plugin/src/steering_wheel_display.cpp 0.00% 17 Missing ⚠️
...utoware_overlay_rviz_plugin/src/signal_display.cpp 0.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7774      +/-   ##
==========================================
- Coverage   28.40%   28.40%   -0.01%     
==========================================
  Files        1585     1585              
  Lines      115451   115461      +10     
  Branches    49231    49233       +2     
==========================================
  Hits        32794    32794              
- Misses      73707    73717      +10     
  Partials     8950     8950              
Flag Coverage Δ *Carryforward flag
differential 0.00% <0.00%> (?)
total 28.40% <ø> (+<0.01%) ⬆️ Carriedforward from c1dbd5b

*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.

@KhalilSelyan KhalilSelyan merged commit 6029eae into autowarefoundation:main Jul 1, 2024
37 of 39 checks passed
@KhalilSelyan
Copy link
Contributor Author

image

mitukou1109 pushed a commit to mitukou1109/autoware.universe that referenced this pull request Jul 2, 2024
…dation#7774)

* feat: add handle angle scale property to signal display

* fix: set default steering angle to 0.0° when not received

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* fix: set default steering angle to N/A when not received and check for msg_ptr instead of float value

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* chore: update steering wheel font size and wheel icon center is bigger

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* chore: update steering wheel display to center the steering angle text

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* chore: align steering angle text in both negative and positive cases

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

---------

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>
palas21 pushed a commit to palas21/autoware.universe that referenced this pull request Jul 12, 2024
…dation#7774)

* feat: add handle angle scale property to signal display

* fix: set default steering angle to 0.0° when not received

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* fix: set default steering angle to N/A when not received and check for msg_ptr instead of float value

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* chore: update steering wheel font size and wheel icon center is bigger

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* chore: update steering wheel display to center the steering angle text

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* chore: align steering angle text in both negative and positive cases

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

---------

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>
Signed-off-by: palas21 <palas21@itu.edu.tr>
tby-udel pushed a commit to tby-udel/autoware.universe that referenced this pull request Jul 14, 2024
…dation#7774)

* feat: add handle angle scale property to signal display

* fix: set default steering angle to 0.0° when not received

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* fix: set default steering angle to N/A when not received and check for msg_ptr instead of float value

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* chore: update steering wheel font size and wheel icon center is bigger

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* chore: update steering wheel display to center the steering angle text

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* chore: align steering angle text in both negative and positive cases

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

---------

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>
KhalilSelyan added a commit that referenced this pull request Jul 22, 2024
* feat: add handle angle scale property to signal display

* fix: set default steering angle to 0.0° when not received

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* fix: set default steering angle to N/A when not received and check for msg_ptr instead of float value

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* chore: update steering wheel font size and wheel icon center is bigger

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* chore: update steering wheel display to center the steering angle text

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

* chore: align steering angle text in both negative and positive cases

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>

---------

Signed-off-by: KhalilSelyan <khalil@leodrive.ai>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:common Common packages from the autoware-common repository. (auto-assigned) run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) type:documentation Creating or refining documentation. (auto-assigned) type:ui-ux User interface, user experience, graphical user interfaces.
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Export editable rviz property for steering wheel in overlay
2 participants