Skip to content

ci: add libeigen3-dev to evaluator job to fix planning_node build#41

Merged
caixuf merged 3 commits into
mainfrom
copilot/fix-evaluator-job-failure-again
Jul 11, 2026
Merged

ci: add libeigen3-dev to evaluator job to fix planning_node build#41
caixuf merged 3 commits into
mainfrom
copilot/fix-evaluator-job-failure-again

Conversation

Copilot AI commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

The evaluator CI job was failing because libplanning_node.so was never built, causing the entire planning→control chain to be absent at runtime (0 Hz on planning/trajectory, control/raw_cmd, control/cmd; missing topology edges; collision detected).

Root cause

libeigen3-dev was missing from the CI install step. The dependency chain:

  • CMakeLists.txt: find_package(Eigen3) fails → FLOWENGINE_HAVE_EIGEN=FALSE
  • frenet_planner + frenet_bridge static libs are gated on FLOWENGINE_HAVE_EIGEN
  • modules/adas_nodes/CMakeLists.txt: find_library(FRENET_BRIDGE ...) finds nothing → planning_node target silently disabled
  • flow_launcher can't dlopen the planning plugin → no planning → no control → collision → all FAIL

Changes

  • .github/workflows/ci.yml: Add libeigen3-dev to the evaluator and integration-test job install steps (both jobs build modules/adas_nodes plugins)
-run: sudo apt-get update -qq && sudo apt-get install -y -qq libcjson-dev python3
+run: sudo apt-get update -qq && sudo apt-get install -y -qq libcjson-dev python3 libeigen3-dev
  • .gitignore: Add build_*/ pattern alongside existing build-*/ to cover underscore-named local build directories

Copilot AI changed the title [WIP] Fix failing GitHub Actions job evaluator ci: add libeigen3-dev to evaluator job to fix planning_node build Jul 11, 2026
Copilot AI requested a review from caixuf July 11, 2026 02:29
@caixuf caixuf marked this pull request as ready for review July 11, 2026 03:11
@caixuf caixuf merged commit 0cae87d into main Jul 11, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants