Skip to content

feat(capture): export config and cut over SharedCamera transport#205

Open
maxxgx wants to merge 2 commits into
max/comp-config-2-robotfrom
max/comp-config-3-camera
Open

feat(capture): export config and cut over SharedCamera transport#205
maxxgx wants to merge 2 commits into
max/comp-config-2-robotfrom
max/comp-config-3-camera

Conversation

@maxxgx

@maxxgx maxxgx commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Decorates camera drivers with @export_config and adds capture/transport/builtin.py.
  • Cuts over camera publisher stdin envelopes to camera: ComponentConfig.
  • Updates SharedCamera.from_config() and tutorial notebook SharedCamera usage.

Stacked on #204.

Note: collect_train_deploy.ipynb is included here (not PR4) so pyrefly passes after the SharedCamera construction API change.

Test plan

  • uv run pytest tests/unit/capture/test_camera_component_config.py tests/unit/capture/test_factory.py tests/unit/capture/test_transport.py

Review & merge order

  1. PR #202 — Core config enginemain
    Core physicalai.config (export, instantiate, validation)

  2. PR #204 — Robot drivers & SharedRobot transport
    Robot config export and SharedRobot owner process startup

  3. PR #205 — Camera drivers & SharedCamera transport (this PR)
    Camera config export and SharedCamera publisher startup

  4. PR #206 — Runtime, InferenceModel & CLI
    End-to-end YAML, physicalai run --config, examples, and user docs

In parallel: PR #203 — Component config design docdocs/design
Detailed design doc; not blocking the code PRs

Decorate camera drivers with @export_config, add transport builtin type
mapping, and cut over camera publisher stdin envelopes to camera:
ComponentConfig. Update SharedCamera.from_config() and the tutorial
notebook SharedCamera usage for the new construction API.

Co-authored-by: Cursor <cursoragent@cursor.com>
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