Skip to content

Commit bac2055

Browse files
authored
feat(diagnostic_graph_aggregator): add config files for gen2 (autowarefoundation#1144)
* create sensing and hardware diagnostic config files for gen2 Signed-off-by: Ericpotato <[email protected]> * set sensing diagnostic config for gen2 Signed-off-by: Ericpotato <[email protected]> * style(pre-commit): autofix * change pndr abbreviation to full name pandar Signed-off-by: Ericpotato <[email protected]> * change diagnostic_graph_aggregator entrance for gen2 Signed-off-by: Ericpotato <[email protected]> * style(pre-commit): autofix * fix typo Signed-off-by: Ericpotato <[email protected]> * style(pre-commit): autofix * fix typo, fix diagnostic_graph_aggregator entrance Signed-off-by: Ericpotato <[email protected]> * fix lsim diagnostic_graph_aggregator entrance Signed-off-by: Ericpotato <[email protected]> * fix gyro_bias node and name, comment out cameras on addon ECU Signed-off-by: Ericpotato <[email protected]> * fix comments Signed-off-by: Ericpotato <[email protected]> * change lsim diagnostic_graph_aggregator config path Signed-off-by: Ericpotato <[email protected]> * style(pre-commit): autofix --------- Signed-off-by: Ericpotato <[email protected]>
1 parent f11570e commit bac2055

File tree

6 files changed

+406
-4
lines changed

6 files changed

+406
-4
lines changed
Lines changed: 73 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,73 @@
1+
files:
2+
- { path: $(dirname)/control.yaml }
3+
- { path: $(dirname)/hardware.yaml }
4+
- { path: $(dirname)/localization.yaml }
5+
- { path: $(dirname)/map.yaml }
6+
- { path: $(dirname)/perception.yaml }
7+
- { path: $(dirname)/planning.yaml }
8+
# - { path: $(dirname)/others.yaml }
9+
- { path: $(dirname)/sensing_gen2.yaml }
10+
- { path: $(dirname)/system.yaml }
11+
- { path: $(dirname)/vehicle.yaml }
12+
13+
units:
14+
- path: /autoware/modes/local
15+
type: ok
16+
17+
- path: /autoware/modes/remote
18+
type: ok
19+
20+
- path: /autoware/modes/stop
21+
type: ok
22+
23+
- path: /autoware/modes/autonomous
24+
type: and
25+
list:
26+
- { type: link, link: /control/autonomous_available }
27+
- { type: link, link: /hardware/autonomous_available }
28+
- { type: link, link: /localization/autonomous_available }
29+
- { type: link, link: /map/autonomous_available }
30+
- { type: link, link: /perception/autonomous_available }
31+
- { type: link, link: /planning/autonomous_available }
32+
- { type: link, link: /sensing/autonomous_available }
33+
- { type: link, link: /system/autonomous_available }
34+
- { type: link, link: /vehicle/autonomous_available }
35+
36+
- path: /autoware/modes/pull_over
37+
type: and
38+
list:
39+
- { type: link, link: /control/pull_over_available }
40+
- { type: link, link: /hardware/pull_over_available }
41+
- { type: link, link: /localization/pull_over_available }
42+
- { type: link, link: /map/pull_over_available }
43+
- { type: link, link: /perception/pull_over_available }
44+
- { type: link, link: /planning/pull_over_available }
45+
- { type: link, link: /sensing/pull_over_available }
46+
- { type: link, link: /system/pull_over_available }
47+
- { type: link, link: /vehicle/pull_over_available }
48+
49+
- path: /autoware/modes/comfortable_stop
50+
type: and
51+
list:
52+
- { type: link, link: /control/comfortable_stop_available }
53+
- { type: link, link: /hardware/comfortable_stop_available }
54+
- { type: link, link: /localization/comfortable_stop_available }
55+
- { type: link, link: /map/comfortable_stop_available }
56+
- { type: link, link: /perception/comfortable_stop_available }
57+
- { type: link, link: /planning/comfortable_stop_available }
58+
- { type: link, link: /sensing/comfortable_stop_available }
59+
- { type: link, link: /system/comfortable_stop_available }
60+
- { type: link, link: /vehicle/comfortable_stop_available }
61+
62+
- path: /autoware/modes/emergency_stop
63+
type: ok
64+
65+
- path: /autoware/debug/tools
66+
type: and
67+
list:
68+
- { type: link, link: /autoware/system/service_log_checker }
69+
70+
- path: /autoware/system/service_log_checker
71+
type: diag
72+
node: service_log_checker
73+
name: response_status
Lines changed: 287 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,287 @@
1+
units:
2+
- path: /sensing/autonomous_available
3+
type: and
4+
list:
5+
- { type: link, link: /sensing/emergency_stop }
6+
- { type: link, link: /sensing/pull_over }
7+
- { type: link, link: /sensing/comfortable_stop }
8+
9+
- path: /sensing/pull_over_available
10+
type: and
11+
list:
12+
- { type: link, link: /sensing/emergency_stop }
13+
- { type: link, link: /sensing/comfortable_stop }
14+
15+
- path: /sensing/comfortable_stop_available
16+
type: and
17+
list:
18+
- { type: link, link: /sensing/emergency_stop }
19+
20+
# ***************************************************
21+
22+
- path: /sensing/emergency_stop
23+
type: and
24+
list:
25+
- { type: link, link: /sensing/imu/002-connection }
26+
- { type: link, link: /sensing/lidar/concat/001-status }
27+
- { type: link, link: /sensing/lidar/pandar/001-connection }
28+
- { type: link, link: /sensing/lidar/pandar/002-temperature }
29+
- { type: link, link: /sensing/lidar/pandar/003-ptp }
30+
# - { type: link, link: /sensing/lidar/pandar/004-rpm } # Not in use
31+
# - { type: link, link: /sensing/lidar/pandar/005-voltage } # Not in use
32+
- { type: link, link: /sensing/camera/001-connection }
33+
34+
- path: /sensing/pull_over
35+
type: and
36+
37+
- path: /sensing/comfortable_stop
38+
type: and
39+
list:
40+
- { type: link, link: /sensing/lidar/pandar/002-temperature }
41+
42+
- path: /sensing/none
43+
type: and
44+
list:
45+
- { type: link, link: /sensing/imu/003-gyro_bias }
46+
47+
# ***************************************************
48+
# Intermediate paths
49+
50+
# camera 1, 7 and 8 on the addon ECU are not in use
51+
- path: /sensing/camera/001-connection
52+
type: and
53+
list:
54+
- { type: link, link: /sensing/camera/0/connection }
55+
# - { type: link, link: /sensing/camera/1/connection }
56+
- { type: link, link: /sensing/camera/2/connection }
57+
- { type: link, link: /sensing/camera/3/connection }
58+
- { type: link, link: /sensing/camera/4/connection }
59+
- { type: link, link: /sensing/camera/5/connection }
60+
- { type: link, link: /sensing/camera/6/connection }
61+
# - { type: link, link: /sensing/camera/7/connection }
62+
# - { type: link, link: /sensing/camera/8/connection }
63+
- { type: link, link: /sensing/camera/9/connection }
64+
- { type: link, link: /sensing/camera/10/connection }
65+
66+
- path: /sensing/lidar/pandar/001-connection
67+
type: and
68+
list:
69+
- { type: link, link: /sensing/lidar/front_left/connection }
70+
- { type: link, link: /sensing/lidar/front_right/connection }
71+
- { type: link, link: /sensing/lidar/side_left/connection }
72+
- { type: link, link: /sensing/lidar/side_right/connection }
73+
- { type: link, link: /sensing/lidar/top/connection }
74+
# - { type: link, link: /sensing/lidar/rear/connection }
75+
76+
- path: /sensing/lidar/pandar/002-temperature
77+
type: and
78+
list:
79+
- { type: link, link: /sensing/lidar/front_left/temperature }
80+
- { type: link, link: /sensing/lidar/front_right/temperature }
81+
- { type: link, link: /sensing/lidar/side_left/temperature }
82+
- { type: link, link: /sensing/lidar/side_right/temperature }
83+
- { type: link, link: /sensing/lidar/top/temperature }
84+
# - { type: link, link: /sensing/lidar/rear/temperature }
85+
86+
- path: /sensing/lidar/pandar/003-ptp
87+
type: and
88+
list:
89+
- { type: link, link: /sensing/lidar/front_left/ptp }
90+
- { type: link, link: /sensing/lidar/front_right/ptp }
91+
- { type: link, link: /sensing/lidar/side_left/ptp }
92+
- { type: link, link: /sensing/lidar/side_right/ptp }
93+
- { type: link, link: /sensing/lidar/top/ptp }
94+
# - { type: link, link: /sensing/lidar/rear/ptp }
95+
96+
# ***************************************************
97+
# Diagnostics paths
98+
99+
# The error code 001-connection simply follows pilot-auo vanilla's convention
100+
# xx1 does not estimate yaw_rate_status
101+
# - path: /sensing/imu/001-monitor
102+
# type: diag
103+
# node: imu_monitor
104+
# name: yaw_rate_status
105+
# timeout: 5.0
106+
107+
- path: /sensing/imu/002-connection
108+
type: diag
109+
node: topic_state_monitor_imu
110+
name: sensing_topic_status
111+
timeout: 1.0
112+
113+
- path: /sensing/imu/003-gyro_bias
114+
type: diag
115+
node: gyro_bias_validator
116+
name: gyro_bias_validator
117+
timeout: 1.0
118+
119+
- path: /sensing/camera/0/connection
120+
type: diag
121+
node: topic_state_monitor_camera0
122+
name: sensing_topic_status
123+
timeout: 1.0
124+
125+
# # camera 1, 7 and 8 on the addon ECU are not in use
126+
# - path: /sensing/camera/1/connection
127+
# type: diag
128+
# node: topic_state_monitor_camera1
129+
# name: sensing_topic_status
130+
# timeout: 1.0
131+
132+
- path: /sensing/camera/2/connection
133+
type: diag
134+
node: topic_state_monitor_camera2
135+
name: sensing_topic_status
136+
timeout: 1.0
137+
138+
- path: /sensing/camera/3/connection
139+
type: diag
140+
node: topic_state_monitor_camera3
141+
name: sensing_topic_status
142+
timeout: 1.0
143+
144+
- path: /sensing/camera/4/connection
145+
type: diag
146+
node: topic_state_monitor_camera4
147+
name: sensing_topic_status
148+
timeout: 1.0
149+
150+
- path: /sensing/camera/5/connection
151+
type: diag
152+
node: topic_state_monitor_camera5
153+
name: sensing_topic_status
154+
timeout: 1.0
155+
156+
- path: /sensing/camera/6/connection
157+
type: diag
158+
node: topic_state_monitor_camera6
159+
name: sensing_topic_status
160+
timeout: 1.0
161+
162+
# camera 1, 7 and 8 on the addon ECU are not in use
163+
# - path: /sensing/camera/7/connection
164+
# type: diag
165+
# node: topic_state_monitor_camera7
166+
# name: sensing_topic_status
167+
# timeout: 1.0
168+
169+
# - path: /sensing/camera/8/connection
170+
# type: diag
171+
# node: topic_state_monitor_camera8
172+
# name: sensing_topic_status
173+
# timeout: 1.0
174+
175+
- path: /sensing/camera/9/connection
176+
type: diag
177+
node: topic_state_monitor_camera9
178+
name: sensing_topic_status
179+
timeout: 1.0
180+
181+
- path: /sensing/camera/10/connection
182+
type: diag
183+
node: topic_state_monitor_camera10
184+
name: sensing_topic_status
185+
timeout: 1.0
186+
187+
- path: /sensing/lidar/concat/001-status
188+
type: diag
189+
node: concatenate_data
190+
name: concat_status
191+
timeout: 1.0
192+
193+
- path: /sensing/lidar/side_left/connection
194+
type: diag
195+
node: /sensing/lidar/side_left/hesai_ros_wrapper_node
196+
name: hesai_status
197+
timeout: 5.0
198+
- path: /sensing/lidar/side_left/ptp
199+
type: diag
200+
node: /sensing/lidar/side_left/hesai_ros_wrapper_node
201+
name: hesai_ptp
202+
timeout: 5.0
203+
- path: /sensing/lidar/side_left/temperature
204+
type: diag
205+
node: /sensing/lidar/side_left/hesai_ros_wrapper_node
206+
name: hesai_temperature
207+
timeout: 5.0
208+
209+
- path: /sensing/lidar/side_right/connection
210+
type: diag
211+
node: /sensing/lidar/side_right/hesai_ros_wrapper_node
212+
name: hesai_status
213+
timeout: 5.0
214+
- path: /sensing/lidar/side_right/ptp
215+
type: diag
216+
node: /sensing/lidar/side_right/hesai_ros_wrapper_node
217+
name: hesai_ptp
218+
timeout: 5.0
219+
- path: /sensing/lidar/side_right/temperature
220+
type: diag
221+
node: /sensing/lidar/side_right/hesai_ros_wrapper_node
222+
name: hesai_temperature
223+
timeout: 5.0
224+
225+
- path: /sensing/lidar/front_left/connection
226+
type: diag
227+
node: /sensing/lidar/front_left/hesai_ros_wrapper_node
228+
name: hesai_status
229+
timeout: 5.0
230+
- path: /sensing/lidar/front_left/ptp
231+
type: diag
232+
node: /sensing/lidar/front_left/hesai_ros_wrapper_node
233+
name: hesai_ptp
234+
timeout: 5.0
235+
- path: /sensing/lidar/front_left/temperature
236+
type: diag
237+
node: /sensing/lidar/front_left/hesai_ros_wrapper_node
238+
name: hesai_temperature
239+
timeout: 5.0
240+
241+
- path: /sensing/lidar/front_right/connection
242+
type: diag
243+
node: /sensing/lidar/front_right/hesai_ros_wrapper_node
244+
name: hesai_status
245+
timeout: 5.0
246+
- path: /sensing/lidar/front_right/ptp
247+
type: diag
248+
node: /sensing/lidar/front_right/hesai_ros_wrapper_node
249+
name: hesai_ptp
250+
timeout: 5.0
251+
- path: /sensing/lidar/front_right/temperature
252+
type: diag
253+
node: /sensing/lidar/front_right/hesai_ros_wrapper_node
254+
name: hesai_temperature
255+
timeout: 5.0
256+
257+
- path: /sensing/lidar/top/connection
258+
type: diag
259+
node: /sensing/lidar/top/hesai_ros_wrapper_node
260+
name: hesai_status
261+
timeout: 5.0
262+
- path: /sensing/lidar/top/ptp
263+
type: diag
264+
node: /sensing/lidar/top/hesai_ros_wrapper_node
265+
name: hesai_ptp
266+
timeout: 5.0
267+
- path: /sensing/lidar/top/temperature
268+
type: diag
269+
node: /sensing/lidar/top/hesai_ros_wrapper_node
270+
name: hesai_temperature
271+
timeout: 5.0
272+
273+
# - path: /sensing/lidar/rear/connection
274+
# type: diag
275+
# node: /sensing/lidar/rear/hesai_ros_wrapper_node
276+
# name: hesai_status
277+
# timeout: 5.0
278+
# - path: /sensing/lidar/rear/ptp
279+
# type: diag
280+
# node: /sensing/lidar/rear/hesai_ros_wrapper_node
281+
# name: hesai_ptp
282+
# timeout: 5.0
283+
# - path: /sensing/lidar/rear/temperature
284+
# type: diag
285+
# node: /sensing/lidar/rear/hesai_ros_wrapper_node
286+
# name: hesai_temperature
287+
# timeout: 5.0

autoware_launch/launch/autoware.launch.xml

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,18 @@
3939
<arg name="system_run_mode" default="online" description="run mode in system"/>
4040
<arg name="launch_system_monitor" default="true" description="launch system monitor"/>
4141
<arg name="launch_dummy_diag_publisher" default="false" description="launch dummy diag publisher"/>
42-
<arg name="diagnostic_graph_aggregator_graph_path" default="$(find-pkg-share autoware_launch)/config/system/tier4_diagnostics/autoware-main.yaml" description="diagnostic graph config"/>
42+
43+
<let
44+
name="diagnostic_graph_aggregator_graph_path"
45+
value="$(find-pkg-share autoware_launch)/config/system/tier4_diagnostics/autoware-main_gen2.yaml"
46+
if="$(eval &quot;'aip_xx1_gen2' == '$(var sensor_model)'&quot;)"
47+
/>
48+
<let
49+
name="diagnostic_graph_aggregator_graph_path"
50+
value="$(find-pkg-share autoware_launch)/config/system/tier4_diagnostics/autoware-main.yaml"
51+
unless="$(eval &quot;'aip_xx1_gen2' == '$(var sensor_model)'&quot;)"
52+
/>
53+
4354
<!-- Tools -->
4455
<arg name="rviz" default="true" description="launch rviz"/>
4556
<arg name="rviz_config_name" default="autoware.rviz" description="rviz config name"/>

autoware_launch/launch/components/tier4_system_component.launch.xml

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,16 @@
11
<?xml version="1.0"?>
22
<launch>
33
<arg name="diagnostic_graph_aggregator_param_path" default="$(find-pkg-share autoware_diagnostic_graph_aggregator)/config/default.param.yaml"/>
4-
<arg name="diagnostic_graph_aggregator_graph_path" default="$(find-pkg-share autoware_launch)/config/system/tier4_diagnostics/autoware-main.yaml"/>
4+
<arg
5+
name="diagnostic_graph_aggregator_graph_path"
6+
default="$(find-pkg-share autoware_launch)/config/system/tier4_diagnostics/autoware-main_gen2.yaml"
7+
if="$(eval &quot;'aip_xx1_gen2' == '$(var sensor_model)'&quot;)"
8+
/>
9+
<arg
10+
name="diagnostic_graph_aggregator_graph_path"
11+
default="$(find-pkg-share autoware_launch)/config/system/tier4_diagnostics/autoware-main.yaml"
12+
unless="$(eval &quot;'aip_xx1_gen2' == '$(var sensor_model)'&quot;)"
13+
/>
514

615
<!-- Depending on the sensor_model, different topics should be monitored -->
716
<let name="state_monitor_topic_yaml" value="topics_gen2.yaml" if="$(eval &quot;'aip_xx1_gen2' == '$(var sensor_model)'&quot;)"/>

0 commit comments

Comments
 (0)