mirror of
https://github.com/huggingface/lerobot.git
synced 2026-08-08 17:39:44 +00:00
2e8345a5cc
LeRobot's documentation build passes `--not_python_module`, which tells doc-builder there is no importable Python package and disables `[[autodoc]]` entirely. The result is that all 90+ pages are hand-written guides and there is no generated API reference at all. This is the machinery to change that. It deliberately contains no docstring changes of its own — every docstring edit lives in the follow-up PR, so this one can be reviewed as tooling and configuration alone. **The standard.** `docs/source/writing_docstrings.mdx` is the contract: Google section headers with Hugging Face type formatting, the machine-checked argument line, `**Attributes**:`, doc-builder cross-references, fenced doctest examples. It also records three behaviours that are not discoverable from the source and were verified against a local build: `[[autodoc]]` silently skips members with no docstring; doc-builder does not inherit docstrings from base classes, so a registered config shim whose body is `pass` renders every field with no description; and module-level aliases resolve to the canonical class. **Autodoc turned on**, with two changes that are not obvious: - `--version main` on the main-docs job. Without `--not_python_module`, doc-builder resolves the version from `lerobot.__version__` and only maps it to the default branch when it contains "dev". transformers relies on that; our main carries 0.6.2. Verified by building both ways — dropping the flag alone would publish the main docs to /lerobot/v0.6.2/ instead of /lerobot/main/ and disable notebook building. - `pre_command` on both jobs. doc-builder ships a mock-deps registry entry for lerobot, so the reusable workflow takes its light-install path, which cannot import the package. The heavy dependencies cannot be mocked either: draccus runs `register_subclass` at import time and `processor/converters.py` calls `functools.singledispatch.register(torch.Tensor)`, which needs a real class. `[dataset]` is the only extra required. Workflow triggers gain `src/**`, since the reference is now generated from docstrings. `docs/source/api/` is excluded from the prettier hook, which reads `[[autodoc]]` member lists as lazy paragraph continuations and joins a ten-entry list onto one line. Nine API reference pages, scaffolded with each module's base class. **Doctests.** `LeRobotDocTestParser` is mandatory rather than optional here: ruff's `docstring-code-format = true` drops the blank line before a closing fence, after which stdlib's `_EXAMPLE_RE` reads the fence as expected output and every example with output fails. It is written against the installed pytest rather than copied from transformers, whose version predates pytest 9's `import_path` signature and its own fix for the `@property` line-number bug. `preprocess_string` also diverges: the upstream fenced-block split puts a single-line example's code in a chunk with no `>>>` in it, so neither the CUDA skip nor the `+IGNORE_RESULT` injection fires for it. **Checkers.** `utils/check_docstrings.py` is the ~300-line core of the 2203-line transformers original; the `@auto_docstring` system, modular propagation, GitPython and `checkers.py` are not ported. `utils/check_config_docstrings.py` checks that every registered robot config documents its port and calibration semantics. **Gates**, all set to values that pass today: ruff `D` with per-file-ignores per unconverted module, `interrogate` at `fail-under = 52` against a measured 52.1%, and Makefile targets wired into the quality workflow. The doctest allowlist ships empty and the `doctest` target handles that, because the files carrying runnable examples arrive with the docstring PR. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
103 lines
4.3 KiB
YAML
103 lines
4.3 KiB
YAML
# Copyright 2025 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.
|
|
|
|
# This workflow handles building documentation for both main branches and PRs.
|
|
name: Documentation
|
|
|
|
on:
|
|
# Allows running this workflow manually from the Actions tab
|
|
workflow_dispatch:
|
|
inputs:
|
|
version:
|
|
description: 'Version tag (e.g. v0.1.2) - Leave empty for standard main build'
|
|
required: false
|
|
type: string
|
|
|
|
# Triggers on pushes to main that touch the docs or the sources the API reference is generated from.
|
|
# `src/**` is included because the API reference is built from docstrings via `[[autodoc]]`: without it,
|
|
# published API pages would go stale as soon as a docstring changed.
|
|
push:
|
|
branches:
|
|
- main
|
|
paths:
|
|
- "docs/**"
|
|
- "src/**"
|
|
|
|
# Same for pull requests, so a docstring change gets a preview build and a broken `[[autodoc]]` path
|
|
# fails the PR rather than main.
|
|
pull_request:
|
|
branches:
|
|
- main
|
|
paths:
|
|
- "docs/**"
|
|
- "src/**"
|
|
|
|
release:
|
|
types: [published]
|
|
|
|
# Ensures that only the latest commit for a PR or branch is built, canceling older runs.
|
|
concurrency:
|
|
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
|
|
cancel-in-progress: true
|
|
|
|
jobs:
|
|
# This job builds and deploys the official documentation.
|
|
build_main_docs:
|
|
name: Build Main Docs
|
|
if: >
|
|
(github.event_name == 'push' || github.event_name == 'workflow_dispatch' || github.event_name == 'release') &&
|
|
github.repository == 'huggingface/lerobot'
|
|
permissions:
|
|
contents: read
|
|
uses: huggingface/doc-builder/.github/workflows/build_main_documentation.yml@6108e850ae1cf2f71bb0815a600bcd50c39abfa7 # main
|
|
with:
|
|
commit_sha: ${{ github.sha }}
|
|
package: lerobot
|
|
# doc-builder ships a mock-deps registry entry for lerobot, so the reusable workflow takes its
|
|
# "light install" path: `pip install ./lerobot --no-deps` plus a handful of real dependencies.
|
|
# That is not enough to import lerobot — draccus runs `register_subclass` at import time and
|
|
# `processor/converters.py` calls `functools.singledispatch.register(torch.Tensor)`, neither of
|
|
# which works against a mock. Install the package for real before the build.
|
|
pre_command: uv pip install "./lerobot[dataset]"
|
|
# `--version main` is load-bearing: without `--not_python_module`, doc-builder falls back to
|
|
# `lerobot.__version__` and only maps that to the default branch when it contains "dev". Our main
|
|
# branch carries a release version (0.6.2), so omitting this would publish the main docs to
|
|
# /lerobot/v0.6.2/ instead of /lerobot/main/ and disable notebook building.
|
|
additional_args: >-
|
|
${{
|
|
(github.event_name == 'release' && format('--version {0}', github.event.release.tag_name)) ||
|
|
(inputs.version != '' && format('--version {0}', inputs.version)) ||
|
|
'--version main'
|
|
}}
|
|
secrets:
|
|
token: ${{ secrets.HUGGINGFACE_PUSH }}
|
|
hf_token: ${{ secrets.HF_DOC_BUILD_PUSH }}
|
|
|
|
# This job builds a preview of the documentation for a pull request.
|
|
# The result of this job triggers the 'Upload PR Documentation' workflow.
|
|
build_pr_docs:
|
|
name: Build PR Docs
|
|
if: github.event_name == 'pull_request' && github.repository == 'huggingface/lerobot'
|
|
permissions:
|
|
contents: read
|
|
pull-requests: write
|
|
uses: huggingface/doc-builder/.github/workflows/build_pr_documentation.yml@6108e850ae1cf2f71bb0815a600bcd50c39abfa7 # main
|
|
with:
|
|
commit_sha: ${{ github.event.pull_request.head.sha }}
|
|
pr_number: ${{ github.event.number }}
|
|
package: lerobot
|
|
# See the comment on build_main_docs. The PR workflow passes its own `--version pr_<n>`, so no
|
|
# additional_args are needed here.
|
|
pre_command: uv pip install "./lerobot[dataset]"
|