Skip to content

Commit fe76ae6

Browse files
committed
fix(demos/manymove_industrial): declare manymove-planning component + shellcheck source directives
CI gate hit two issues: 1. Smoke test POST /components/manymove-planning/scripts/<x>/executions returned 404 because the manifest only declared manymove-bt; the container_scripts directory name (manymove-planning) had no matching component. Added a manymove-planning component entry mirroring the moveit_pick_place demo pattern (moveit-planning component + same-named container_scripts dir). 2. shellcheck SC1091 on every 'source /opt/...setup.bash' line because those files do not exist on the host runner. Added 'shellcheck source=/dev/null' directives, matching the multi_ecu_aggregation convention.
1 parent 96253c3 commit fe76ae6

5 files changed

Lines changed: 12 additions & 0 deletions

File tree

demos/manymove_industrial/config/manymove_industrial_manifest.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,11 @@ components:
3838
type: "controller"
3939
description: "BehaviorTree.CPP orchestration of pick/place sequences"
4040
area: planning
41+
- id: manymove-planning
42+
name: "manymove planning subsystem"
43+
type: "controller"
44+
description: "Logical grouping for planning-side container scripts (collision inject, restore, self-test)"
45+
area: planning
4146
- id: move-group
4247
name: "MoveIt move_group"
4348
type: "planner"

demos/manymove_industrial/container_scripts/manymove-planning/arm-self-test/script.bash

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
#!/bin/bash
22
set -eu
33

4+
# shellcheck source=/dev/null
45
source /opt/ros/jazzy/setup.bash
56

67
CODE="MANYMOVE_SELFTEST"

demos/manymove_industrial/container_scripts/manymove-planning/inject-collision/script.bash

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,9 @@
77
# variables when sourced from a fresh shell.
88
set -e
99

10+
# shellcheck source=/dev/null
1011
source /opt/ros/jazzy/setup.bash
12+
# shellcheck source=/dev/null
1113
source /opt/manymove_ws/install/setup.bash
1214

1315
# Default xArm7 prefix in the upstream launch is empty; the BT keys live in

demos/manymove_industrial/container_scripts/manymove-planning/inject-soft-fault/script.bash

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,9 @@
55
# RETRIES_EXHAUSTED.
66
set -e
77

8+
# shellcheck source=/dev/null
89
source /opt/ros/jazzy/setup.bash
10+
# shellcheck source=/dev/null
911
source /opt/manymove_ws/install/setup.bash
1012

1113
ros2 action send_goal --feedback /add_collision_object \

demos/manymove_industrial/container_scripts/manymove-planning/restore-normal/script.bash

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,9 @@
44
# operators can rerun without restarting the demo.
55
set -e
66

7+
# shellcheck source=/dev/null
78
source /opt/ros/jazzy/setup.bash
9+
# shellcheck source=/dev/null
810
source /opt/manymove_ws/install/setup.bash
911

1012
PREFIX="${ROBOT_PREFIX:-}"

0 commit comments

Comments
 (0)