Add cloudini integration and streamline README quickstart#98
Merged
shouhengyi merged 3 commits intostagefrom Mar 10, 2026
Merged
Add cloudini integration and streamline README quickstart#98shouhengyi merged 3 commits intostagefrom
shouhengyi merged 3 commits intostagefrom
Conversation
Integrates cloudini (https://github.com/facontidavide/cloudini) as a pipeline task that decodes CompressedPointCloud2 messages from MCAP topics. Includes three opt-out mechanisms: skip if dependencies not installed, per-task YAML flag, and global CLOUDINI_ENABLED setting (default true). Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…tcut Consolidated the quickstart into a clear 3-step flow with an inline service table, direct claude mcp add command, and a tip for Claude Code users to just paste the repo link and specify their environment.
There was a problem hiding this comment.
Pull request overview
Adds Cloudini pointcloud decoding support to the pipeline system and updates docs to make getting started and opting out straightforward.
Changes:
- Introduces
DecodePointCloudTaskto decode Cloudini-compressed pointcloud topics and write CSV/NPZ outputs with opt-out controls. - Adds a vendored Cloudini WASM decoder plus new settings and dependency-group wiring (
cloudini). - Updates README quickstart and adds a dedicated Cloudini runbook with configuration guidance.
Reviewed changes
Copilot reviewed 7 out of 13 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
uv.lock |
Adds the cloudini dependency group metadata and locks wasmtime artifacts. |
pyproject.toml |
Adds cloudini dependency group; excludes vendored decoder from Ruff. |
settings.py |
Adds global Cloudini enable flag and default resolution setting. |
src/pipeline/tasks/cloudini/decode_pointcloud.py |
New pipeline task implementation for decoding and writing pointcloud outputs. |
src/pipeline/tasks/cloudini/vendor/cloudini_decoder.py |
Vendored WASM-based decoder implementation used by the task. |
src/pipeline/tasks/cloudini/__init__.py |
Adds module package init/docstring. |
test/pipeline/tasks/cloudini/test_decode_pointcloud.py |
Adds unit tests for availability flags, opt-out behavior, extraction, and writing. |
doc/runbooks/cloudini.md |
Documents installation, YAML configuration, opt-out mechanisms, and outputs. |
README.md |
Streamlines quickstart and links the Cloudini integration documentation. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Collaborator
Author
|
@shouhengyi please test at earliest convienience |
Collaborator
|
LGTM |
Collaborator
|
Approved
…On Mon, Mar 9, 2026 at 2:10 PM Arun Venkatadri ***@***.***> wrote:
*arunvenkatadri* left a comment (Extelligence-ai/bagel#98)
<#98 (comment)>
@shouhengyi <https://github.com/shouhengyi> please test at earliest
convienience
—
Reply to this email directly, view it on GitHub
<#98 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AIM3Z4ZORLCV2AUZTB2JQUT4P4XMFAVCNFSM6AAAAACWMKK5RWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHM2DAMRWHEZTKNRRGE>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
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.
Summary
Test plan