Skip to content

fix(bitbots_ros_control): dynamixel driver include#756

Open
texhnolyze wants to merge 1 commit intomainfrom
fix/use-local-dynamixel-driver
Open

fix(bitbots_ros_control): dynamixel driver include#756
texhnolyze wants to merge 1 commit intomainfrom
fix/use-local-dynamixel-driver

Conversation

@texhnolyze
Copy link
Contributor

to point to <dynamixel_workbench_toolbox/dynamixel_driver.h> the
correct include path for the pixi environment and not a local/global
installation coming with the dynamixel workbench GUI.

Checklist

  • Run pixi run build
  • Write documentation
  • Test on your machine
  • Test on the robot
  • Create issues for future work
  • Triage this PR and label it

@texhnolyze texhnolyze self-assigned this Mar 8, 2026
@github-project-automation github-project-automation bot moved this to 🆕 New in Software Mar 8, 2026
@texhnolyze texhnolyze moved this from 🆕 New to 🔖 Ready in Software Mar 8, 2026
@texhnolyze texhnolyze added bug Something isn't working lowlevel labels Mar 8, 2026
@texhnolyze texhnolyze requested review from Flova, ayin21 and jaagut March 8, 2026 21:17
to point to `<dynamixel_workbench_toolbox/dynamixel_driver.h>` the
correct include path for the `pixi` environment and not a local/global
installation coming with the dynamixel workbench GUI.
@texhnolyze texhnolyze force-pushed the fix/use-local-dynamixel-driver branch from 30e9c1c to f924c5d Compare March 8, 2026 21:18
@github-project-automation github-project-automation bot moved this from 🔖 Ready to 👀 In review in Software Mar 8, 2026
@Flova
Copy link
Member

Flova commented Mar 8, 2026

Why did this work without it? In e.g. the ci only the pixie headers are present.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working lowlevel

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

2 participants