Skip to content

sensing_control: add a mock producer that runs the management channel without cameras - #1470

Closed
gareth-morgan-nv wants to merge 1 commit into
gmorgan/EgoCapVideoStage/ProducerControlfrom
gmorgan/EgoCapVideoStage/MockProducer
Closed

gareth-morgan-nv wants to merge 1 commit into
gmorgan/EgoCapVideoStage/ProducerControlfrom
gmorgan/EgoCapVideoStage/MockProducer

Conversation

@gareth-morgan-nv

@gareth-morgan-nv gareth-morgan-nv commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Summary

sensing_mock_producer: a producer with no cameras. It plays looped H.264 files through the same management channel, retry loop and direct sender as camera_plugin_sensing, with --simulate-fault-seconds to show the unplugged-camera path, so the PC side can be tried without an Orin. An end-to-end test drives it through a real video_stage_worker.

It needs both the management-channel library (#1465) and the video stage's sender, so it is stacked on the producer-control PR (#1469) and builds with BUILD_SENSING_CONTROL=ON (and, for the end-to-end test, BUILD_VIDEO_STAGE=ON).

Validation

  • Built with BUILD_SENSING_CONTROL=ON alone (mock producer and library build; the end-to-end test is not registered) and with BUILD_VIDEO_STAGE=ON as well.
  • sensing_mock_producer_e2e (7 tests) passes only when given an input stream through SENSING_E2E_INPUT (it otherwise reads a local tmp/vs/orin_like.h264); without one all 7 are skipped and ctest still reports it as passed.
  • Not run: anything on an Orin.

🤖 Generated with Claude Code

… without cameras

sensing_mock_producer plays looped H.264 files through the same management channel, retry loop
and direct sender as camera_plugin_sensing, with --simulate-fault-seconds to show the
unplugged-camera path, so the PC side can be tried without an Orin. An end-to-end test drives it
through a real video_stage_worker.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Signed-off-by: Gareth Morgan <gmorgan@nvidia.com>
@coderabbitai

coderabbitai Bot commented Nov 10, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

@gareth-morgan-nv

Copy link
Copy Markdown
Contributor Author

Closing as superseded: the camera path is now the camera_streamer architecture (SIPL/Argus sender, AVS1 over TCP, PC receiver and LiveKit bridge; see #1473-#1479). Not merged; the branch and commits are kept.

🤖 Generated with Claude Code

@gareth-morgan-nv

Copy link
Copy Markdown
Contributor Author

This PR is superseded by the camera_streamer PRs:

🤖 Generated with Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants