Skip to content

Commit 1bee188

Browse files
authored
Merge branch 'main' into refactor/occupancy_grid_map_outlier_filter
2 parents af4df4b + 7da5afd commit 1bee188

File tree

726 files changed

+112923
-11211
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

726 files changed

+112923
-11211
lines changed

.cspell.json

+5-2
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
{
2-
"ignorePaths": ["perception/bytetrack/lib/**"],
2+
"ignorePaths": [
3+
"perception/bytetrack/lib/**",
4+
"planning/behavior_velocity_intersection_module/scripts/**"
5+
],
36
"ignoreRegExpList": [],
4-
"words": ["dltype", "tvmgen"]
7+
"words": ["dltype", "tvmgen", "fromarray"]
58
}

.github/CODEOWNERS

+27-20
Large diffs are not rendered by default.

.github/PULL_REQUEST_TEMPLATE/small-change.md

+4
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,10 @@ Not applicable.
1515

1616
Not applicable.
1717

18+
## Interface changes
19+
20+
<!-- Describe any changed interfaces, such as topics, services, or parameters, including debugging interfaces -->
21+
1822
## Pre-review checklist for the PR author
1923

2024
The PR author **must** check the checkboxes below when creating the PR.

.github/PULL_REQUEST_TEMPLATE/standard-change.md

+13
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,19 @@
1818

1919
<!-- Describe any changed interfaces, such as topics, services, or parameters. -->
2020

21+
### ROS Topic Changes
22+
23+
<!-- | Topic Name | Type | Direction | Update Description | -->
24+
<!-- | ---------------- | ------------------- | --------- | ------------------------------------------------------------- | -->
25+
<!-- | `/example_topic` | `std_msgs/String` | Subscribe | Description of what the topic is used for in the system | -->
26+
<!-- | `/another_topic` | `sensor_msgs/Image` | Publish | Also explain if it is added / modified / deleted with the PR | -->
27+
28+
### ROS Parameter Changes
29+
30+
<!-- | Parameter Name | Default Value | Update Description | -->
31+
<!-- | -------------------- | ------------- | --------------------------------------------------- | -->
32+
<!-- | `example_parameters` | `1.0` | Describe the parameter and also explain the updates | -->
33+
2134
## Effects on system behavior
2235

2336
<!-- Describe how this PR affects the system behavior. -->

.github/workflows/build-and-test-arm64.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ jobs:
2323
build-depends-repos: build_depends.repos
2424
steps:
2525
- name: Check out repository
26-
uses: actions/checkout@v3
26+
uses: actions/checkout@v4
2727

2828
- name: Show disk space before the tasks
2929
run: df -h

.github/workflows/build-and-test-differential-arm64.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ jobs:
3333
build-depends-repos: build_depends.repos
3434
steps:
3535
- name: Check out repository
36-
uses: actions/checkout@v3
36+
uses: actions/checkout@v4
3737
with:
3838
fetch-depth: 0
3939

.github/workflows/build-and-test-differential.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ jobs:
3232
build-depends-repos: build_depends.repos
3333
steps:
3434
- name: Check out repository
35-
uses: actions/checkout@v3
35+
uses: actions/checkout@v4
3636
with:
3737
fetch-depth: 0
3838

@@ -80,7 +80,7 @@ jobs:
8080
container: ghcr.io/autowarefoundation/autoware-openadk:latest-prebuilt-cuda
8181
steps:
8282
- name: Check out repository
83-
uses: actions/checkout@v3
83+
uses: actions/checkout@v4
8484
with:
8585
fetch-depth: 0
8686

.github/workflows/build-and-test.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ jobs:
2525
build-depends-repos: build_depends.repos
2626
steps:
2727
- name: Check out repository
28-
uses: actions/checkout@v3
28+
uses: actions/checkout@v4
2929

3030
- name: Show disk space before the tasks
3131
run: df -h

