docs(install): fix required system libs, replay size, and library-ins…#2320
Draft
KrishnaH96 wants to merge 4 commits into
Draft
docs(install): fix required system libs, replay size, and library-ins…#2320KrishnaH96 wants to merge 4 commits into
KrishnaH96 wants to merge 4 commits into
Conversation
…tall noise (#2119) - ubuntu.md: add libgl1/libegl1 — OpenGL runtime for the always-installed open3d + rerun-sdk core deps (matches scripts/install.sh); without them the visualizer fails at runtime with "libGL.so.1: cannot open shared object file" on minimal/headless/Docker installs. Drop pre-commit (a dev dependency-group tool, irrelevant to library users). - osx.md: drop pre-commit. - ubuntu.md/osx.md: note the inline `export PATH` is session-only (uv persists via the shell profile; open a new terminal). - go2/index.md: correct replay first-run download to ~85 MB LFS / ~220 MB SQLite DB (was "~2.4 GB of LiDAR/video"), resolving the README vs go2 inconsistency. Validated by hand on Ubuntu 24.04 per #2119.
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
…#2119) - ubuntu.md: mention optional graphviz for blueprint-graph visualization (dimos logs "graphviz not found, skipping blueprint graph" without it; everything else still works). - go2/index.md: note the first MuJoCo sim run also clones mujoco_menagerie and pulls a ~60 MB mujoco_sim asset bundle from LFS (one-time).
…issing (#2119) Agentic blueprints use OpenAI for text-to-speech (SpeakSkill) and, by default, the LLM agent. Without OPENAI_API_KEY the run previously crashed deep in start_all_modules with an opaque ExceptionGroup (a SpeakSkill.start failure), after already deploying modules and downloading assets. Add an openai_api_key_set blueprint requirement check (mirroring ollama_installed) on _common_agentic, so a missing key now fails at blueprint build with a clear, actionable message + key link — before any modules deploy. This also makes the OpenAI dependency of the -ollama variant explicit (it uses OpenAI TTS even with a local LLM).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of #2119 (validate README / install examples). Small, hand-validated install-doc fixes (tested on Ubuntu 24.04).
Changes
ubuntu.md— addlibgl1 libegl1(OpenGL runtime for the always-installedopen3d+rerun-sdkcore deps; matchesscripts/install.sh). Without them the visualizer fails at runtime withlibGL.so.1: cannot open shared object fileon minimal/headless/Docker installs. Also droppre-commit(a dev[dependency-groups]tool, not needed by library users).osx.md— droppre-commit.ubuntu.md/osx.md— note the inlineexport PATHis session-only (uv persists via the shell profile; open a new terminal).go2/index.md— correct the replay first-run download to ~85 MB LFS / ~220 MB SQLite DB (was "~2.4 GB of LiDAR/video"), resolving the README-vs-go2size inconsistency raised on Validate readme/install examples #2119.Notes
Draft — description to be expanded. Broader #2119 findings (agentic-runfile import crash on the PyPI wheel,
base[unitree]pulling torch, dev-install command divergence) will be summarized separately.Contributor License Agreement