Skip to content

Conversation

@pkehr
Copy link
Contributor

@pkehr pkehr commented Aug 7, 2025

Summary

This PR introduces the RobotManager class, which enables managing and controlling multiple robots within a world environment.

Changes

  • Replaces direct usage of World.robot and RobotDescription.current_robot_description with:
    • RobotManager.get_active_robot()
    • RobotManager.get_robot_description()
  • Enables specifying which robot to use inside a with_robot statement by passing the respective robot object.
    Example: with_real_robot(pr2_robot)
  • Adds a test case that swaps and verifies the active robot.

@pkehr pkehr closed this Aug 7, 2025
@pkehr pkehr reopened this Aug 8, 2025
@pkehr
Copy link
Contributor Author

pkehr commented Aug 8, 2025

Opened the pr again for review (excluding CI for now)

This uses justin as secondary robot instead of tiago, as justin is ros2 compatible and resources are available in iai_robot
Copy link
Collaborator

@LucaKro LucaKro left a comment

Choose a reason for hiding this comment

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

some small clarifications and method signature typing, but otherwise good work

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