.github/workflows/check-build-depends.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ jobs:
2020
build-depends-repos: build_depends.repos
2121
steps:
2222
- name: Check out repository
23-
uses: actions/checkout@v3
23+
uses: actions/checkout@v4
2424

2525
- name: Remove exec_depend
2626
uses: autowarefoundation/autoware-github-actions/remove-exec-depend@v1

.github/workflows/clang-tidy-pr-comments-manually.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ jobs:
1111
runs-on: ubuntu-latest
1212
steps:
1313
- name: Check out repository
14-
uses: actions/checkout@v3
14+
uses: actions/checkout@v4
1515

1616
- name: Download analysis results
1717
run: |
@@ -36,7 +36,7 @@ jobs:
3636
3737
- name: Check out PR head
3838
if: ${{ steps.check-fixes-yaml-existence.outputs.exists == 'true' }}
39-
uses: actions/checkout@v3
39+
uses: actions/checkout@v4
4040
with:
4141
repository: ${{ steps.set-variables.outputs.pr-head-repo }}
4242
ref: ${{ steps.set-variables.outputs.pr-head-ref }}

.github/workflows/clang-tidy-pr-comments.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ jobs:
1313
runs-on: ubuntu-latest
1414
steps:
1515
- name: Check out repository
16-
uses: actions/checkout@v3
16+
uses: actions/checkout@v4
1717

1818
- name: Download analysis results
1919
run: |
@@ -37,7 +37,7 @@ jobs:
3737
3838
- name: Check out PR head
3939
if: ${{ steps.check-fixes-yaml-existence.outputs.exists == 'true' }}
40-
uses: actions/checkout@v3
40+
uses: actions/checkout@v4
4141
with:
4242
repository: ${{ steps.set-variables.outputs.pr-head-repo }}
4343
ref: ${{ steps.set-variables.outputs.pr-head-ref }}

.github/workflows/delete-closed-pr-docs.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ jobs:
1010
runs-on: ubuntu-latest
1111
steps:
1212
- name: Check out repository
13-
uses: actions/checkout@v3
13+
uses: actions/checkout@v4
1414
with:
1515
fetch-depth: 0
1616

.github/workflows/deploy-docs.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ jobs:
3030
runs-on: ubuntu-latest
3131
steps:
3232
- name: Check out repository
33-
uses: actions/checkout@v3
33+
uses: actions/checkout@v4
3434
with:
3535
fetch-depth: 0
3636
ref: ${{ github.event.pull_request.head.sha }}

.github/workflows/json-schema-check.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ jobs:
1010
outputs:
1111
run-check: ${{ steps.paths_filter.outputs.json_or_yaml }}
1212
steps:
13-
- uses: actions/checkout@v3
13+
- uses: actions/checkout@v4
1414
- uses: dorny/paths-filter@v3
1515
id: paths_filter
1616
with:
@@ -25,7 +25,7 @@ jobs:
2525
runs-on: ubuntu-latest
2626
steps:
2727
- name: Check out repository
28-
uses: actions/checkout@v3
28+
uses: actions/checkout@v4
2929

3030
- name: Run json-schema-check
3131
uses: autowarefoundation/autoware-github-actions/json-schema-check@v1

build_depends.repos

+3-4
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,7 @@ repositories:
4141
type: git
4242
url: https://github.com/MORAI-Autonomous/MORAI-ROS2_morai_msgs.git
4343
version: main
44-
#vehicle
45-
vehicle/sample_vehicle_launch:
44+
universe/external/glog: # TODO: to use isGoogleInitialized() API in v0.6.0. Remove when the rosdep glog version is updated to v0.6.0 (already updated in Ubuntu 24.04)
4645
type: git
47-
url: https://github.com/autowarefoundation/sample_vehicle_launch.git
48-
version: main
46+
url: https://github.com/tier4/glog.git
47+
version: v0.6.0_t4-ros

common/autoware_auto_common/include/autoware_auto_common/common/types.hpp

