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(perception_rviz_plugin): add 2d dummy bus with height #903

Conversation

taikitanaka3
Copy link
Contributor

@taikitanaka3 taikitanaka3 commented May 15, 2022

Description

current 2d car is moving by default but for some functions in avoidance , no stopping are , intersection occlusion spot needs stopping obstacle so I add rviz plugins for stopping bus.

  • add 2d dummy stopping bus
    image

image

image

Related links

Tests performed

by psim

Notes for reviewers

run psim and add panel from + button
image

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.

  • The PR follows the pull request guidelines.
  • The PR has been properly tested.
  • The PR has been reviewed by the code owners.

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.
  • The PR is ready for merge.

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

@codecov
Copy link

codecov bot commented May 15, 2022

Codecov Report

Merging #903 (8f26b4b) into main (3bcb248) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##            main    #903      +/-   ##
========================================
- Coverage   9.54%   9.53%   -0.02%     
========================================
  Files        933     933              
  Lines      57727   57815      +88     
  Branches   10408   10408              
========================================
  Hits        5510    5510              
- Misses     47687   47775      +88     
  Partials    4530    4530              
Flag Coverage Δ *Carryforward flag
differential 0.00% <0.00%> (?)
total 9.54% <0.00%> (ø) Carriedforward from fd7d7e1

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

Impacted Files Coverage Δ
...ier4_perception_rviz_plugin/src/tools/car_pose.cpp 0.00% <0.00%> (ø)
...ption_rviz_plugin/src/tools/interactive_object.hpp 0.00% <ø> (ø)
..._perception_rviz_plugin/src/tools/unknown_pose.cpp 0.00% <0.00%> (ø)
...rception_rviz_plugin/src/tools/pedestrian_pose.cpp 0.00% <0.00%> (ø)
...ption_rviz_plugin/src/tools/delete_all_objects.cpp 0.00% <0.00%> (ø)
...ption_rviz_plugin/src/tools/interactive_object.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 3bcb248...8f26b4b. Read the comment docs.

@taikitanaka3
Copy link
Contributor Author

@HiroIshida
[Q] if I put bus in rviz it's a bit heavy processing point cloud.
is there anyways to make computational cost lighter?

@HiroIshida
Copy link
Contributor

HiroIshida commented May 16, 2022

@taikitanaka3

my observation

I think visualization rather than algorithm may cause the problem. Probably, the number of pointcloud to visualize causes the problem.

experiment

I tried you new rviz plugin with the setting below:
setup

Considering the possibility that newly implemented algorithm in
#527 make the simulation slower, I embeddedd the time measurement function to the dummy perception publisher's timer callback, and measure the difference between processing time of ego-centric (new) and object centric point cloud generation methods. The result below reveals that point cloud generation algorithm does affect much here.

Note that typical processing time of the algorithms should be
object-centric without raytracing < ego-centric << object-centric with raytracing .
I omitted object-centric with raytracing in this experiment.

newer algorithm, Ego-centric (with raytracing in principle)

[dummy_perception_publisher_node-35] [INFO] [1652690826.930240479] [simulation.dummy_perception_publisher]: callback elapsed time: 32.6487
[dummy_perception_publisher_node-35] [INFO] [1652690827.027392244] [simulation.dummy_perception_publisher]: callback elapsed time: 29.8299
[dummy_perception_publisher_node-35] [INFO] [1652690827.124755941] [simulation.dummy_perception_publisher]: callback elapsed time: 27.1835
[dummy_perception_publisher_node-35] [INFO] [1652690827.223872107] [simulation.dummy_perception_publisher]: callback elapsed time: 26.2822

old algorithm, Object-centric (disabled raytracing)

[dummy_perception_publisher_node-35] [INFO] [1652691405.282255247] [simulation.dummy_perception_publisher]: callback elapsed time: 23.8785
[dummy_perception_publisher_node-35] [INFO] [1652691405.383950450] [simulation.dummy_perception_publisher]: callback elapsed time: 25.5756
[dummy_perception_publisher_node-35] [INFO] [1652691405.482130112] [simulation.dummy_perception_publisher]: callback elapsed time: 23.7911
[dummy_perception_publisher_node-35] [INFO] [1652691405.582512018] [simulation.dummy_perception_publisher]: callback elapsed time: 24.1669

I then though maybe visualization (or entailing communication) take much computational effort a lot, so I compare htop result turning on and off the rviz. The result below shows significant computational load decline after turning off rviz. I'm not certainly sure, but from the observation so far, visualization rather than algorithm should be improved in some way.

with_rviz

without_rviz
.png)

@HiroIshida
Copy link
Contributor

HiroIshida commented May 16, 2022

@taikitanaka3
Let me give bit more time to probe this issue (my next part-time shift is 5/18). Now I realized that just comparing processing time between the algorithm is not enough.

@HiroIshida
Copy link
Contributor

HiroIshida commented May 17, 2022

@taikitanaka3
I think you are right. Newer (ego-centric) method take up much computation time, although computation time dose not varies between the two.

when running with object centric mode: <param name="object_centric_pointcloud" value="true"/>

$ ps -aux |grep dummy_perception_publisher_node|head -1| awk '{print $2}' | xargs -i pidstat -p {} 5 100
Linux 5.14.0-1036-oem (stonet4) 	05/18/2022 	_x86_64_	(12 CPU)

05:41:54 AM   UID       PID    %usr %system  %guest   %wait    %CPU   CPU  Command
05:41:59 AM  1000    188562    3.80    0.00    0.00    0.40    3.80     6  dummy_perceptio
05:42:04 AM  1000    188562    4.00    0.00    0.00    0.00    4.00     5  dummy_perceptio
05:42:09 AM  1000    188562    4.00    0.00    0.00    0.20    4.00     3  dummy_perceptio
05:42:14 AM  1000    188562    4.00    0.00    0.00    0.20    4.00     6  dummy_perceptio
05:42:19 AM  1000    188562    3.80    0.00    0.00    0.40    3.80     7  dummy_perceptio
^C
Average:     1000    188562    3.92    0.00    0.00    0.24    3.92     -  dummy_perceptio

when running with ego-centric mode: <param name="object_centric_pointcloud" value="false"/>

$ ps -aux |grep dummy_perception_publisher_node|head -1| awk '{print $2}' | xargs -i pidstat -p {} 5 100
Linux 5.14.0-1036-oem (stonet4) 	05/18/2022 	_x86_64_	(12 CPU)

05:46:31 AM   UID       PID    %usr %system  %guest   %wait    %CPU   CPU  Command
05:46:36 AM  1000    199150   13.60    0.40    0.00    0.80   14.00     5  dummy_perceptio
05:46:41 AM  1000    199150   13.60    1.00    0.00    0.60   14.60     7  dummy_perceptio
05:46:46 AM  1000    199150   13.20    0.80    0.00    0.60   14.00     3  dummy_perceptio
05:46:51 AM  1000    199150   13.20    1.00    0.00    0.80   14.20     4  dummy_perceptio
05:46:56 AM  1000    199150   13.40    1.20    0.00    0.60   14.60     9  dummy_perceptio
^C
Average:     1000    199150   13.40    0.88    0.00    0.68   14.28     -  dummy_perceptio

As a workaround, you can set <param name="object_centric_pointcloud" value="true"/> to reduce the computational load. But by doing this, the generated pointcloud may be poor as previously mentioned in #527

I have some idea to reduce the computational load when ego-centric mode. Let me some time to try the idea and make a new PR.

@HiroIshida
Copy link
Contributor

HiroIshida commented May 18, 2022

