Skip to content

Add explicit angles dependency - #22

Open
Plumezz wants to merge 1 commit into
blackcoffeerobotics:masterfrom
Plumezz:add-angles-dependency
Open

Add explicit angles dependency#22
Plumezz wants to merge 1 commit into
blackcoffeerobotics:masterfrom
Plumezz:add-angles-dependency

Conversation

@Plumezz

@Plumezz Plumezz commented Jul 25, 2026

Copy link
Copy Markdown

This PR adds angles as an explicit dependency of vector_pursuit_controller.

src/vector_pursuit_controller.cpp directly includes angles/angles.h and uses APIs including angles::normalize_angle and angles::normalize_angle_positive in the controller implementation.

Currently, angles appears to be available transitively through Navigation2 dependencies such as nav2_controller or nav2_costmap_2d. This change makes the package manifest and CMake configuration match the direct source usage.

Changes:

  • Add <depend>angles</depend> to package.xml.
  • Add find_package(angles REQUIRED) to CMakeLists.txt.
  • Add angles to the target/export dependency list.

Signed-off-by: Guanxiao Qi <944843236@qq.com>
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.

1 participant