Feat/ptp example by asdwjure · Pull Request #1617 · luxonis/depthai-core

asdwjure · 2026-01-09T17:09:00Z

Purpose

Add PTP example.

Specification

None / not applicable

Dependencies & Potential Impact

None / not applicable

Deployment Plan

None / not applicable

Testing & Validation

None / not applicable

Summary by CodeRabbit

Release Notes

New Features
- Added TIME_PTP frame synchronization mode for multi-device camera coordination.
Improvements
- Updated multi-device frame synchronization examples to support both external and PTP synchronization modes.
- Enhanced device initialization behavior when using PTP synchronization.
Tests
- Added comprehensive integration tests for frame synchronization across multiple devices with external and PTP modes.

…erence Signed-off-by: Jure Rebernik <jure.rebernik@luxonis.com>

Signed-off-by: Jure Rebernik <jure.rebernik@luxonis.com>

moratom

Thanks!

We should merge the PTP example with FSync if possible, otherwise extract common functionality to a third file and reuse (and name examples clearly _ptp _fsync).

moratom · 2026-03-05T06:02:42Z

Can we merge the fsync & ptp examples?
90% of the code should be the same.

If not, we should name them, separate the common parts in a separate file in MultiDevice.

Ok, will separate. We could combine the code but at the cost of higher example complexity and we want the examples to be as easy to understand as possible...

moratom · 2026-03-05T06:03:06Z

+SYNC_THRESHOLD_SEC = 1.0 / (2 * TARGET_FPS)  # Max drift to accept as "in sync"
+SET_MANUAL_EXPOSURE = False  # Set to True to use manual exposure settings
+# DEVICE_INFOS: list[dai.DeviceInfo] = ["IP_MASTER", "IP_SLAVE_1"] # Insert the device IPs here, e.g.:
+DEVICE_INFOS = [dai.DeviceInfo(ip) for ip in ["192.168.0.146", "192.168.0.149"]] # The master camera needs to be first here


Is this true also for PTP that master needs to be first?
In any case, we should remove specific IPs from the script.

Not necessarily, but havent tested if we reverse this...
I will remove the IPs

moratom · 2026-03-05T06:03:14Z

CamelCase for consistency

moratom · 2026-03-05T06:03:47Z


 # "version if applicable"
-set(DEPTHAI_DEVICE_RVC4_VERSION "0.0.1+077602b7ebda52b86afe3fce4e289c449e6846f7")
+set(DEPTHAI_DEVICE_RVC4_VERSION "0.0.1+6caea117b7ae35099a5fe6723e21922c87e0b290")


RVC2 likely needs to be updated as well, as the enum is on the shared interface

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

…d ptp tests Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

This reverts commit bb6a950.

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

…to feat/ptp_example

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

asdwjure changed the base branch from main to develop January 9, 2026 17:09

asdwjure force-pushed the feat/ptp_example branch 3 times, most recently from 956bc21 to be97b93 Compare January 14, 2026 09:23

asdwjure requested a review from moratom January 14, 2026 09:25

asdwjure force-pushed the feat/ptp_example branch 4 times, most recently from f9cf89f to 64e5c4a Compare January 29, 2026 10:46

Matevz Morato and others added 6 commits March 4, 2026 20:42

Add a multi-device example

febe318

Update the example to sync based on start of exposure

da55620

Update the example

68344b8

multi_device.py: autodetect camera socket, print frame timestamp diff…

cb0376b

…erence Signed-off-by: Jure Rebernik <jure.rebernik@luxonis.com>

ptp: update example description

6f221db

Signed-off-by: Jure Rebernik <jure.rebernik@luxonis.com>

CameraControl, CameraControlBindings: add ptp options

9639251

Signed-off-by: Jure Rebernik <jure.rebernik@luxonis.com>

asdwjure force-pushed the feat/ptp_example branch 2 times, most recently from bed8c0b to a453743 Compare March 4, 2026 19:51

examples: MultiDevice: enable PTP via the new bindings

6d3acad

Signed-off-by: Jure Rebernik <jure.rebernik@luxonis.com>

