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(rviz_plugin): console meter is too large on the Rviz with FHD display, isn't it? #587

Merged
merged 3 commits into from
May 25, 2022

Conversation

TakaHoribe
Copy link
Contributor

@TakaHoribe TakaHoribe commented Mar 26, 2022

Note: must be merged with autowarefoundation/autoware_launch#44

Description

I wanted to adjust the size of the plugins to our screen size.

I modified the default size so that these plugins are displayed at smaller sizes on a screen smaller than 4k.
Note: I'm not satisfied with my approach. (Although I hate the current size more.) Any better solution is welcome.

Now on FHD: Steering, you are not the main character.
Screenshot from 2022-03-25 19-02-47

With this PR. It's cool. Nice.

on 4k.
Screenshot from 2022-03-25 19-02-03

on FHD.
Screenshot from 2022-03-25 19-00-51

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.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@codecov
Copy link

codecov bot commented Mar 26, 2022

Codecov Report

Merging #587 (6dc12a9) into main (1696c2b) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##            main    #587      +/-   ##
========================================
- Coverage   9.57%   9.54%   -0.03%     
========================================
  Files        939     939              
  Lines      58181   58355     +174     
  Branches   10638   10638              
========================================
  Hits        5570    5570              
- Misses     47892   48066     +174     
  Partials    4719    4719              
Flag Coverage Δ *Carryforward flag
differential 0.00% <0.00%> (?)
total 9.57% <0.00%> (ø) Carriedforward from 1696c2b

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

Impacted Files Coverage Δ
...r4_planning_rviz_plugin/src/tools/max_velocity.cpp 0.00% <0.00%> (ø)
...r4_vehicle_rviz_plugin/src/tools/console_meter.cpp 0.00% <0.00%> (ø)
...4_vehicle_rviz_plugin/src/tools/steering_angle.cpp 0.00% <0.00%> (ø)
...ier4_vehicle_rviz_plugin/src/tools/turn_signal.cpp 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1696c2b...6dc12a9. Read the comment docs.

@wep21
Copy link
Contributor

wep21 commented May 23, 2022

@TakaHoribe Are you still working on this PR?

@TakaHoribe
Copy link
Contributor Author

@wep21 I didn't touch this PR recently but will make it. I would be appreciated if someone could help.

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>
Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>
@TakaHoribe
Copy link
Contributor Author

now i'm taking it

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>
@TakaHoribe
Copy link
Contributor Author

@wep21 @yukkysaito now it is ready for review

@TakaHoribe
Copy link
Contributor Author

FHD
Screenshot from 2022-05-24 13-23-45

4K
Screenshot from 2022-05-24 13-21-29

@yukkysaito
Copy link
Contributor

yukkysaito commented May 25, 2022

LGTM

Ultra wide display
Screenshot from 2022-05-26 01-40-14

@TakaHoribe TakaHoribe merged commit 4832255 into autowarefoundation:main May 25, 2022
@TakaHoribe TakaHoribe deleted the feature/rviz-plugin-scaling branch May 25, 2022 16:46
@TakaHoribe TakaHoribe mentioned this pull request Jun 5, 2022
3 tasks
ktro2828 pushed a commit to ktro2828/autoware.universe that referenced this pull request Jun 7, 2022
…splay, isn't it? (autowarefoundation#587)

* feat(tier4_planning/vehicle_plugin): make plugins size scalable

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* remove space

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* scaling

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>
Signed-off-by: ktro2828 <kotaro.uetake@tier4.jp>
boyali referenced this pull request in boyali/autoware.universe Sep 28, 2022
…splay, isn't it? (tier4#587)

* feat(tier4_planning/vehicle_plugin): make plugins size scalable

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* remove space

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* scaling

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>
boyali referenced this pull request in boyali/autoware.universe Oct 3, 2022
…splay, isn't it? (tier4#587)

* feat(tier4_planning/vehicle_plugin): make plugins size scalable

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* remove space

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* scaling

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>
boyali referenced this pull request in boyali/autoware.universe Oct 3, 2022
…splay, isn't it? (tier4#587)

* feat(tier4_planning/vehicle_plugin): make plugins size scalable

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* remove space

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* scaling

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>
boyali referenced this pull request in boyali/autoware.universe Oct 19, 2022
…splay, isn't it? (tier4#587)

* feat(tier4_planning/vehicle_plugin): make plugins size scalable

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* remove space

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* scaling

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>
satoshi-ota pushed a commit to satoshi-ota/autoware.universe that referenced this pull request Jun 23, 2023
…0.8.1

feat(behavior_path_planner): use module status idle before approval and keep candiate running module when approved module succeed autowarefoundation#3928
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.

3 participants