Repository navigation
Ivan/fix/doc codeblocks skip2 - #2052
Conversation
Fixes doc-codeblocks CI: nav_stack.md, g1/index.md, and visualization.md contain illustrative blocks that can't run in CI (placeholder syntax, missing imports across subprocess boundaries, hardware/network dependencies, gRPC server startup). Per docs/agents/docs/codeblocks.md, mark each with `skip`.
Greptile SummaryThis PR fixes CI failures introduced by the previous PR (#2043) by adding the
Confidence Score: 5/5Safe to merge — adds All changes are targeted and mechanical: blocks that reference hardware, placeholder module imports, undefined variables, or environment-mutating shell commands are correctly flagged No files require special attention. Important Files Changed
Sequence DiagramsequenceDiagram
participant PR as Pull Request
participant GH as GitHub Actions
participant BIN as bin/run-doc-codeblocks
participant MB as md-babel-py
PR->>GH: Push / synchronize
GH->>BIN: /entrypoint.sh bash -c "./bin/run-doc-codeblocks --ci --no-cache"
BIN->>MB: md-babel-py run --lang python,sh,node ./docs --recursive
loop "Each code block in docs/**"
MB->>MB: Check fence flags
alt "flag == skip"
MB-->>MB: Skip block (no execution)
else no flag
MB->>MB: Execute block
alt Execution succeeds
MB-->>BIN: OK
else Execution fails
MB-->>GH: CI FAILURE
end
end
end
BIN-->>GH: All blocks passed
GH-->>PR: doc-codeblocks check green
Reviews (2): Last reviewed commit: "checking if code change is required" | Re-trigger Greptile |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
* perf(rerun): render voxel maps as Points3D spheres instead of Boxes3D (dimensionalOS#1793) * feat(api): add porcelain api with connect (dimensionalOS#1779) * fix(rerun): dimos-viewer version upgrade (dimensionalOS#1785) * Task: Add meaningful manipulation tests (dimensionalOS#1765) * feat: never leave child processes alive after the parent (dimensionalOS#1886) * Fix mac UDP (dimensionalOS#1789) * Remove sim from base install (dimensionalOS#1878) * fix(tests): fix flakey porcelain test (dimensionalOS#1900) * feat(go2): rage mode via webrtc (dimensionalOS#1903) * feat:rust native modules (dimensionalOS#1794) Co-authored-by: leshy <lesh@sysphere.org> * Feat/memory2 - plotting, examples, recorder module, semantic search (dimensionalOS#1769) Co-authored-by: RD <63036454+ruthwikdasyam@users.noreply.github.com> * fix attributes (dimensionalOS#1918) * Jeff/fix/rconnect2 (dimensionalOS#1784) * fix(types): resolve mypy 3.10 errors (dimensionalOS#1921) * feat(go2): go2 SDK adapter + nix cyclonedds setup (dimensionalOS#1885) Co-authored-by: Mustafa Bhadsorawala <39084056+mustafab0@users.noreply.github.com> * Revert "Jeff/fix/rconnect2" (dimensionalOS#1924) * feat(modules): async modules (dimensionalOS#1920) * chore(logging): enforce usage of setup_logger (dimensionalOS#1895) * feat(api): peek_stream (dimensionalOS#1909) * CI/macos: skip failing tests (dimensionalOS#1944) * fix(ci): restrict code-cleanup workflow to Linux runners only (dimensionalOS#1948) * Fix/memory2 go2 autorecorder (dimensionalOS#1925) * Convert patrol module to be an async module (dimensionalOS#1939) * feat(tool-streams): add tool streams (dimensionalOS#1713) * G1 Low level connection module coordinator integration (dimensionalOS#1954) * feat(sim): MuJoCo sim teleop for xarm + piper (dimensionalOS#1958) * Show child process cpu usage in dtop (dimensionalOS#1880) Co-authored-by: leshy <lesh@sysphere.org> * Task: OpenArm Integration with DimOS (dimensionalOS#1897) Co-authored-by: Paul Nechifor <paul@nechifor.net> Co-authored-by: RD <63036454+ruthwikdasyam@users.noreply.github.com> Co-authored-by: Andrew Lauer <69774903+aclauer@users.noreply.github.com> Co-authored-by: leshy <lesh@sysphere.org> Co-authored-by: Jeff Hykin <jeff.hykin@gmail.com> * md-babel-py bump (dimensionalOS#1994) * new hong kong office recording (dimensionalOS#1991) * chore(security): update packages with issues (dimensionalOS#1989) * Make whisper an optional extra with faster-whisper by default (dimensionalOS#1877) * Drop perception. Update docs. (dimensionalOS#1888) * Add codecov (dimensionalOS#1889) * go2 lidar timestamps bugfix (dimensionalOS#1992) * go2 cli (dimensionalOS#1990) * refactor(core): memory2 for dtop (dimensionalOS#2004) * fix: adaptive timestamp correction (dimensionalOS#2021) Co-authored-by: Ivan Nikolic <lesh@sysphere.org> * Disable codecov status check (dimensionalOS#2026) * fixes replay mem leak (dimensionalOS#2025) * fix: launch arm blueprints with --simulation flag (dimensionalOS#2027) * small fixes for release (dimensionalOS#2019) * chore: add problems back * Change Codecov branch from 'dev' to 'main' * docs: remove mentions of dev * Ivan/fix/go2recording2 (dimensionalOS#2034) * mapper memory leak fix, measurement transforms (dimensionalOS#2031) * Feat/1759 doc codeblocks ci (dimensionalOS#1884) * Nav Stack 0.1 * fix(security): upgrade some packages (dimensionalOS#2040) * go2 apriltag recordings (dimensionalOS#2044) * Ivan/fix/doc codeblocks added skip2 (dimensionalOS#2052) * feat Aruco/Apriltag generator (dimensionalOS#2037) * Split tests off of self hosted runner (dimensionalOS#1901) Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> * fix(verify): exclude unitree-dds extra to avoid cyclonedds C lib Upstream introduced `dimos[unitree-dds]` which depends on `cyclonedds==0.10.5`. The Python binding tries to compile against the CycloneDDS C library, which is not installed on the fork's ubuntu-latest runner. This makes verify fail with: Could not locate cyclonedds. Try to set CYCLONEDDS_HOME or CMAKE_PREFIX_PATH Upstream's own `ci.yml` (line 36) already passes `--no-extra dds --no-extra unitree-dds`; align the fork's verify with it. Co-authored-by: Cursor <cursoragent@cursor.com> --------- Co-authored-by: RD <63036454+ruthwikdasyam@users.noreply.github.com> Co-authored-by: Paul Nechifor <paul@nechifor.net> Co-authored-by: Mustafa Bhadsorawala <39084056+mustafab0@users.noreply.github.com> Co-authored-by: Sam Bull <git@sambull.org> Co-authored-by: Andrew Lauer <69774903+aclauer@users.noreply.github.com> Co-authored-by: leshy <lesh@sysphere.org> Co-authored-by: Jeff Hykin <jeff.hykin@gmail.com> Co-authored-by: stash <pomichterstash@gmail.com> Co-authored-by: Dan Vi <bogwi@tutamail.com> Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com> Co-authored-by: jiangtao-huazhijian <jiang.tao@topsunpower.cc> Co-authored-by: Cursor <cursoragent@cursor.com>
CI broken on prev PR #2043