Skip to content

feat(poi): multi-container DLStreamer scaling and benchmark reliability#227

Merged
avinash-palleti merged 2 commits into
intel-retail:mainfrom
sainijit:main
May 29, 2026
Merged

feat(poi): multi-container DLStreamer scaling and benchmark reliability#227
avinash-palleti merged 2 commits into
intel-retail:mainfrom
sainijit:main

Conversation

@sainijit
Copy link
Copy Markdown
Contributor

  • Fix DLStreamer config generation to distribute cameras across both lp-video containers (round-robin)
  • Improve alert thumbnail saving and stale alert filtering
  • Add E2E latency measurement from alerts API
  • Better single-run benchmark handling with early exit on alert

PR Checklist

  • Added label to the Pull Request for easier discoverability and search
  • Commit Message meets guidelines as indicated in the URL https://github.com/intel-retail/performance-tools/blob/main/CONTRIBUTING.md
  • Every commit is a single defect fix and does not mix feature addition or changes
  • Unit Tests have been added for new changes
  • Updated Documentation as relevant to the changes
  • All commented code has been removed
  • If you've added a dependency, you've ensured license is compatible with repository license and clearly outlined the added dependency.
  • PR change contains code related to security
  • PR introduces changes that breaks compatibility with other modules (If YES, please provide details below)

What are you changing?

Issue this PR will close

close: #issue_number

Anything the reviewer should know when reviewing this PR?

Test Instructions if applicable

If the there are associated PRs in other repositories, please link them here (i.e. intel-retail/performance-tools )

sainijit and others added 2 commits May 29, 2026 14:52
- Fix DLStreamer config generation to distribute cameras across both
  lp-video containers (round-robin)
- Improve alert thumbnail saving and stale alert filtering
- Add E2E latency measurement from alerts API
- Better single-run benchmark handling with early exit on alert

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Check both lp-video and lp-video-2 in readiness poll
- Fix Camera_02 fallback to rename references (not use Camera_01 as-is)
- Recreate poi-backend so env vars (RTSP_PREWARM_CAMERAS, etc.) refresh
- Wait for web health before scene cleanup docker exec calls
- Fix cmd_generate/cmd_clean ordering: reinit before dlstreamer config

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 updates the POI stream-density benchmark flow to better support multiple DLStreamer containers and reduce benchmark wait time.

Changes:

  • Distributes generated DLStreamer pipelines across lp-video and lp-video-2.
  • Reworks scaling to start extra camera services and recreate DLStreamer containers with readiness polling.
  • Adds parallel scene cleanup/import and service recreation behavior.

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

Comment thread benchmark-scripts/poi_stream_density.py Outdated
Comment thread benchmark-scripts/poi_stream_density.py Outdated
Comment thread benchmark-scripts/poi_stream_density.py
Comment thread benchmark-scripts/poi_stream_density.py Outdated
Comment thread benchmark-scripts/poi_stream_density.py
@avinash-palleti avinash-palleti merged commit 1c135d8 into intel-retail:main May 29, 2026
7 checks passed
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.

3 participants