+2
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,9 @@ namespace types
3737
// We don't currently require code to comply to MISRA, but we should try to where it is
3838
// easily possible.
3939
using bool8_t = bool;
40+
#if __cplusplus < 201811L || !__cpp_char8_t
4041
using char8_t = char;
42+
#endif
4143
using uchar8_t = unsigned char;
4244
// If we ever compile on a platform where this is not true, float32_t and float64_t definitions
4345
// need to be adjusted.

common/autoware_auto_perception_rviz_plugin/include/autoware_auto_perception_rviz_plugin/object_detection/object_polygon_detail.hpp

+5-1
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,9 @@ struct ObjectPropertyValues
5757
float alpha{0.999F};
5858
};
5959

60+
// Control object marker visualization
61+
enum class ObjectFillType { Skeleton, Fill };
62+
6063
// Map defining colors according to value of label field in ObjectClassification msg
6164
const std::map<
6265
autoware_auto_perception_msgs::msg::ObjectClassification::_label_type, ObjectPropertyValues>
@@ -87,7 +90,8 @@ get_shape_marker_ptr(
8790
const autoware_auto_perception_msgs::msg::Shape & shape_msg,
8891
const geometry_msgs::msg::Point & centroid, const geometry_msgs::msg::Quaternion & orientation,
8992
const std_msgs::msg::ColorRGBA & color_rgba, const double & line_width,
90-
const bool & is_orientation_available = true);
93+
const bool & is_orientation_available = true,
94+
const ObjectFillType fill_type = ObjectFillType::Skeleton);
9195

