Repository navigation
Conversation
1 task
❌ 4 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.
The actual three-language blueprint is recorded to self-describing MCAP, producers are stopped, then independent decoding and DimOS replay verify every payload in per-stream order. A separately generated external message package is recorded by the unchanged native recorder and decoded from its embedded schema. This layer contains substantive integration demos, viewer-decoder checks and cumulative README/evidence; required production storage/schema code is already in the runtime layer.
Base: #4363. This is layer 6/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:
180109d0648f83c2c6c58d92dad20213ef7d1495.Validation: Candidate-built native recorder/relays: 16 Rust tests passed; 3 SQLite/MCAP/TF interoperability cases passed. Nine blueprint rows independently decoded and byte-exact replayed after producers stopped. Three external Telemetry rows recorded over Zenoh and decoded with pinned Foxglove libraries without viewer-side custom package. Unknown external native stamp layouts use reception time; payload Header remains preserved. Hooks passed. Full LCM/manual viewer acceptance and GitHub CI remain 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 #4370. Exact commit
180109d0648f83c2c6c58d92dad20213ef7d1495, 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.