Skip to content

content: Add interfaces for individual steering of motors to Raph ROS 2 API#130

Merged
bjsowa merged 4 commits intodevelopmentfrom
content/individual-motor-interfaces
Mar 13, 2026
Merged

content: Add interfaces for individual steering of motors to Raph ROS 2 API#130
bjsowa merged 4 commits intodevelopmentfrom
content/individual-motor-interfaces

Conversation

@jasiex01
Copy link
Contributor

@jasiex01 jasiex01 commented Mar 3, 2026

No description provided.

@jasiex01 jasiex01 requested a review from Krzemien97 as a code owner March 3, 2026 11:00
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Mar 3, 2026

Deploying fictionlab-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 4e8d9df
Status: ✅  Deploy successful!
Preview URL: https://47b0ab5f.fictionlab-docs.pages.dev
Branch Preview URL: https://content-individual-motor-int.fictionlab-docs.pages.dev

View logs

@jasiex01 jasiex01 requested review from Copilot and removed request for Krzemien97 March 3, 2026 11:02
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Raph Rover ROS 2 API documentation to include newly supported low-level motor/servo steering interfaces, aligning the docs with the expanded control surface of the controller.

Changes:

  • Document new subscribed topics for direct left/right steering servo commands (cmd_servo_l, cmd_servo_r).
  • Document new subscribed topics for per-wheel motor commands (cmd_wheel_fl/fr/rl/rr).
  • Update drivetrain parameter docs (adjust cmd_vel_steering_angle_velocity default; remove servo_calibration_power_threshold) and add link refs for the new message types.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jasiex01 jasiex01 requested a review from bjsowa March 3, 2026 11:08

Sets a new user state for all the LEDs in the specified LED panel.

- `controller/cmd_servo_l`, `controller/cmd_servo_r`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we move this as the last entry? Also the cmd_wheel and cmd_servo descriptions should be almost identical, please unify them

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@jasiex01 jasiex01 requested a review from bjsowa March 13, 2026 12:02
@bjsowa bjsowa merged commit d021a7b into development Mar 13, 2026
4 checks passed
@jasiex01 jasiex01 deleted the content/individual-motor-interfaces branch March 13, 2026 13:17
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.

3 participants