Skip to content

feat(poi): benchmark device/precision propagation and reliability fixes#226

Merged
sainijit merged 1 commit into
intel-retail:mainfrom
sainijit:main
May 26, 2026
Merged

feat(poi): benchmark device/precision propagation and reliability fixes#226
sainijit merged 1 commit into
intel-retail:mainfrom
sainijit:main

Conversation

@sainijit
Copy link
Copy Markdown
Contributor

  • Add --resource_config CLI arg to poi_stream_density.py so DEVICE and MODEL_PRECISION are propagated through all stream-density re-init iterations (make benchmark DEVICE=all-gpu.env now works end-to-end)
  • Catch PermissionError in _save_alert_thumbnails: logs a clear warning instead of crashing when results/ is root-owned
  • Clarify true E2E latency measurement: the primary VLM metrics-file latency (e.g. 128ms) is frame capture → alert dispatch (start set via user_log_start_time with DLStreamer frame timestamp, end via log_end_time at alert dispatch). Corrects misleading comments that said the metric excluded DLStreamer pipeline latency.

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 )

- Add --resource_config CLI arg to poi_stream_density.py so DEVICE and
  MODEL_PRECISION are propagated through all stream-density re-init
  iterations (make benchmark DEVICE=all-gpu.env now works end-to-end)
- Catch PermissionError in _save_alert_thumbnails: logs a clear warning
  instead of crashing when results/ is root-owned
- Clarify true E2E latency measurement: the primary VLM metrics-file
  latency (e.g. 128ms) is frame capture → alert dispatch (start set via
  user_log_start_time with DLStreamer frame timestamp, end via
  log_end_time at alert dispatch). Corrects misleading comments that
  said the metric excluded DLStreamer pipeline latency.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@sainijit sainijit merged commit 8e5a847 into intel-retail:main May 26, 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.

2 participants