Added rosbag2 dataset recording functionality

[ola31]

Changes

• Added rosbag_recorder package
• Added rosbag2 recording support when collecting LeRobot datasets
• Update ros-ci.yml and ros-lint.yml for rosbag_recorder package

Related PR

• Added a guide for rosbag2 recording ai_worker_website#52

[Copilot]

Pull Request Overview

This PR adds rosbag2 recording capabilities to the physical_ai_tools package. When collecting LeRobot datasets, users can now optionally record rosbag2 files alongside the dataset collection process.

Key changes:

• New rosbag_recorder package implementing a ROS 2 service for controlling rosbag2 recording
• Integration with physical_ai_server to manage rosbag recording lifecycle during dataset collection
• UI support in physical_ai_manager for enabling/disabling rosbag2 recording

Reviewed Changes

Copilot reviewed 33 out of 34 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
rosbag_recorder/srv/SendCommand.srv	Service definition for controlling rosbag recording with PREPARE, START, STOP, STOP_AND_DELETE, and FINISH commands
rosbag_recorder/src/service_bag_recorder.cpp	C++ implementation of the rosbag recorder service node
rosbag_recorder/include/rosbag_recorder/service_bag_recorder.hpp	Header file defining the ServiceBagRecorder class
rosbag_recorder/CMakeLists.txt	Build configuration for the rosbag_recorder package
rosbag_recorder/package.xml	Package manifest with dependencies
physical_ai_server/physical_ai_server.py	Integration logic to handle rosbag recording state transitions
physical_ai_server/communication/communicator.py	Client implementation for rosbag recording service calls
physical_ai_server/utils/parameter_utils.py	New utility function to parse topic lists
physical_ai_server/data_processing/data_manager.py	Support for rosbag path management and README card generation
physical_ai_manager UI files	Frontend support for rosbag2 recording toggle

Files not reviewed (1)

• physical_ai_manager/package-lock.json: Language not supported

Comments suppressed due to low confidence (2)

physical_ai_server/physical_ai_server/physical_ai_server.py:1

• [nitpick] Inconsistent naming: Python code uses record_rosbag2 (snake_case) while JavaScript uses recordRosBag2 (camelCase). While this follows language conventions, the inconsistency between 'Rosbag2' and 'RosBag2' capitalization should be consistent. Consider using either 'rosbag2' or 'rosBag2' uniformly across the codebase.

#!/usr/bin/env python3

physical_ai_server/physical_ai_server/data_processing/data_manager.py:544

• Normal methods should have 'self', rather than 'info_json_path', as their first parameter.

    def get_robot_type_from_info_json(info_json_path):

---

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

[DongyunRobotis]

LGTM

[Seongoo]

Great work!