Skip to content

First addition of multi-robot plans - #350

Open
pkehr wants to merge 16 commits into
cram2:devfrom
pkehr:multirobot
Open

First addition of multi-robot plans#350
pkehr wants to merge 16 commits into
cram2:devfrom
pkehr:multirobot

Conversation

@pkehr

@pkehr pkehr commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

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

pkehr commented Aug 8, 2025

Copy link
Copy Markdown
Contributor Author

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

@LucaKro LucaKro left a comment

Copy link
Copy Markdown
Collaborator

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

Comment thread src/pycram/multirobot/robot_manager.py Outdated
Comment thread src/pycram/multirobot/robot_manager.py Outdated
Comment thread src/pycram/multirobot/robot_manager.py Outdated
Comment thread src/pycram/multirobot/robot_manager.py
Comment thread src/pycram/multirobot/robot_manager.py Outdated
Comment thread src/pycram/multirobot/robot_manager.py
Comment thread src/pycram/multirobot/robot_manager.py Outdated
Comment thread src/pycram/process_module.py Outdated
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