Skip to content

fix(pointcloud): retain source records and headers in cloud operations - #4802

Closed
TomCC7 wants to merge 52 commits into
mainfrom
cc/fix/cdr-pointcloud-record-helpers
Closed

TomCC7 wants to merge 52 commits into
mainfrom
cc/fix/cdr-pointcloud-record-helpers

Conversation

@TomCC7

@TomCC7 TomCC7 commented Oct 23, 2026 •

Copy link
Copy Markdown
Member

Superseded by #4800. The implementation and regression tests from this PR are included in the consolidated message utility layer, with domain-module APIs and explicit ownership semantics.

The original branch and commits are preserved. Review the combined changes in #4800; the downstream TF layer #4803 now builds directly on it.

TomCC7 and others added 30 commits October 21, 2026 11:36

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

go2 replay realtime (arm64)

Details
Benchmark suite Current: b136925 Previous: 0ea05f6 Ratio
peak memory 1657.801 MB 1699.445 MB 0.98
peak threads 360 threads 357 threads 1.01
network (transport) 2722.648 MB 2725.914 MB 1.00
disk write 2.109 MB 1.469 MB 1.44
instructions 121.062 G 136.571 G 0.89

This comment was automatically generated by workflow using github-action-benchmark.

@TomCC7 TomCC7 left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

failed to understand why this needs a new pr, all message util addition can be added together

Base automatically changed from cc/refactor/cdr-geometry-value-helpers to main October 26, 2026 19:07
@TomCC7
TomCC7 removed this pull request from stack #4804 October 26, 2026 19:10
@TomCC7 TomCC7 closed this Oct 26, 2026

This branch was previously deployed

1 inactive deployment
cachix — b1369256 Deployed Oct 23, 2026 by TomCC7 via cachix-build-macos #14114
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