Skip to content

VPP speed improvements#1820

Closed
MaticTonin wants to merge 2 commits into
developfrom
bugfix/VPP_60_FPS
Closed

VPP speed improvements#1820
MaticTonin wants to merge 2 commits into
developfrom
bugfix/VPP_60_FPS

Conversation

@MaticTonin

Copy link
Copy Markdown
Collaborator

Purpose

Improve Neural Assisted Stereo throughput on RVC4. The VPP path was limiting stereo.disparity output to roughly 30 FPS even when 60 FPS was requested. This change removes unnecessary VPP frame copies and reduces energy-map computation overhead, allowing NAS to reach ~60 FPS with the optimized FW.

Dependencies & Potential Impact

No external dependency changes.

Deployment Plan

Roll out with the updated RVC4 firmware package.

Rollback: revert this PR or boot devices with the previous RVC4 FW package.

Monitoring: use the updated NAS profiling example or BenchmarkIn telemetry to confirm stereo.disparity FPS, latency, CPU, memory, and temperature.

Testing & Validation

Validated on RVC4 device 10.12.161.25.

Old FW @ 60 FPS request:

  • FPS: 29.83
  • Avg latency: 191.74 ms

New FW @ 60 FPS:

  • FPS: 59.98
  • Avg latency: 100.12 ms

Result:

  • Throughput: 2.01x
  • Latency reduction: 47.8%

Also validated new FW @ 30 FPS:

  • FPS: 30.53
  • Avg latency: 115.45 ms

Old FW @ 30 FPS:

  • FPS: 29.86
  • Avg latency: 212.31 ms

Added C++ test:

  • [NeuralAssistedStereo] Requested 60 FPS reaches at least 50 FPS

AI Usage

Assisted-by: OpenAI:Codex

Submitted code was reviewed by a human: YES

The author is taking the responsibility for the contribution: YES

@coderabbitai

coderabbitai Bot commented May 28, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

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

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 21fd5cff-d496-4113-a8fb-992a75959271

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
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bugfix/VPP_60_FPS

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@moratom moratom requested a review from JakubFara May 29, 2026 10:00
@moratom moratom closed this Jun 5, 2026
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