mirror of
https://github.com/huggingface/lerobot.git
synced 2026-08-08 17:39:44 +00:00
741005d719
Every docstring change for the API reference, on top of the infrastructure PR which contains none. Two halves: a repo-wide pass over what the renderer cannot handle, and `src/lerobot/robots/` taken to 100% as the worked example. **Renderer fixes, repo-wide.** Both of these render incorrectly the moment `[[autodoc]]` is on, and both were verified against a local build: - 24 Sphinx roles across three files. They are unsupported and render as literal `:pymeth:` text. Method references become doc-builder cross-references; the ones pointing at instance attributes become inline code, since attributes get no autodoc anchor and a cross-reference would be a dead link. - 43 `Attributes:` sections across 27 files. doc-builder parses a bare `Attributes:` as a synonym for `Parameters:` — `Robot`'s attributes rendered inside `<paramsdesc>`, presenting `config_class` and `name` to readers as constructor arguments when the actual parameter is `config`. Where the original carried no type, the type comes from the real class annotation rather than being invented. The four base classes every other module inherits from — `robot.py`, `teleoperator.py`, `motors_bus.py`, `camera.py` — are rewritten to the standard, since subclasses document only their deviations from that text. Three docstring errors corrected in passing: `Teleoperator.get_action` pointed at `observation_features`, which `Teleoperator` does not have; `send_feedback` documented a `Returns:` for a method returning `None`; and `config_class` was typed `RobotConfig` instead of `type[TeleoperatorConfig]`. **`robots/`, 109/306 -> 306/306.** The configuration dataclasses were the substantial part. Their fields were documented only with `#` comments above each field, which doc-builder cannot see: before this, `SO101FollowerConfig` rendered all eleven of its fields with not one description. Each config now carries an `Args:` block on the concrete registered class, covering inherited fields too, because doc-builder renders only a class's own docstring and several of these configs are thin multiple-inheritance shims whose body is `pass`. The inline comments are kept rather than removed, so fields stay annotated in the source as well as on the rendered page. Note this leaves each field described twice, and only the `Args:` block is checked against the signature by `make check-docstrings`, so the two can drift. Writing them turned up things worth stating plainly on the page rather than leaving in a comment: which configs have no serial port at all because they talk over a network or the cloud (Reachy 2, Unitree G1, LeKiwi's client, EarthRover), which manage their own calibration so `calibration_dir` does nothing, that OpenArm's default joint limits are deliberately tiny until `side` is set, and that reBot's `port` means a different thing depending on `can_adapter`. Two pre-existing docstring bugs that the doctest infrastructure surfaced are fixed here: `SerialMotorsBus` used `>>>` inside a ```bash block to show CLI output, which doctest read as Python and failed on with a SyntaxError, and `MotorsBus.torque_disabled`'s example referenced an undefined name. `ensure_safe_goal_position` gains a genuinely executing example so the doctest gate is not vacuous. **Gates ratcheted**, each of which the infrastructure PR left deliberately loose: - `check_docstrings.py`'s ignore list emptied — the ten objects it held all had bare `Attributes:` sections, now converted. - `check_config_docstrings.py`'s ignore list emptied — every registered robot config documents its port and calibration semantics. - `robots/` removed from the ruff `D` per-file-ignores, as are the two package-root files, whose one-line docstring issues are fixed here. D100 and D104 are ignored globally instead: they ask for a banner on every file and every `__init__.py`, which appears on no rendered page. - `interrogate` raised 52 -> 55 against a measured 55.3%. - The four `robots/` files carrying examples added to the doctest allowlist, which shipped empty. Verified: all 66 changed files under `src/lerobot/` are provably docstring-only (AST with docstrings stripped is byte-identical to main), no comment line is removed anywhere in `robots/`, and 708 tests pass. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
313 lines
11 KiB
Python
313 lines
11 KiB
Python
#!/usr/bin/env python
|
|
|
|
# Copyright 2024 The HuggingFace Inc. team. All rights reserved.
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
from __future__ import annotations
|
|
|
|
import time
|
|
from typing import TYPE_CHECKING, Any
|
|
|
|
from lerobot.cameras import make_cameras_from_configs
|
|
from lerobot.lerobot_types import RobotAction, RobotObservation
|
|
from lerobot.utils.import_utils import _reachy2_sdk_available, require_package
|
|
|
|
from ..robot import Robot
|
|
from ..utils import ensure_safe_goal_position
|
|
from .configuration_reachy2 import Reachy2RobotConfig
|
|
|
|
if TYPE_CHECKING or _reachy2_sdk_available:
|
|
from reachy2_sdk import ReachySDK
|
|
else:
|
|
ReachySDK = None
|
|
|
|
# {lerobot_keys: reachy2_sdk_keys}
|
|
REACHY2_NECK_JOINTS = {
|
|
"neck_yaw.pos": "head.neck.yaw",
|
|
"neck_pitch.pos": "head.neck.pitch",
|
|
"neck_roll.pos": "head.neck.roll",
|
|
}
|
|
|
|
REACHY2_ANTENNAS_JOINTS = {
|
|
"l_antenna.pos": "head.l_antenna",
|
|
"r_antenna.pos": "head.r_antenna",
|
|
}
|
|
|
|
REACHY2_R_ARM_JOINTS = {
|
|
"r_shoulder_pitch.pos": "r_arm.shoulder.pitch",
|
|
"r_shoulder_roll.pos": "r_arm.shoulder.roll",
|
|
"r_elbow_yaw.pos": "r_arm.elbow.yaw",
|
|
"r_elbow_pitch.pos": "r_arm.elbow.pitch",
|
|
"r_wrist_roll.pos": "r_arm.wrist.roll",
|
|
"r_wrist_pitch.pos": "r_arm.wrist.pitch",
|
|
"r_wrist_yaw.pos": "r_arm.wrist.yaw",
|
|
"r_gripper.pos": "r_arm.gripper",
|
|
}
|
|
|
|
REACHY2_L_ARM_JOINTS = {
|
|
"l_shoulder_pitch.pos": "l_arm.shoulder.pitch",
|
|
"l_shoulder_roll.pos": "l_arm.shoulder.roll",
|
|
"l_elbow_yaw.pos": "l_arm.elbow.yaw",
|
|
"l_elbow_pitch.pos": "l_arm.elbow.pitch",
|
|
"l_wrist_roll.pos": "l_arm.wrist.roll",
|
|
"l_wrist_pitch.pos": "l_arm.wrist.pitch",
|
|
"l_wrist_yaw.pos": "l_arm.wrist.yaw",
|
|
"l_gripper.pos": "l_arm.gripper",
|
|
}
|
|
|
|
REACHY2_VEL = {
|
|
"mobile_base.vx": "vx",
|
|
"mobile_base.vy": "vy",
|
|
"mobile_base.vtheta": "vtheta",
|
|
}
|
|
|
|
|
|
class Reachy2Robot(Robot):
|
|
"""[Reachy 2](https://www.pollen-robotics.com/reachy/), the humanoid by Pollen Robotics."""
|
|
|
|
config_class = Reachy2RobotConfig
|
|
name = "reachy2"
|
|
|
|
def __init__(self, config: Reachy2RobotConfig):
|
|
"""Build the robot from its configuration.
|
|
|
|
Args:
|
|
config (`Reachy2RobotConfig`):
|
|
The robot's configuration. Its `port` and `cameras` determine what is connected.
|
|
"""
|
|
require_package("reachy2_sdk", extra="reachy2")
|
|
super().__init__(config)
|
|
|
|
self.config = config
|
|
self.robot_type = self.config.type
|
|
self.use_external_commands = self.config.use_external_commands
|
|
|
|
self.reachy: None | ReachySDK = None
|
|
self.cameras = make_cameras_from_configs(config.cameras)
|
|
|
|
self.logs: dict[str, float] = {}
|
|
|
|
self.joints_dict: dict[str, str] = self._generate_joints_dict()
|
|
|
|
@property
|
|
def observation_features(self) -> dict[str, Any]:
|
|
"""The values this robot reports, and their types or shapes.
|
|
|
|
Returns:
|
|
`dict`: Keys as returned by [`~robots.Robot.get_observation`], mapped to a scalar type for
|
|
proprioceptive values or to a `(height, width, channels)` shape for images.
|
|
"""
|
|
return {**self.motors_features, **self.camera_features}
|
|
|
|
@property
|
|
def action_features(self) -> dict[str, type]:
|
|
"""The values this robot accepts, and their types.
|
|
|
|
Returns:
|
|
`dict`: Keys accepted by [`~robots.Robot.send_action`], mapped to their type.
|
|
"""
|
|
return self.motors_features
|
|
|
|
@property
|
|
def camera_features(self) -> dict[str, tuple[int | None, int | None, int]]:
|
|
"""The shape of each configured camera's frames.
|
|
|
|
Returns:
|
|
`dict[str, tuple[int | None, int | None, int]]`: Camera name mapped to
|
|
`(height, width, channels)`.
|
|
"""
|
|
return {cam: (self.cameras[cam].height, self.cameras[cam].width, 3) for cam in self.cameras}
|
|
|
|
@property
|
|
def motors_features(self) -> dict[str, type]:
|
|
"""The joints this robot exposes, given which parts are enabled in the config.
|
|
|
|
Returns:
|
|
`dict[str, type]`: Joint name mapped to `float`, including the mobile base's velocity
|
|
components when `with_mobile_base` is set.
|
|
"""
|
|
if self.config.with_mobile_base:
|
|
return {
|
|
**dict.fromkeys(
|
|
self.joints_dict.keys(),
|
|
float,
|
|
),
|
|
**dict.fromkeys(
|
|
REACHY2_VEL.keys(),
|
|
float,
|
|
),
|
|
}
|
|
else:
|
|
return dict.fromkeys(self.joints_dict.keys(), float)
|
|
|
|
@property
|
|
def is_connected(self) -> bool:
|
|
"""Whether every device this robot uses is connected.
|
|
|
|
Returns:
|
|
`bool`: `True` only when the robot and all its cameras are connected.
|
|
"""
|
|
return self.reachy.is_connected() if self.reachy is not None else False
|
|
|
|
def connect(self, calibrate: bool = False) -> None:
|
|
"""Connect to the robot and its cameras, then apply the configured settings.
|
|
|
|
Args:
|
|
calibrate (`bool`, *optional*, defaults to `False`):
|
|
Accepted for interface compatibility and ignored: Reachy 2 manages its own calibration.
|
|
|
|
Raises:
|
|
DeviceAlreadyConnectedError: If the robot is already connected.
|
|
"""
|
|
self.reachy = ReachySDK(self.config.ip_address)
|
|
if not self.is_connected:
|
|
raise ConnectionError()
|
|
|
|
for cam in self.cameras.values():
|
|
cam.connect()
|
|
|
|
self.configure()
|
|
|
|
def configure(self) -> None:
|
|
"""Apply the operating mode, gains and limits from the configuration to the robot."""
|
|
if self.reachy is not None:
|
|
self.reachy.turn_on()
|
|
self.reachy.reset_default_limits()
|
|
|
|
@property
|
|
def is_calibrated(self) -> bool:
|
|
"""Whether the robot is calibrated.
|
|
|
|
Returns:
|
|
`bool`: `True` when no calibration is needed before use.
|
|
"""
|
|
return True
|
|
|
|
def calibrate(self) -> None:
|
|
"""Calibrate the robot and store the result.
|
|
|
|
Interactive: prompts on stdin and asks you to move the robot through the required positions.
|
|
"""
|
|
pass
|
|
|
|
def _generate_joints_dict(self) -> dict[str, str]:
|
|
joints = {}
|
|
if self.config.with_neck:
|
|
joints.update(REACHY2_NECK_JOINTS)
|
|
if self.config.with_l_arm:
|
|
joints.update(REACHY2_L_ARM_JOINTS)
|
|
if self.config.with_r_arm:
|
|
joints.update(REACHY2_R_ARM_JOINTS)
|
|
if self.config.with_antennas:
|
|
joints.update(REACHY2_ANTENNAS_JOINTS)
|
|
return joints
|
|
|
|
def _get_state(self) -> dict[str, float]:
|
|
if self.reachy is not None:
|
|
pos_dict = {k: self.reachy.joints[v].present_position for k, v in self.joints_dict.items()}
|
|
if not self.config.with_mobile_base:
|
|
return pos_dict
|
|
vel_dict = {k: self.reachy.mobile_base.odometry[v] for k, v in REACHY2_VEL.items()}
|
|
return {**pos_dict, **vel_dict}
|
|
else:
|
|
return {}
|
|
|
|
def get_observation(self) -> RobotObservation:
|
|
"""Read the robot's current state and a frame from each camera.
|
|
|
|
Returns:
|
|
`dict[str, Any]`: Keys matching [`~robots.Robot.observation_features`].
|
|
|
|
Raises:
|
|
DeviceNotConnectedError: If the robot is not connected.
|
|
"""
|
|
obs_dict: RobotObservation = {}
|
|
|
|
# Read Reachy 2 state
|
|
before_read_t = time.perf_counter()
|
|
obs_dict.update(self._get_state())
|
|
self.logs["read_pos_dt_s"] = time.perf_counter() - before_read_t
|
|
|
|
# Capture images from cameras
|
|
for cam_key, cam in self.cameras.items():
|
|
obs_dict[cam_key] = cam.read_latest()
|
|
|
|
return obs_dict
|
|
|
|
def send_action(self, action: RobotAction) -> RobotAction:
|
|
"""Command the robot to move towards a target configuration.
|
|
|
|
Args:
|
|
action (`dict[str, Any]`):
|
|
Target values, keyed as in [`~robots.Robot.action_features`].
|
|
|
|
Returns:
|
|
`dict[str, Any]`: The action actually sent, which may be clipped by `max_relative_target`.
|
|
|
|
Raises:
|
|
DeviceNotConnectedError: If the robot is not connected.
|
|
"""
|
|
if self.reachy is not None:
|
|
if not self.is_connected:
|
|
raise ConnectionError()
|
|
|
|
before_write_t = time.perf_counter()
|
|
|
|
vel = {}
|
|
goal_pos = {}
|
|
for key, val in action.items():
|
|
if key not in self.joints_dict:
|
|
if key not in REACHY2_VEL:
|
|
raise KeyError(f"Key '{key}' is not a valid motor key in Reachy 2.")
|
|
else:
|
|
vel[REACHY2_VEL[key]] = float(val)
|
|
else:
|
|
if not self.use_external_commands and self.config.max_relative_target is not None:
|
|
goal_pos[key] = float(val)
|
|
goal_present_pos = {
|
|
key: (
|
|
goal_pos[key],
|
|
self.reachy.joints[self.joints_dict[key]].present_position,
|
|
)
|
|
}
|
|
safe_goal_pos = ensure_safe_goal_position(
|
|
goal_present_pos, float(self.config.max_relative_target)
|
|
)
|
|
val = safe_goal_pos[key]
|
|
self.reachy.joints[self.joints_dict[key]].goal_position = float(val)
|
|
|
|
if self.config.with_mobile_base:
|
|
self.reachy.mobile_base.set_goal_speed(vel["vx"], vel["vy"], vel["vtheta"])
|
|
|
|
# We don't send the goal positions if we control Reachy 2 externally
|
|
if not self.use_external_commands:
|
|
self.reachy.send_goal_positions()
|
|
if self.config.with_mobile_base:
|
|
self.reachy.mobile_base.send_speed_command()
|
|
|
|
self.logs["write_pos_dt_s"] = time.perf_counter() - before_write_t
|
|
return action
|
|
|
|
def disconnect(self) -> None:
|
|
"""Disconnect from the robot and its cameras.
|
|
|
|
Raises:
|
|
DeviceNotConnectedError: If the robot is not connected.
|
|
"""
|
|
if self.reachy is not None:
|
|
for cam in self.cameras.values():
|
|
cam.disconnect()
|
|
if self.config.disable_torque_on_disconnect:
|
|
self.reachy.turn_off_smoothly()
|
|
self.reachy.disconnect()
|