Repository navigation
Conversation
This was referenced Oct 1, 2026
❌ 5 Tests Failed:
View the top 3 failed test(s) by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
This was referenced Oct 1, 2026
This was referenced Oct 1, 2026
This branch was successfully deployed
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
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.
Unused old rich message classes, obsolete interoperability implementations and the root Python dimos-lcm dependency are removed after consumers migrate. Raw LCM transport remains intentionally. This is partial retirement: the proprietary M20 onboard DrDDS bridge still uses the old generated message repository and cannot be declared migrated without its ABI-compatible AArch64 SDK/toolchain.
Base: #4364. This is layer 7/7, replacing the combined draft #4257 through the user-approved split. Original PR1–3 and the old feat/cdr-runtime-cutover branch/history are preserved. Reassembly uses selected files/hunks and forward merges, not a claim that the original commits cleanly cherry-pick.
Published HEAD:
060d4e13f32b76360c0905981967fe22f40a6539.Validation: 17 authoring and 97 offline runtime tests passed (4 LCM-named cases deselected); native CMake archive/Cargo crate built offline and relocated package acceptance passed; lock and hooks passed. The original publication checkpoint matched preserved complete reference cb98281 (tree 28ff1e5e6fbe0e95400e845770dce3936f553ac0). Subsequent differences are explicitly limited to the approved two-tutorial documentation, its validation/workflow changes, the shared CDR reference test dependency/lock, the shared CDR reference test dependency/lock, the shared CDR reference test dependency/lock, and four import-formatting fixes from autofix; no production API changes were introduced. GitHub CI is pending.
This remains a proposal with a deliberate old-message API/wire break. CDR replaces encoding; LCM/Zenoh remain transports. No old-recording fallback is promised. No merge or package registry release is requested. External gates remain M20 SDK, host multicast, Foxglove sign-in and human/full-pose visual acceptance. Benchmarks have not been waived or weakened.
Published replacement chain: #4247 → #4252 → #4254 → #4362 → #4363 → #4364 → #4365. Preserved full reference: #4257 at
cb982817bde248baa70a6fa69cc9ecc275c43707.Tutorial follow-up: the checkout workflow is documented in
docs/development/messages-in-repository.md; the installed-runtime/custom-project workflow is indocs/development/messages-external-project.md(runtime layer onward); both sharemessage-reference.md. Matching review wheels are required; pip does not resolve uv-only local sources. Clean isolated canonical .msg addition→uv build→wheel install→value/codec checks passed in 112 seconds without installing the DimOS runtime. Existing isolated external source/wheel/sdist/editable acceptance and exact three-language coordinator evidence are retained; embedded C++ includes are now checked correctly. Required toolchains, SDK artifacts and harness acquisition are explicit.Initial authoring CI exposed a premature root Rust workspace join (lock/Nix source mismatch) and a cold docs build missing Fast CDR. The generated crate remains standalone in authoring and joins the root workspace at runtime cutover; the existing Fast CDR setup now precedes package installation. Locks/assertions/gates were not relaxed. New exact-HEAD CI remains pending.
Authoring namespace test runner now invokes python -m pytest so a clean checkout imports the lightweight message namespace; 11 focused project tests passed. Self-hosted setup now explicitly installs CMake before building Fast CDR, and rosbags==0.11.0 and mcap>=1.2.0 are declared in the shared test group with its locked dependency; eight focused ownership/MCAP cases and hooks passed. Exact updated-head checks remain pending.
Superseded by naming-only replacement #4371. Exact commit
060d4e13f32b76360c0905981967fe22f40a6539, source tree and parent-relative diff are unchanged.Verified native seven-layer stack #4372: #4247 → #4366 → #4367 → #4368 → #4369 → #4370 → #4371. Stack: #4247
All branches use cc/ and the ordered PR bases preserve the prior commit ancestry. Old branches/history are retained. Replacement PR checks are pending and prior CI retains its exact SHA/run/ref provenance. No merge, force push or release.