Skip to content

feat: render CDR memory recordings without value-class wrappers - #4705

Closed
TomCC7 wants to merge 1 commit into
cc/feat/cdr-recording-converter-integrationfrom
cc/feat/cdr-memory-rerun
Closed

TomCC7 wants to merge 1 commit into
cc/feat/cdr-recording-converter-integrationfrom
cc/feat/cdr-memory-rerun

Conversation

@TomCC7

@TomCC7 TomCC7 commented Oct 20, 2026

Copy link
Copy Markdown
Member

dimos mem rerun previously skipped every plain generated message because discovery required legacy to_rerun() methods. It now uses the shared external CDR adapters to export images/compressed images, clouds, pose/odometry, calibration and TF. Generated classes remain plain values; existing local rendering helpers remain supported during dependency retirement.

Parent: #4704. Message Encoding Refactor

The shared adapter now supports full Odometry fields (including covariance arrays), canonical TF frames and CameraInfo. Camera pairing and first-frame timing are retained, including compressed-image inputs. Source headers get an exact integer-nanosecond timeline. The live bridge also adapts generated TF into its existing numerical frame-tree layout helper when axes are enabled.

Changes Files Added Removed
Memory renderer and shared/live viewer adapters 3 142 42
Existing and generated viewer regressions 2 119 1
Copyable converter-to-Rerun walkthrough 1 6 0
Installed-package CI 1 1 1

Manual workflow with an existing trusted recording and new output names:

dimos mem convert old.db converted.mcap --dry-run
dimos mem convert old.db converted.mcap
dimos mem rerun converted.mcap --seconds 5 --no-gui

The final command writes an RRD; open that file with your Rerun viewer for visual acceptance. No GUI/hardware starts with --no-gui.

Validation: 41 source + 41 isolated installed-wheel checks, strict typing across 3 production files, configured scoped hooks pass except unavailable Nix. Actual installed root CLI exports the first five seconds of the real converted go2_mid360_stairs recording: color_image 70, fastlio_lidar 47, fastlio_odometry 129, lidar 38, odom 92. The 9.3 MiB RRD was inspected with the official Rerun CLI: all five entities, EncodedImage/Points3D/Transform3D/TextDocument and source timeline are present. Tests include exact epoch+1ns, real RRD axes/values and camera/TF timing; no human visual acceptance claimed.

Wheel SHA256 b45a70228180eb4085d3bc178c500ed6181c07a9056b86f9ec4fc19ec19e17ed; local actual RRD SHA256 8b50c3bcacef4edbe18530c6c7228b3b34a152fda9e54fa53f7c3a75f735e953. The local RRD is preserved but has not been delivered through Library. Old local geometry/frame-tree helpers remain behind adapters, so complete legacy dependency retirement remains open. Hosted/simulation/other graphs are still being migrated. No CI polling, hardware, merge, release or history rewrite.

@TomCC7 TomCC7 closed this Oct 20, 2026
@TomCC7
TomCC7 deleted the cc/feat/cdr-memory-rerun branch October 20, 2026 18:12
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.

1 participant