@taikitanaka3
I made a PR that reduce the aboud 50% computational load. (in the measurement, I used this PR's bus objects)
#926

@taikitanaka3
Copy link
Contributor Author

@HiroIshida
Thank you I will take a look and test this at today evening.

@taikitanaka3 taikitanaka3 marked this pull request as ready for review May 28, 2022 03:36
@taikitanaka3 taikitanaka3 requested a review from h-ohta May 28, 2022 03:36
@taikitanaka3 taikitanaka3 enabled auto-merge (squash) May 28, 2022 03:38
@taikitanaka3 taikitanaka3 requested a review from shmpwk May 30, 2022 06:38
Copy link
Contributor

@shmpwk shmpwk left a comment

Choose a reason for hiding this comment

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

@taikitanaka3
Could you add info about this nice feature at README?

Signed-off-by: Shumpei Wakabayashi <[email protected]>
@shmpwk
Copy link
Contributor

shmpwk commented May 30, 2022

Screenshot from 2022-05-30 16-41-59
I added icon for 2d Dummy Bus rviz tool by 8f26b4b

@taikitanaka3 taikitanaka3 merged commit cb104b5 into autowarefoundation:main May 30, 2022
@taikitanaka3 taikitanaka3 deleted the feat-add-2d-dummy-stopping-bus-to-simulator branch May 31, 2022 05:42
ktro2828 pushed a commit to ktro2828/autoware.universe that referenced this pull request Jun 7, 2022
…oundation#903)

* feat(perception_rviz_plugin): add 2d dummy bus with height

Signed-off-by: tanaka3 <[email protected]>

* ci(pre-commit): autofix

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Signed-off-by: ktro2828 <[email protected]>
YoheiMishina pushed a commit to YoheiMishina/autoware.universe that referenced this pull request Jun 9, 2022
* release v0.4.0

* check if gdown command exists (autowarefoundation#707)

* Add nodelets of tlr nodes (autowarefoundation#715)

* Add classifier nodelet

Signed-off-by: Daisuke Nishimatsu <[email protected]>

* Replace boost::shared_ptr into std::shared_ptr

Signed-off-by: Daisuke Nishimatsu <[email protected]>

* Add lock guard

Signed-off-by: Daisuke Nishimatsu <[email protected]>

* Add detetcor nodelet

Signed-off-by: Daisuke Nishimatsu <[email protected]>

* Integrate main into node

Signed-off-by: Daisuke Nishimatsu <[email protected]>

* Add SubscriberStatusCallback

Signed-off-by: Daisuke Nishimatsu <[email protected]>

* add image_transport_decompresser nodelet

Signed-off-by: Yukihiro Saito <[email protected]>

* Add visualizer nodelet

Signed-off-by: Daisuke Nishimatsu <[email protected]>

* fixed bug

Signed-off-by: Yukihiro Saito <[email protected]>

* Fix plugin name

Signed-off-by: Daisuke Nishimatsu <[email protected]>

* Launch nodelet

Signed-off-by: Daisuke Nishimatsu <[email protected]>

* Fix classifier constructor

Signed-off-by: Daisuke Nishimatsu <[email protected]>

* add decompresser node

Signed-off-by: Yukihiro Saito <[email protected]>

* fix typo

Signed-off-by: Yukihiro Saito <[email protected]>

* fixed bug

Signed-off-by: Yukihiro Saito <[email protected]>

* fixed bug

Signed-off-by: Yukihiro Saito <[email protected]>

* cosmetic change

Signed-off-by: Yukihiro Saito <[email protected]>

* add param

Signed-off-by: Yukihiro Saito <[email protected]>

* fix bug

Signed-off-by: Yukihiro Saito <[email protected]>

* Fix build warning

Signed-off-by: Daisuke Nishimatsu <[email protected]>

* change rgb

Signed-off-by: Yukihiro Saito <[email protected]>

* change rgb

Signed-off-by: Yukihiro Saito <[email protected]>

Co-authored-by: Yukihiro Saito <[email protected]>

* remove libutils dependency when unable gpu (autowarefoundation#761)

* Fix typo cliped -> clipped (autowarefoundation#776)

Signed-off-by: Kosuke Takeuchi <[email protected]>

* install launch when disable gpu (autowarefoundation#829)

* add launch and xml install when disable gpu

* remove unnecessary install

* change raw pointer to vector and shared_ptr (#817)

* change raw pointer to vector and shared_ptr

* fix bug

* Fix/cublas dependency (autowarefoundation#849)

* fix cublas depencency

* fix cublas depencency

* remove ROS1 packages temporarily

Signed-off-by: mitsudome-r <[email protected]>

* Revert "remove ROS1 packages temporarily"

This reverts commit 9b343cf4c11815735a176ef31f9e6e9948f20c74.

Signed-off-by: mitsudome-r <[email protected]>

* add COLCON_IGNORE to ros1 packages

Signed-off-by: mitsudome-r <[email protected]>

* Rename launch files to launch.xml (autowarefoundation#28)

* Port traffic light classifier (autowarefoundation#70)

* port package.xml and CMakeLists.txt

Signed-off-by: mitsudome-r <[email protected]>

* port ROS messages

Signed-off-by: mitsudome-r <[email protected]>

* port to ros2

Signed-off-by: mitsudome-r <[email protected]>

* make wget buildtool_depend

Signed-off-by: mitsudome-r <[email protected]>

* remove main function

Signed-off-by: mitsudome-r <[email protected]>

* add connectCb as a timer callback

Signed-off-by: mitsudome-r <[email protected]>

* fix indentation

Signed-off-by: mitsudome-r <[email protected]>

* remove catkin includes and libraries from CMakeLists.txt

Signed-off-by: mitsudome-r <[email protected]>

* [traffic_light_ssd_fine_detector] port to ROS2 (autowarefoundation#113)

Signed-off-by: mitsudome-r <[email protected]>

* Ros2 port traffic light map (autowarefoundation#99)

* remove colcon ignore

* ported CMakelists

* port package

* remove colcon ignore

* ported CMakelists

* port package

* ported traffic_light_map_detector to ROS2 - compiling

* port launch and config files, tidied up

* change rclcpp duration arguments

* launch file corrections

* lookupTransform uses exact timestamp

* Convert calls of Duration to Duration::from_seconds where appropriate (autowarefoundation#131)

* Rename h files to hpp (autowarefoundation#142)

* Change includes

* Rename files

* Adjustments to make things compile

* Other packages

* Adjust copyright notice on 532 out of 699 source files (autowarefoundation#143)

* Use quotes for includes where appropriate (autowarefoundation#144)

* Use quotes for includes where appropriate

* Fix lint tests

* Make tests pass hopefully

* fixing trasient_local in ROS2 packages (autowarefoundation#160)

* adding linters in traffic_light_map_based_detector (autowarefoundation#179)

* traffic_light_classifier: Fix engine save dir (autowarefoundation#250)

Signed-off-by: wep21 <[email protected]>

* traffic light map based detector: Fix launch (autowarefoundation#253)

Signed-off-by: wep21 <[email protected]>

* Ros2 v0.8.0 traffic light map based detector (autowarefoundation#262)

* change alpha (autowarefoundation#903)

Signed-off-by: Yukihiro Saito <[email protected]>

* fix typos in perception (autowarefoundation#862)

Co-authored-by: Yukihiro Saito <[email protected]>
Co-authored-by: Kazuki Miyahara <[email protected]>

* Ros2 v0.8.0 traffic light classifier (autowarefoundation#261)

* fix typos in perception (autowarefoundation#862)

* update README.md in perception (autowarefoundation#1007)

* update traffic light recognition model (autowarefoundation#1086)

* update traffic light recognition model

* download model when hash has changed

* fix CMakeLists

* udpate tl model to scale ai dataset one (autowarefoundation#1118)

Co-authored-by: Kazuki Miyahara <[email protected]>
Co-authored-by: Satoshi Tanaka <[email protected]>
Co-authored-by: Taichi Higashide <[email protected]>

* Ros2 v0.8.0 traffic light ssd fine detector (autowarefoundation#260)

* fix typos in perception (autowarefoundation#862)

* update README.md in perception (autowarefoundation#1007)

* update traffic light recognition model (autowarefoundation#1086)

* update traffic light recognition model

* download model when hash has changed

* fix CMakeLists

* udpate tl model to scale ai dataset one (autowarefoundation#1118)

Co-authored-by: Kazuki Miyahara <[email protected]>
Co-authored-by: Satoshi Tanaka <[email protected]>
Co-authored-by: Taichi Higashide <[email protected]>

* Rename ROS-related .yaml to .param.yaml (autowarefoundation#352)

* Rename ROS-related .yaml to .param.yaml

Signed-off-by: Kenji Miyake <[email protected]>

* Remove prefix 'default_' of yaml files

Signed-off-by: Kenji Miyake <[email protected]>

* Rename vehicle_info.yaml to vehicle_info.param.yaml

Signed-off-by: Kenji Miyake <[email protected]>

* Rename diagnostic_aggregator's param files

Signed-off-by: Kenji Miyake <[email protected]>

* Fix overlooked parameters

Signed-off-by: Kenji Miyake <[email protected]>

* Ros2 v0.8.0 beta rm std msgs ssd traffic light (autowarefoundation#395) (autowarefoundation#400)

* rm std_msgs float32

* fix typo

Co-authored-by: taikitanaka3 <[email protected]>

* add use_sim-time option (autowarefoundation#454)

* Fix rolling build errors (autowarefoundation#1225)

* Add missing include files

Signed-off-by: Kenji Miyake <[email protected]>

* Replace rclcpp::Duration

Signed-off-by: Kenji Miyake <[email protected]>

* Use reference for exceptions

Signed-off-by: Kenji Miyake <[email protected]>

* Use from_seconds

Signed-off-by: Kenji Miyake <[email protected]>

* Sync public repo (autowarefoundation#1228)

* [simple_planning_simulator] add readme (autowarefoundation#424)

* add readme of simple_planning_simulator

Signed-off-by: Takamasa Horibe <[email protected]>

* Update simulator/simple_planning_simulator/README.md

* set transit_margin_time to intersect. planner (autowarefoundation#460)

* Fix pose2twist (autowarefoundation#462)

Signed-off-by: Takagi, Isamu <[email protected]>

* Ros2 vehicle info param server (autowarefoundation#447)

* add vehicle_info_param_server

* update vehicle info

* apply format

* fix bug

* skip unnecessary search

* delete vehicle param file

* fix bug

* Ros2 fix topic name part2 (autowarefoundation#425)

* Fix topic name of traffic_light_classifier

Signed-off-by: Takagi, Isamu <[email protected]>

* Fix topic name of traffic_light_visualization

Signed-off-by: Takagi, Isamu <[email protected]>

* Fix topic name of traffic_light_ssd_fine_detector

Signed-off-by: Takagi, Isamu <[email protected]>

* Fix topic name of traffic_light_map_based_detector

Signed-off-by: Takagi, Isamu <[email protected]>

* Fix lint traffic_light_recognition

Signed-off-by: Takagi, Isamu <[email protected]>

* Fix lint traffic_light_ssd_fine_detector

Signed-off-by: Takagi, Isamu <[email protected]>

* Fix lint traffic_light_classifier

Signed-off-by: Takagi, Isamu <[email protected]>

* Fix lint traffic_light_classifier

Signed-off-by: Takagi, Isamu <[email protected]>

* Fix lint traffic_light_ssd_fine_detector

Signed-off-by: Takagi, Isamu <[email protected]>

* Fix issues in hdd_reader (autowarefoundation#466)

* Fix some issues detected by Coverity Scan and Clang-Tidy

* Update launch command

* Add more `close(new_sock)`

* Simplify the definitions of struct

* fix: re-construct laneletMapLayer for reindex RTree (autowarefoundation#463)

* Rviz overlay render fix (autowarefoundation#461)

* Moved painiting in SteeringAngle plugin to update()

Signed-off-by: Adam Dabrowski <[email protected]>

* super class now back to MFD

Signed-off-by: Adam Dabrowski <[email protected]>

* uncrustified

Signed-off-by: Adam Dabrowski <[email protected]>

* acquire data in mutex

Signed-off-by: Adam Dabrowski <[email protected]>

* back to RTD as superclass

Signed-off-by: Adam Dabrowski <[email protected]>

* Rviz overlay render in update (autowarefoundation#465)

* Moved painiting in SteeringAngle plugin to update()

Signed-off-by: Adam Dabrowski <[email protected]>

* super class now back to MFD

Signed-off-by: Adam Dabrowski <[email protected]>

* uncrustified

Signed-off-by: Adam Dabrowski <[email protected]>

* acquire data in mutex

Signed-off-by: Adam Dabrowski <[email protected]>

* removed unnecessary includes and some dead code

Signed-off-by: Adam Dabrowski <[email protected]>

* Adepted remaining vehicle plugin classes to render-in-update concept. Returned to MFD superclass

Signed-off-by: Adam Dabrowski <[email protected]>

* restored RTD superclass

Signed-off-by: Adam Dabrowski <[email protected]>

Co-authored-by: Takamasa Horibe <[email protected]>
Co-authored-by: tkimura4 <[email protected]>
Co-authored-by: Takagi, Isamu <[email protected]>
Co-authored-by: Kazuki Miyahara <[email protected]>
Co-authored-by: Makoto Tokunaga <[email protected]>
Co-authored-by: Adam Dąbrowski <[email protected]>

* Unify Apache-2.0 license name (autowarefoundation#1242)

* Remove use_sim_time for set_parameter (autowarefoundation#1260)

Signed-off-by: wep21 <[email protected]>

* handle wrong route (autowarefoundation#1312)

Signed-off-by: wep21 <[email protected]>

Co-authored-by: Yukihiro Saito <[email protected]>

* Dealing with wrong image size (autowarefoundation#1320)

* Dealing with wrong image size

* Fix typo

Signed-off-by: wep21 <[email protected]>

Co-authored-by: Yukihiro Saito <[email protected]>

* Perception components (autowarefoundation#1368)

* [bev_optical_flow]: component node

Signed-off-by: wep21 <[email protected]>

* [object_merger]: component node

Signed-off-by: wep21 <[email protected]>

* [object_range_splitter]: component node

Signed-off-by: wep21 <[email protected]>

* [shape_estimation]: component node

Signed-off-by: wep21 <[email protected]>

* [map_based_prediction]: component node

Signed-off-by: wep21 <[email protected]>

* [naive_path_prediction]: component node

Signed-off-by: wep21 <[email protected]>

* [roi_image_saver]: component node

Signed-off-by: wep21 <[email protected]>

* [lidar_apollo_instance_segmentation]: component node

Signed-off-by: wep21 <[email protected]>

* [object_flow_fusion]: component node

Signed-off-by: wep21 <[email protected]>

* [traffic_light_map_based_detector]: component node

Signed-off-by: wep21 <[email protected]>

* [dynamic_object_visualization]: component node

Signed-off-by: wep21 <[email protected]>

* Fix typo

Signed-off-by: wep21 <[email protected]>

* Use sensor data qos for traffic light recognition (autowarefoundation#1440)

Signed-off-by: wep21 <[email protected]>

* Fix/ssd fine detector (autowarefoundation#1421) (autowarefoundation#1468)

* modify fitInFrame

* cosmetic change

* bug fix

* bug fix

* cosmetic change

Co-authored-by: Yukihiro Saito <[email protected]>

* Feature/tl map based detector ros2 (autowarefoundation#1475)

* refactor and rectified distortion (autowarefoundation#1397)

* refactor and rectified distortion

* fix typo

* cosmetic change

* add readme

* cosmetic change

* cosmetic change

* update readme

* Update perception/traffic_light_recognition/traffic_light_map_based_detector/src/node.cpp

Co-authored-by: Akihito Ohsato <[email protected]>

* use image geometry (autowarefoundation#1439)

Co-authored-by: Akihito Ohsato <[email protected]>

* Apply lint

Signed-off-by: wep21 <[email protected]>

Co-authored-by: Yukihiro Saito <[email protected]>
Co-authored-by: Akihito Ohsato <[email protected]>

* Add pre-commit (autowarefoundation#1560)

* add pre-commit

* add pre-commit-config

* add additional settings for private repository

* use default pre-commit-config

* update pre-commit setting

* Ignore whitespace for line breaks in markdown

* Update .github/workflows/pre-commit.yml

Co-authored-by: Kazuki Miyahara <[email protected]>

* exclude svg

* remove pretty-format-json

* add double-quote-string-fixer

* consider COLCON_IGNORE file when seaching modified package

* format file

* pre-commit fixes

* Update pre-commit.yml

* Update .pre-commit-config.yaml

Co-authored-by: Kazuki Miyahara <[email protected]>
Co-authored-by: pre-commit <[email protected]>
Co-authored-by: Kenji Miyake <[email protected]>

* Fix build error with TensorRT v8 (autowarefoundation#1612)

* Fix build error with TensorRT v8

Signed-off-by: wep21 <[email protected]>

* Fix typo

Signed-off-by: wep21 <[email protected]>

* Add markdownlint and prettier (autowarefoundation#1661)

* Add markdownlint and prettier

Signed-off-by: Kenji Miyake <[email protected]>

* Ignore .param.yaml

Signed-off-by: Kenji Miyake <[email protected]>

* Apply format

Signed-off-by: Kenji Miyake <[email protected]>

* suppress warnings for declare parameters (autowarefoundation#1724)

* fix for lanelet2_extension

* fix for traffic light ssd fine detector

* fix for topic_state_monitor

* fix for dummy diag publisher

* fix for remote cmd converter

* fix for vehicle_info_util

* fix for multi object tracker

* fix for freespace planner

* fix for autoware_error_monitor

* add Werror for multi object tracker

* fix for multi object tracker

* add Werror for liraffic light ssd fine detector

* add Werror for topic state monitor

* add Werror

* add Werror

* add Werror

* add Werror

* fix style

* suppress warnings for traffic light classifier (autowarefoundation#1762)

* add Werror

* fix type

* fix unused

* Fix -Wunused-parameter (autowarefoundation#1836)

* Fix -Wunused-parameter

Signed-off-by: Kenji Miyake <[email protected]>

* Fix mistake

Signed-off-by: Kenji Miyake <[email protected]>

* fix spell

* Fix lint issues

Signed-off-by: Kenji Miyake <[email protected]>

* Ignore flake8 warnings

Signed-off-by: Kenji Miyake <[email protected]>

Co-authored-by: Hiroki OTA <[email protected]>

* Fix compiler warnings (autowarefoundation#1837)

* Fix -Wunused-private-field

Signed-off-by: Kenji Miyake <[email protected]>

* Fix -Wunused-variable

Signed-off-by: Kenji Miyake <[email protected]>

* Fix -Wformat-security

Signed-off-by: Kenji Miyake <[email protected]>

* Fix -Winvalid-constexpr

Signed-off-by: Kenji Miyake <[email protected]>

* Fix -Wdelete-non-abstract-non-virtual-dtor

Signed-off-by: Kenji Miyake <[email protected]>

* Fix -Wdelete-abstract-non-virtual-dtor

Signed-off-by: Kenji Miyake <[email protected]>

* Fix -Winconsistent-missing-override

Signed-off-by: Kenji Miyake <[email protected]>

* Fix -Wrange-loop-construct

Signed-off-by: Kenji Miyake <[email protected]>

* Fix "invalid application of 'sizeof' to an incomplete type"

Signed-off-by: Kenji Miyake <[email protected]>

* Ignore -Wgnu-anonymous-struct and -Wnested-anon-types

Signed-off-by: Kenji Miyake <[email protected]>

* Fix lint

Signed-off-by: Kenji Miyake <[email protected]>

* Ignore -Wno-deprecated-declarations in CUDA-related packages

Signed-off-by: Kenji Miyake <[email protected]>

* Fix mistake

Signed-off-by: Kenji Miyake <[email protected]>

* Fix -Wunused-parameter

Signed-off-by: Kenji Miyake <[email protected]>

* Fix clang warnings (autowarefoundation#1859)

* Fix -Wreturn-std-move

Signed-off-by: Kenji Miyake <[email protected]>

* Fix -Wunused-private-field

Signed-off-by: Kenji Miyake <[email protected]>

* Ignore -Wnonportable-include-path for mussp

Signed-off-by: Kenji Miyake <[email protected]>

* Fix -Wunused-const-variable

Signed-off-by: Kenji Miyake <[email protected]>

* Fix "can not be used when making a shared object"

Signed-off-by: Kenji Miyake <[email protected]>

* Invoke code formatter at pre-commit (autowarefoundation#1935)

* Run ament_uncrustify at pre-commit

* Reformat existing files
* Fix copyright and cpplint errors

Signed-off-by: Kenji Miyake <[email protected]>
Co-authored-by: Kenji Miyake <[email protected]>

* add sort-package-xml hook in pre-commit (autowarefoundation#1881)

* add sort xml hook in pre-commit

* change retval to exit_status

* rename

* add prettier plugin-xml

* use early return

* add license note

* add tier4 license

* restore prettier

* change license order

* move local hooks to public repo

* move prettier-xml to pre-commit-hooks-ros

* update version for bug-fix

* apply pre-commit

* Fix readme traffic light map based detector (autowarefoundation#2282)

* fix type of ~input/route in readme

* fix some grammar mistakes

* Change formatter to clang-format and black (autowarefoundation#2332)

* Revert "Temporarily comment out pre-commit hooks"

This reverts commit 748e9cdb145ce12f8b520bcbd97f5ff899fc28a3.

* Replace ament_lint_common with autoware_lint_common

Signed-off-by: Kenji Miyake <[email protected]>

* Remove ament_cmake_uncrustify and ament_clang_format

Signed-off-by: Kenji Miyake <[email protected]>

* Apply Black

Signed-off-by: Kenji Miyake <[email protected]>

* Apply clang-format

Signed-off-by: Kenji Miyake <[email protected]>

* Fix build errors

Signed-off-by: Kenji Miyake <[email protected]>

* Fix for cpplint

* Fix include double quotes to angle brackets

Signed-off-by: Kenji Miyake <[email protected]>

* Apply clang-format

Signed-off-by: Kenji Miyake <[email protected]>

* Fix build errors

Signed-off-by: Kenji Miyake <[email protected]>

* Add COLCON_IGNORE (autowarefoundation#500)

Signed-off-by: Kenji Miyake <[email protected]>

* [traffic_light_ssd_fine_detector] support autoware auto msgs (autowarefoundation#514)

* remove COLCON_IGNORE

* support autoware auto msgs

* fix sort pacakagge

* [traffic_light_map_based_detector] support autoware auto msgs (autowarefoundation#511)

* [traffic_light_classifier] support autoware auto msgs (autowarefoundation#504)

* update README of traffic_light_classifier (autowarefoundation#649)

* update README of traffic_light_classifier

* Update label explanation (autowarefoundation#652)

* Update README.md

* fix format

* fix markdown lint

* update readme

Co-authored-by: Taichi Higashide <[email protected]>

* add readme for traffic light ssd fine detector (autowarefoundation#637)

* declare variable before subscriber registration (autowarefoundation#705)

* update traffic light classifier readme (autowarefoundation#726)

* Update traffic light topic name (autowarefoundation#729)

* Update traffic light topic name

Signed-off-by: wep21 <[email protected]>

* Update traffic light topic name in perception

Signed-off-by: wep21 <[email protected]>

Co-authored-by: mitsudome-r <[email protected]>
Co-authored-by: Daichi Murakami <[email protected]>
Co-authored-by: Daisuke Nishimatsu <[email protected]>
Co-authored-by: Yukihiro Saito <[email protected]>
Co-authored-by: Taichi Higashide <[email protected]>
Co-authored-by: Kosuke Takeuchi <[email protected]>
Co-authored-by: Nikolai Morin <[email protected]>
Co-authored-by: Ryohsuke Mitsudome <[email protected]>
Co-authored-by: simon-t4 <[email protected]>
Co-authored-by: nik-tier4 <[email protected]>
Co-authored-by: Kazuki Miyahara <[email protected]>
Co-authored-by: Satoshi Tanaka <[email protected]>
Co-authored-by: Kenji Miyake <[email protected]>
Co-authored-by: taikitanaka3 <[email protected]>
Co-authored-by: tkimura4 <[email protected]>
Co-authored-by: Takamasa Horibe <[email protected]>
Co-authored-by: Takagi, Isamu <[email protected]>
Co-authored-by: Makoto Tokunaga <[email protected]>
Co-authored-by: Adam Dąbrowski <[email protected]>
Co-authored-by: Akihito Ohsato <[email protected]>
Co-authored-by: Keisuke Shima <[email protected]>
Co-authored-by: pre-commit <[email protected]>
Co-authored-by: Hiroki OTA <[email protected]>
Co-authored-by: Takeshi Ishita <[email protected]>
Co-authored-by: Kenji Miyake <[email protected]>
Co-authored-by: kminoda <[email protected]>
Co-authored-by: Yusuke Muramatsu <[email protected]>
taikitanaka3 referenced this pull request in tier4/autoware.universe Jun 27, 2022
* feat(perception_rviz_plugin): add 2d dummy bus with height

Signed-off-by: tanaka3 <[email protected]>

* ci(pre-commit): autofix

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
taikitanaka3 referenced this pull request in tier4/autoware.universe Jun 27, 2022
* feat(perception_rviz_plugin): add 2d dummy bus with height

Signed-off-by: tanaka3 <[email protected]>

* ci(pre-commit): autofix

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
tkimura4 referenced this pull request in tier4/autoware.universe Jun 27, 2022
* feat(tier4_perception_rviz_plugin): interactive pedestrian for rviz plugin (#534)

* feat(tier4_perception_rviz_plugin): add mouse event util function

Signed-off-by: satoshi-ota <[email protected]>

* feat(tier4_perception_rviz_plugin): delete option for QT

Signed-off-by: satoshi-ota <[email protected]>

* fix(tier4_perception_rviz_plugin): remove interactive_pedestrian.cpp

Signed-off-by: satoshi-ota <[email protected]>

* feat(tier4_perception_rviz_plugin): expand conventional 2D dummy pedestrian to be able to interact

Signed-off-by: satoshi-ota <[email protected]>

* fix(tier4_perception_rviz_plugin): change for void

Signed-off-by: satoshi-ota <[email protected]>

* fix(tier4_perception_rviz_plugin): not use std::numeric_limits::epsilon()

Signed-off-by: satoshi-ota <[email protected]>

* feat(tier4_perception_rviz_plugin): plugin supports interactive manipulation for dummy CAR & UNKNOWN object (#554)

* fix(tier4_perception_rviz_plugin): split pedestrian.hpp(cpp) into two parts

Signed-off-by: satoshi-ota <[email protected]>

* fix(tier4_perception_rviz_plugin): modify logic of velocity & pose calculation

Signed-off-by: satoshi-ota <[email protected]>

* feat(tier4_perception_rviz_plugin): plugin supports 2D Interactive dummy car

Signed-off-by: satoshi-ota <[email protected]>

* feat(tier4_perception_rviz_plugin): plugin supports 2D Interactive unknown object

Signed-off-by: satoshi-ota <[email protected]>

* refactor(tier4_perception_rviz_plugin): use common class

Signed-off-by: satoshi-ota <[email protected]>

* fix(tier4_perception_rviz_plugin): fix dummy pedestrian shape (#571)

* fix(tier4_perception_rviz_plugin): set interactive object velocity to zero on releasing the object (#686)

Signed-off-by: satoshi-ota <[email protected]>

* fix(tier4_perception_rviz_plugin): modify build error in rolling (#791)

* fix(tier4_perception_rviz_plugin): modify build error in rolling

Signed-off-by: wep21 <[email protected]>

* fix(tier4_perception_rviz_plugin): add target compile definitions

Signed-off-by: wep21 <[email protected]>

* feat(perception_rviz_plugin): add 2d dummy bus with height (#903)

* feat(perception_rviz_plugin): add 2d dummy bus with height

Signed-off-by: tanaka3 <[email protected]>

* ci(pre-commit): autofix

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Co-authored-by: Satoshi OTA <[email protected]>
Co-authored-by: Takeshi Miura <[email protected]>
Co-authored-by: Daisuke Nishimatsu <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
boyali referenced this pull request in boyali/autoware.universe Sep 28, 2022
* feat(perception_rviz_plugin): add 2d dummy bus with height

Signed-off-by: tanaka3 <[email protected]>

* ci(pre-commit): autofix

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
boyali referenced this pull request in boyali/autoware.universe Oct 3, 2022
* feat(perception_rviz_plugin): add 2d dummy bus with height

Signed-off-by: tanaka3 <[email protected]>

* ci(pre-commit): autofix

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
boyali referenced this pull request in boyali/autoware.universe Oct 3, 2022
* feat(perception_rviz_plugin): add 2d dummy bus with height

Signed-off-by: tanaka3 <[email protected]>

* ci(pre-commit): autofix

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
boyali referenced this pull request in boyali/autoware.universe Oct 19, 2022
* feat(perception_rviz_plugin): add 2d dummy bus with height

Signed-off-by: tanaka3 <[email protected]>

* ci(pre-commit): autofix

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
YoshiRi pushed a commit to YoshiRi/autoware.universe that referenced this pull request Oct 16, 2023
…idance

hotfix(avoidance): cherry pick bug fix commits
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.

4 participants