9296
AUTOWARE_AUTO_PERCEPTION_RVIZ_PLUGIN_PUBLIC visualization_msgs::msg::Marker::SharedPtr
9397
get_2d_shape_marker_ptr(

common/autoware_auto_perception_rviz_plugin/include/autoware_auto_perception_rviz_plugin/object_detection/object_polygon_display_base.hpp

+13-1
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,12 @@ class AUTOWARE_AUTO_PERCEPTION_RVIZ_PLUGIN_PUBLIC ObjectPolygonDisplayBase
112112
m_confidence_interval_property->addOption("95%", 2);
113113
m_confidence_interval_property->addOption("99%", 3);
114114

115+
m_object_fill_type_property = new rviz_common::properties::EnumProperty(
116+
"Object Fill Type", "skeleton", "Change object fill type in visualization", this);
117+
m_object_fill_type_property->addOption(
118+
"skeleton", static_cast<int>(detail::ObjectFillType::Skeleton));
119+
m_object_fill_type_property->addOption("Fill", static_cast<int>(detail::ObjectFillType::Fill));
120+
115121
// iterate over default values to create and initialize the properties.
116122
for (const auto & map_property_it : detail::kDefaultObjectPropertyValues) {
117123
const auto & class_property_values = map_property_it.second;
@@ -189,9 +195,13 @@ class AUTOWARE_AUTO_PERCEPTION_RVIZ_PLUGIN_PUBLIC ObjectPolygonDisplayBase
189195
const bool & is_orientation_available) const
190196
{
191197
const std_msgs::msg::ColorRGBA color_rgba = get_color_rgba(labels);
198+
const auto fill_type =
199+
static_cast<detail::ObjectFillType>(m_object_fill_type_property->getOptionInt());
200+
192201
if (m_display_type_property->getOptionInt() == 0) {
193202
return detail::get_shape_marker_ptr(
194-
shape_msg, centroid, orientation, color_rgba, line_width, is_orientation_available);
203+
shape_msg, centroid, orientation, color_rgba, line_width, is_orientation_available,
204+
fill_type);
195205
} else if (m_display_type_property->getOptionInt() == 1) {
196206
return detail::get_2d_shape_marker_ptr(
197207
shape_msg, centroid, orientation, color_rgba, line_width, is_orientation_available);
@@ -526,6 +536,8 @@ class AUTOWARE_AUTO_PERCEPTION_RVIZ_PLUGIN_PUBLIC ObjectPolygonDisplayBase
526536
rviz_common::properties::EnumProperty * m_simple_visualize_mode_property;
527537
// Property to set confidence interval of state estimations
528538
rviz_common::properties::EnumProperty * m_confidence_interval_property;
539+
// Property to set visualization type
540+
rviz_common::properties::EnumProperty * m_object_fill_type_property;
529541
// Property to enable/disable label visualization
530542
rviz_common::properties::BoolProperty m_display_label_property;
531543
// Property to enable/disable uuid visualization

common/autoware_auto_perception_rviz_plugin/src/object_detection/object_polygon_detail.cpp

+19-7
Original file line numberDiff line numberDiff line change
@@ -495,23 +495,37 @@ visualization_msgs::msg::Marker::SharedPtr get_shape_marker_ptr(
495495
const autoware_auto_perception_msgs::msg::Shape & shape_msg,
496496
const geometry_msgs::msg::Point & centroid, const geometry_msgs::msg::Quaternion & orientation,
497497
const std_msgs::msg::ColorRGBA & color_rgba, const double & line_width,
498-
const bool & is_orientation_available)
498+
const bool & is_orientation_available, const ObjectFillType fill_type)
499499
{
500500
auto marker_ptr = std::make_shared<Marker>();
501501
marker_ptr->ns = std::string("shape");
502+
marker_ptr->color = color_rgba;
503+
marker_ptr->scale.x = line_width;
502504

503505
using autoware_auto_perception_msgs::msg::Shape;
504506
if (shape_msg.type == Shape::BOUNDING_BOX) {
505-
marker_ptr->type = visualization_msgs::msg::Marker::LINE_LIST;
506-
calc_bounding_box_line_list(shape_msg, marker_ptr->points);
507+
if (fill_type == ObjectFillType::Skeleton) {
508+
marker_ptr->type = visualization_msgs::msg::Marker::LINE_LIST;
509+
calc_bounding_box_line_list(shape_msg, marker_ptr->points);
510+
} else if (fill_type == ObjectFillType::Fill) {
511+
marker_ptr->type = visualization_msgs::msg::Marker::CUBE;
512+
marker_ptr->scale = shape_msg.dimensions;
513+
marker_ptr->color.a = 0.75f;
514+
}
507515
if (is_orientation_available) {
508516
calc_bounding_box_direction_line_list(shape_msg, marker_ptr->points);
509517
} else {
510518
calc_bounding_box_orientation_line_list(shape_msg, marker_ptr->points);
511519
}
512520
} else if (shape_msg.type == Shape::CYLINDER) {
513-
marker_ptr->type = visualization_msgs::msg::Marker::LINE_LIST;
514-
calc_cylinder_line_list(shape_msg, marker_ptr->points);
521+
if (fill_type == ObjectFillType::Skeleton) {
522+
marker_ptr->type = visualization_msgs::msg::Marker::LINE_LIST;
523+
calc_cylinder_line_list(shape_msg, marker_ptr->points);
524+
} else if (fill_type == ObjectFillType::Fill) {
525+
marker_ptr->type = visualization_msgs::msg::Marker::CYLINDER;
526+
marker_ptr->scale = shape_msg.dimensions;
527+
marker_ptr->color.a = 0.75f;
528+
}
515529
} else if (shape_msg.type == Shape::POLYGON) {
516530
marker_ptr->type = visualization_msgs::msg::Marker::LINE_LIST;
517531
calc_polygon_line_list(shape_msg, marker_ptr->points);
@@ -523,8 +537,6 @@ visualization_msgs::msg::Marker::SharedPtr get_shape_marker_ptr(
523537
marker_ptr->action = visualization_msgs::msg::Marker::MODIFY;
524538
marker_ptr->pose = to_pose(centroid, orientation);
525539
marker_ptr->lifetime = rclcpp::Duration::from_seconds(0.15);
526-
marker_ptr->scale.x = line_width;
527-
marker_ptr->color = color_rgba;
528540

529541
return marker_ptr;
530542
}

0 commit comments

Comments
 (0)