Skip to content

Commit

Permalink
fix: change to GenIOU
Browse files Browse the repository at this point in the history
Signed-off-by: badai-nguyen <[email protected]>
  • Loading branch information
badai-nguyen committed Jan 10, 2025
1 parent 99c0c88 commit 25e33d1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,6 @@
<remap from="input/detected_rois" to="$(var input/detected_rois)"/>
<remap from="input/rough_rois" to="$(var input/rough_rois)"/>
<remap from="input/camera_info" to="input/camera_info"/>
<param name="max_iou_threshold" value="0.0"/>
<param name="max_iou_threshold" value="-0.5"/>
</node>
</launch>
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ void TrafficLightSelectorNode::objectsCallback(
static_cast<int>(detected_roi.feature.roi.y_offset) - det_roi_shift_y, 0,
static_cast<int>(image_height_ - detected_roi.feature.roi.height));

Check warning on line 130 in perception/autoware_traffic_light_selector/src/traffic_light_selector_node.cpp

View check run for this annotation

Codecov / codecov/patch

perception/autoware_traffic_light_selector/src/traffic_light_selector_node.cpp#L125-L130

Added lines #L125 - L130 were not covered by tests

double iou = utils::calIOU(expect_roi.roi, detected_roi_shifted);
double iou = utils::getGenIoU(expect_roi.roi, detected_roi_shifted);

Check warning on line 132 in perception/autoware_traffic_light_selector/src/traffic_light_selector_node.cpp

View check run for this annotation

Codecov / codecov/patch

perception/autoware_traffic_light_selector/src/traffic_light_selector_node.cpp#L132

Added line #L132 was not covered by tests
if (iou > max_iou) {
max_iou = iou;
max_iou_roi = detected_roi.feature.roi;

Check warning on line 135 in perception/autoware_traffic_light_selector/src/traffic_light_selector_node.cpp

View check run for this annotation

Codecov / codecov/patch

perception/autoware_traffic_light_selector/src/traffic_light_selector_node.cpp#L135

Added line #L135 was not covered by tests
Expand Down

0 comments on commit 25e33d1

Please sign in to comment.