Skip to content

Disable test-intra-camera-sync.py until stabilized - #14626

Merged
Nir-Az merged 1 commit into
developmentfrom
Nir-Az-patch-2
Jan 20, 2026
Merged

Nir-Az merged 1 commit into
developmentfrom
Nir-Az-patch-2

Conversation

@Nir-Az

@Nir-Az Nir-Az commented Jan 20, 2026

Copy link
Copy Markdown
Collaborator

Disable test-intra-camera-sync.py until stabilized
@ipilchin1 FYI

-D- found <pyrealsense2.device: D457 (S/N: 220422302666  FW: 5.17.1.7  UNLOCKED  on GMSL)>
-D- in <module 'pyrealsense2' from '/mnt/ci/jenkins_ws/workspace/LRS_jetson_compile_pipeline_build/arm64/static/Release/pyrealsense2.cpython-39-aarch64-linux-gnu.so'>

___
-I- Test: Intra-camera depth-color synchronization with MASTER sync mode
-I- Using profiles: Depth 640x480@30fps, Color 640x480@30fps
-I- Enabling global time on both sensors...
-I- Global time enabled on depth sensor
-I- Global time enabled on color sensor
-I- Setting depth sensor to MASTER sync mode (inter_cam_sync_mode=1)...
-I- Started intra-camera multi-stream capture
-I- Stabilization period: 2.0 seconds...
-I- Collecting synchronized frames for 10.0 seconds...
-I- Collected 301 depth frames and 75 color frames
-E- Traceback (most recent call last):
      File "/mnt/ci/jenkins_ws/workspace/LRS_jetson_compile_pipeline_build/unit-tests/live/camera-sync/test-intra-camera-sync.py", line 325, in <module>
        test.check(len(color_frames) >= expected_min_frames,
    Sufficient color frames collected (75 >= 240)
-I- Synchronization analysis results:
-I-   Max timestamp gap: 432.108 ms
-I-   95th percentile gap: 234.208 ms
-I-   Average timestamp gap: 106.007 ms
-I-   Sync percentage: 24.9%
-I-   Total frame pairs: 301
-I-   Synced pairs (within 3.0ms): 75
-I- First few aligned frame pairs:
-I-   Pair 1: Gap 1.020ms, Depth#73, Color#3, HW gap: 1.022ms
-I-   Pair 2: Gap 1.020ms, Depth#74, Color#4, HW gap: 1.022ms
-I-   Pair 3: Gap 1.020ms, Depth#75, Color#5, HW gap: 1.022ms
-I-   Pair 4: Gap 32.229ms, Depth#76, Color#5, HW gap: 32.294ms
-I-   Pair 5: Gap 65.476ms, Depth#77, Color#5, HW gap: 65.610ms
-I-   Pair 6: Gap 98.720ms, Depth#78, Color#5, HW gap: 98.926ms
-I-   Pair 7: Gap 132.050ms, Depth#79, Color#5, HW gap: 132.242ms
-I-   Pair 8: Gap 165.384ms, Depth#80, Color#5, HW gap: 165.559ms
-I-   Pair 9: Gap 198.718ms, Depth#81, Color#5, HW gap: 198.875ms
-I-   Pair 10: Gap 232.051ms, Depth#82, Color#5, HW gap: 232.191ms
-E- Traceback (most recent call last):
      File "/mnt/ci/jenkins_ws/workspace/LRS_jetson_compile_pipeline_build/unit-tests/live/camera-sync/test-intra-camera-sync.py", line 356, in <module>
        test.check(sync_results['p95_gap'] <= SYNC_GAP_THRESHOLD_MS,
    95th percentile timestamp gap 234.208ms <= 3.0ms threshold
-W- Note: Max gap (432.108ms) exceeds threshold but p95 is within bounds
-E- Traceback (most recent call last):
      File "/mnt/ci/jenkins_ws/workspace/LRS_jetson_compile_pipeline_build/unit-tests/live/camera-sync/test-intra-camera-sync.py", line 362, in <module>
        test.check(sync_results['sync_percentage'] >= 95.0,
    Synchronization percentage 24.9% >= 95%
-I- Frame continuity analysis:
-I-   Depth sensor: 301 frames, 0 discontinuities
-I-   Color sensor: 75 frames, 15 discontinuities
-E- Traceback (most recent call last):
      File "/mnt/ci/jenkins_ws/workspace/LRS_jetson_compile_pipeline_build/unit-tests/live/camera-sync/test-intra-camera-sync.py", line 390, in <module>
        test.check(color_continuous,
    Color frame continuity: 15 drops <= 5% of 75 frames
-E- Test failed

___

Copilot AI review requested due to automatic review settings January 20, 2026 19:13

Copilot AI left a comment

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.

Pull request overview

This PR disables the intra-camera synchronization test (test-intra-camera-sync.py) by adding a test:donotrun directive. The test is currently failing on Jetson platforms with the D457 camera, showing insufficient color frame collection (75 vs expected 240), poor synchronization (24.9% vs expected 95%), and excessive frame drops.

Changes:

  • Add test:donotrun directive to prevent test execution until stability issues are resolved

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Nir-Az
Nir-Az merged commit cdddd3e into development Jan 20, 2026
47 of 48 checks passed
@Nir-Az
Nir-Az deleted the Nir-Az-patch-2 branch January 20, 2026 19:26
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.

2 participants