tier4 / CalibrationTools

sensor calibration tools for autonomous driving and robotics
GNU General Public License v3.0
121 stars 44 forks source link

fix(sensor_calibration_manager): fixed sensor calibration manager related issues #170

Closed knzo25 closed 7 months ago

knzo25 commented 7 months ago

Description

Upon the release of the new api, two problems were encountered:

Tickets:

Tests performed

After the Launcher configuration widget, the sensor calibration manager executes the respective launchers. In this condition, even if we close the sensor calibration manager via widget or console, no zombies remain

(can be checked with ps -aux | grep ros) or similar

Previously, if we ran the sensor calibration manager until after the launchers were executed with no autoware nor rosbag (which publishes the /tf_static), even if we started autoware or the rosbag afterwards, the calibration tool never became able to calibrate due to the design of the internal state machine. With this PR, is should be fixed

Notes for reviewers

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.

knzo25 commented 7 months ago

@vividf Did you confirm that the changes fixes the issues on your environment? (the two issues this PR addresses)

vividf commented 7 months ago

@knzo25 Yes, the button will be enabled even /tf comes later, also all of the processes are killed.

knzo25 commented 7 months ago

@vividf Thanks for the check !