Skip to content

dimSLAM docs - #3658

Open
jeff-hykin wants to merge 7 commits into
jeff/feat/alfred_dim_slamfrom
jeff/docs/dim_slam
Open

dimSLAM docs#3658
jeff-hykin wants to merge 7 commits into
jeff/feat/alfred_dim_slamfrom
jeff/docs/dim_slam

Conversation

@jeff-hykin

Copy link
Copy Markdown
Member

Adds dimos/mapping/dim_slam/README.md: ports, the six bring-up steps for a robot dimSLAM has never run on (cameras, extrinsics, sources, IMU, run, verify), and a table of the settings that have no sane default.

Docs only, no code change.

@codecov

codecov Bot commented Aug 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

@@                    Coverage Diff                     @@
##           jeff/feat/alfred_dim_slam    #3658   +/-   ##
==========================================================
  Coverage                      77.79%   77.79%           
==========================================================
  Files                           1296     1296           
  Lines                         123918   123918           
  Branches                       10852    10852           
==========================================================
  Hits                           96402    96402           
  Misses                         24375    24375           
  Partials                        3141     3141           
Flag Coverage Δ
OS-ubuntu-24.04-arm 72.78% <ø> (ø)
OS-ubuntu-latest 74.62% <ø> (ø)
Py-3.10 74.62% <ø> (+<0.01%) ⬆️
Py-3.11 74.62% <ø> (-0.01%) ⬇️
Py-3.12 74.62% <ø> (+<0.01%) ⬆️
Py-3.13 74.62% <ø> (+<0.01%) ⬆️
Py-3.14 74.62% <ø> (+<0.01%) ⬆️
Py-3.14t 74.62% <ø> (+<0.01%) ⬆️
SelfHosted-Large 30.11% <ø> (ø)
SelfHosted-Linux 35.30% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@greptile-apps

greptile-apps Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds a dimSLAM bring-up guide covering ports, camera and transform setup, odometry fusion, IMU configuration, execution, verification, and robot-specific settings. The guide now correctly identifies OdometryPath as the visualization component used by demo-cuvslam-realsense.

Confidence Score: 5/5

No blocking failure remains.

The documented demo component was checked against the current blueprint, implementation, registry, and Python sources; the README matches the available component.

T-Rex T-Rex Logs

What T-Rex did

  • A reproducible validation script parsed the README and demo blueprint, confirmed OdometryPath is the component used by the demo, ensured OdometryHist does not appear in Python sources, and verified the component registry entry.
  • The prior README text named OdometryHist at line 115 was captured, and subsequent validation ran with all repository assertions passing and exit code 0.
  • Artifacts including the validation script and the passing logs were prepared and uploaded to support review.

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (23): Last reviewed commit: "Merge branch 'rb_alfred' into rb_docs" | Re-trigger Greptile

Comment thread dimos/mapping/dim_slam/README.md Outdated
@jeff-hykin jeff-hykin changed the title dim_slam: bring-up guide for a new robot dimSLAM docs Aug 24, 2026
@jeff-hykin
jeff-hykin force-pushed the jeff/feat/alfred_dim_slam branch from f8c3827 to 2582f20 Compare August 25, 2026 08:30
@jeff-hykin
jeff-hykin force-pushed the jeff/feat/alfred_dim_slam branch from 2582f20 to 34cea65 Compare August 25, 2026 08:50
@jeff-hykin
jeff-hykin force-pushed the jeff/feat/alfred_dim_slam branch from 34cea65 to 549b7c5 Compare August 25, 2026 09:03
@github-actions github-actions Bot added the ready-to-merge Required CI checks have passed on this PR label Aug 25, 2026
@github-actions github-actions Bot added ready-to-merge Required CI checks have passed on this PR and removed ready-to-merge Required CI checks have passed on this PR labels Aug 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Required CI checks have passed on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant