[farbod-nv]

OpenCV 5.0.0 ARM64 wheels lack V4L2 support, causing USB cameras to show a black screen while capture retries indefinitely.

Constrain the camera example’s OpenCV dependencies to <5 and validate V4L2 availability during setup and before capture initialization. Missing backend support now produces an actionable error; disconnected cameras retain normal reconnect behavior.

Summary by CodeRabbit

• Bug Fixes
  • Camera setup now checks that the V4L2 capture backend is available when V4L2 is enabled and reports an error if it is not.
  • V4L2 and video features now require OpenCV versions from 4.5 up to, but not including, 5. Installation guidance reflects this version range.

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

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

• Source mode: Auto
• Repository: NVIDIA/IsaacCapture
• Branch: fix/6954253
• File: .coderabbit.yaml
• Commit: e4b874c
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Enterprise
• Run ID: 48ac663c-687f-42f6-b2d9-b31ee7c11c1f

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

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: NVIDIA/IsaacCapture
• Branch: fix/6954253
• File: .coderabbit.yaml
• Commit: a074401
• Layers: Repository YAML
• Review profile: CHILL
• Plan: Enterprise
• Run ID: 4516b5e3-dc9d-493d-b9fa-286d2abbdbbd

Commits

Reviewing files that changed from the base of the PR and between 23ee7ff and a074401.

Files selected for processing: 7

• examples/camera_viz/pyproject.toml
• examples/camera_viz/scripts/_install_deps.sh
• examples/camera_viz/sources/_v4l2_backend.py
• examples/camera_viz/sources/v4l2.py
• examples/camera_viz/sources/video_file.py
• tests/python/examples/camera_viz/pyproject.toml
• tests/python/examples/camera_viz/test_v4l2_source.py

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 11 remain after this review.

Walkthrough

The camera visualization example now constrains OpenCV to versions from 4.5 up to, but not including, 5. A new check reports whether OpenCV has V4L2 support. Setup runs this check when V4L2 is enabled. V4l2Source loads the checked backend during initialization. Tests cover setup outcomes, backend availability, and camera capture lifecycle behavior.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Sequence Diagram(s)

sequenceDiagram
  participant install_deps
  participant v4l2_backend as _v4l2_backend.py
  participant cv2
  install_deps->>v4l2_backend: Run backend check
  v4l2_backend->>cv2: Import OpenCV and check V4L2 availability
  cv2-->>v4l2_backend: Return availability result
  v4l2_backend-->>install_deps: Return check status

Loading

Merge Risk: Minimal · previous review a0744

The V4L2 dependency and availability checks appear ready to merge after normal checks.

Pre-merge checks: 4 · 1

Failed checks: 1 warning

Check name	Status	Explanation	Resolution
Docstring Coverage		Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within survi…	Write docstrings for the functions missing them to satisfy the coverage threshold.

Passed checks: 4 passed

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly and concisely identifies the main change: fixing V4L2 camera capture on Jetson Thor. It matches the pull request objectives and changes.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 12 functions across 5 files. (2 skipped: 2 unsupported.)

Resolution

Write docstrings for the functions missing them to satisfy the coverage threshold.

Finishing Touches: 💡 1

📝 Generate docstrings: 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

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

_{Comment @coderabbitai help to get the list of available commands.}