Skip to content

Record & Replay camera features#1702

Open
asahtik wants to merge 177 commits intodevelopfrom
feature/record_replay_features
Open

Record & Replay camera features#1702
asahtik wants to merge 177 commits intodevelopfrom
feature/record_replay_features

Conversation

@asahtik
Copy link
Copy Markdown
Contributor

@asahtik asahtik commented Mar 4, 2026

Purpose

Specification

None / not applicable

Dependencies & Potential Impact

None / not applicable

Deployment Plan

None / not applicable

Testing & Validation

None / not applicable

Summary by CodeRabbit

  • New Features

    • Device properties API and Python bindings to get/set device/default properties (buffer sizes, tuning blobs, chunk sizes).
    • Pipeline: set/get default device properties; per-default-device property routing.
    • Holistic recording/examples: full-resolution camera outputs, live viewfinder preview, camera-output synchronization, reduced IMU rate, OpenCV display, improved replay with camera-feature mocking.
    • Utility: read-a-file-from-tar helper and more consistent replay timing/looping.
  • Tests / CI

    • Added device properties on-device tests, centralized test infra, replay test support, and new workflow/flags to enable replay testing.

asahtik and others added 30 commits January 8, 2026 18:25
…s/depthai-core into feature/record_replay_features
Signed-off-by: stas.bucik <stas.bucik@luxonis.com>
Signed-off-by: stas.bucik <stas.bucik@luxonis.com>
Signed-off-by: stas.bucik <stas.bucik@luxonis.com>
Signed-off-by: stas.bucik <stas.bucik@luxonis.com>
Signed-off-by: stas.bucik <stas.bucik@luxonis.com>
Signed-off-by: stas.bucik <stas.bucik@luxonis.com>
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.

Nit: REQUIRE calls which need if(!pipeline.isHolisticReplayEnabled()) could be grouped together behind a single check

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.

Same NIT as for RVC4

Copy link
Copy Markdown
Contributor

@aljazdu aljazdu left a comment

Choose a reason for hiding this comment

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

LGTM - left some comments (mostly nits), which should be considered alongside existing code rabbit comments before merging

asahtik and others added 23 commits March 30, 2026 10:07
…epthai-core into feature/record_replay_features
…epthai-core into feature/record_replay_features
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.

6 participants