asdwjure force-pushed the feat/ptp_example branch from a453743 to 13ed5d1 Compare March 4, 2026 20:02

bump fw for ptp changes

cd6d4b5

Signed-off-by: Jure Rebernik <jure.rebernik@luxonis.com>

asdwjure force-pushed the feat/ptp_example branch from 13ed5d1 to cd6d4b5 Compare March 4, 2026 20:41

moratom reviewed Mar 5, 2026

View reviewed changes

Jure Rebernik and others added 8 commits March 5, 2026 13:13

multi_device_ptp: camelCase and rename example

e43657b

Update fsync example to use threading

65639e3

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Add device names

149a042

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Make C++ example in line with python

07e130d

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Don't exit example if out of sync

81ca8d3

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Make test in line with examples

dd5941d

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Move examples to unified

ab5be0b

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Add PTP to multi device sync example

136490e

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

sb-lxn added 27 commits April 7, 2026 18:12

Tune external fsync/ptp tests

7a8f4b5

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Merge remote-tracking branch 'origin/develop' into feat/ptp_example

4408d27

Bump rvc4 device

a22c6bf

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Merge remote-tracking branch 'origin/develop' into feat/ptp_example

dffdff9

Bump rvc4 device

98a78b3

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Move ptp/fsync test to parent folder

15fd759

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Add initial timeout to multi device sync test

6a5bd43

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Split PTP and fsync test into separate executables

3202247

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Enable split fsync and PTP tests

e433a77

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Lower thresholds for PTP test

c5ef23c

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Fix PTP test not collecting any frames due to typo

e93f9cb

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Merge remote-tracking branch 'origin/develop' into feat/ptp_example

8545785

bump rvc4 device

4d1cc9e

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Fix ptp hil command to only pick 3 devices, add versions for fsync an…

0c42c4c

…d ptp tests Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Remove 60FPS tests and add 45FPS

82d70d0

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Disable uart when running fsync tests

4f21858

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Add adbutils to prepare hil framework

bb6a950

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Revert "Add adbutils to prepare hil framework"

3194b86

This reverts commit bb6a950.

Add adbutils in testing docker

6b6ab20

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Fix test label for ptp test

916109d

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Change testbed for external fsync tests

e61f7e3

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Merge remote-tracking branch 'origin/rvc4_dynamic_camera_features' in…

10f4897

…to feat/ptp_example

Add PTP to fsync types

3fcd02a

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

bump rvc4 device

0031d9b

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Add fsync type check to ptp and fsync tests

714d905

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Rename fsync test struct and fix windows build

9557750

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

Move adbutils import to the function where it is needed. Fixes mac tests

354c5ec

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

sb-lxn force-pushed the feat/ptp_example branch from 621a7c8 to 331ad23 Compare April 16, 2026 06:42

Bump rvc4 device

5b6d60f

Signed-off-by: stas.bucik <stas.bucik@luxonis.com>

sb-lxn force-pushed the feat/ptp_example branch from 331ad23 to 5b6d60f Compare April 16, 2026 09:58

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat/ptp example#1617

Feat/ptp example#1617
asdwjure wants to merge 61 commits intodevelopfrom
feat/ptp_example

asdwjure commented Jan 9, 2026 •

edited by coderabbitai bot

Loading

Uh oh!

moratom left a comment

Uh oh!

moratom Mar 5, 2026

Uh oh!

asdwjure Mar 5, 2026

Uh oh!

moratom Mar 5, 2026

Uh oh!

asdwjure Mar 5, 2026

Uh oh!

moratom Mar 5, 2026

Uh oh!

asdwjure Mar 5, 2026

Uh oh!

moratom Mar 5, 2026

Uh oh!

asdwjure Mar 5, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

asdwjure commented Jan 9, 2026 • edited by coderabbitai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Purpose

Specification

Dependencies & Potential Impact

Deployment Plan

Testing & Validation

Summary by CodeRabbit

Release Notes

Uh oh!

moratom left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

asdwjure commented Jan 9, 2026 •

edited by coderabbitai bot

Loading