Skip to content

Commit 142b243

Browse files
committed
fix: address review feedback on planning params and docker-compose
Add explicit aggregation.enabled: false to planning_params.yaml for clarity. Add comment about CI/default profile mutual exclusivity in docker-compose.yml.
1 parent 209eb19 commit 142b243

File tree

2 files changed

+5
-0
lines changed

2 files changed

+5
-0
lines changed

demos/multi_ecu_aggregation/config/planning_params.yaml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,9 @@ diagnostics:
1717
create_synthetic_components: false
1818
create_synthetic_areas: false
1919

20+
aggregation:
21+
enabled: false
22+
2023
cors:
2124
allowed_origins: ["*"]
2225
allowed_methods: ["GET", "PUT", "POST", "DELETE", "OPTIONS"]

demos/multi_ecu_aggregation/docker-compose.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -57,6 +57,8 @@ services:
5757
networks: [medkit-net]
5858

5959
# --- CI services (headless, no web UI) ---
60+
# NOTE: CI and default profiles are mutually exclusive - do not start both
61+
# simultaneously. CI services reuse hostnames for config compatibility.
6062

6163
# Perception ECU for CI testing
6264
perception-ecu-ci:

0 commit comments

Comments
 (0)