Skip to content

Add reachy_mini mock-deps registry entry#807

Merged
mishig25 merged 1 commit into
huggingface:mainfrom
pollen-robotics:add-reachy-mini-mock-deps
Jul 13, 2026
Merged

Add reachy_mini mock-deps registry entry#807
mishig25 merged 1 commit into
huggingface:mainfrom
pollen-robotics:add-reachy-mini-mock-deps

Conversation

@FabienDanieau

Copy link
Copy Markdown
Contributor

What

Adds src/doc_builder/mock_deps/reachy_mini.txt so the pollen-robotics/reachy_mini docs build on a bare runner via light-install + mock_deps, instead of the custom Docker image the project relied on before custom_container was removed from the build workflows.

Why

reachy_mini pulls in PyGObject (needs libgirepository/cairo system headers at install time), dbus-python, plus hardware/robot and optional-extra packages (gstreamer bindings, mujoco, rerun, placo, opencv…). On the current ubuntu-22.04 build runner none of these are available, and pre_command runs after the package install — too late for the install-time system libs. The registry entry sidesteps this: only the light, wheel-only deps are installed for real; everything native/heavy is mocked at introspection time.

Validation

Reproduced the CI light-install path locally against reachy_mini v1.9.0:

  • doc-builder light-install reachy_mini --check → recognized ✓
  • doc-builder build reachy_mini docs/source --html --cleanexit 0, every API/*.html page generated.
  • Autodoc content is real, not hollowed by the mocks: e.g. media classes that depend on the mocked gi/dbus (GStreamerAudio.get_DoA, …) and the top-level ReachyMini.__init__ render their real signatures, defaults, and docstrings.

Notes

Bare lines are import names (mocked); real: lines are the light deps installed for real. Follows the format of the existing lerobot.txt / transformers.txt entries.

Lets doc-builder build the pollen-robotics/reachy_mini docs on a bare
runner via light-install + mock_deps, replacing the custom Docker image
the project used before custom_container was dropped.

Mocked deps need system libraries (gi/dbus), are hardware/robot or
optional-extra packages, or are heavy wheels not needed to introspect
docstrings. Validated with a full `doc-builder build --html` (exit 0,
all API pages generated with real signatures/docstrings).

Assisted-by: Claude:claude-opus-4-8
@mishig25 mishig25 merged commit 6108e85 into huggingface:main Jul 13, 2026
4 checks passed
FabienDanieau added a commit to pollen-robotics/reachy_mini that referenced this pull request Jul 13, 2026
Same fix as the main doc build: build_pr_documentation.yml@main also
dropped the custom_container input, so PR doc builds would likewise
startup_failure. Deps come from doc-builder's mock-deps registry
(huggingface/doc-builder#807).

Assisted-by: Claude:claude-opus-4-8
FabienDanieau added a commit to pollen-robotics/reachy_mini that referenced this pull request Jul 13, 2026
Same fix as the main doc build: build_pr_documentation.yml@main also
dropped the custom_container input, so PR doc builds would likewise
startup_failure. Deps come from doc-builder's mock-deps registry
(huggingface/doc-builder#807).

Assisted-by: Claude:claude-opus-4-8
FabienDanieau added a commit to pollen-robotics/reachy_mini that referenced this pull request Jul 13, 2026
* docs(ci): drop removed custom_container input from doc build

huggingface/doc-builder dropped the custom_container input from its
reusable build workflows, so passing it now fails the workflow at
startup (startup_failure, 0 jobs). Remove it; the build runs on the
bare runner and gets its deps via doc-builder's mock-deps registry
(see huggingface/doc-builder#807).

Assisted-by: Claude:claude-opus-4-8

* docs(ci): drop custom_container from PR doc build too

Same fix as the main doc build: build_pr_documentation.yml@main also
dropped the custom_container input, so PR doc builds would likewise
startup_failure. Deps come from doc-builder's mock-deps registry
(huggingface/doc-builder#807).

Assisted-by: Claude:claude-opus-4-8
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