Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove streaming source sleep period #8243

Merged
merged 1 commit into from
Jun 17, 2022
Merged

Remove streaming source sleep period #8243

merged 1 commit into from
Jun 17, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jun 17, 2022

Negatively impacts YouTube inference but removes any lag on webcams/RTSP/RTMP etc.

Resolves #4465

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Optimized video stream handling in YOLOv5 data loader for improved performance.

πŸ“Š Key Changes

  • Removed the fixed sleep time based on frames per second (fps) in the video data loader.
  • The sleep time after each frame is now set to 0.0 seconds.

🎯 Purpose & Impact

  • πŸš€ Purpose: This change aims to eliminate unnecessary delays when processing video streams, which can contribute to real-time performance.
  • βœ… Impact: Users may experience faster video processing with reduced latency, making the YOLOv5 model more efficient for real-time applications like surveillance or live video analysis.

Negatively impacts YouTube inference but removes any lag on webcams/RTSP/RTMP etc.

Resolves #4465
@glenn-jocher glenn-jocher self-assigned this Jun 17, 2022
@glenn-jocher glenn-jocher merged commit 86e6add into master Jun 17, 2022
@glenn-jocher glenn-jocher deleted the update/streams branch June 17, 2022 15:59
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
Negatively impacts YouTube inference but removes any lag on webcams/RTSP/RTMP etc.

Resolves ultralytics#4465
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.

Detection Latency with live streaming sources (RTMP)
1 participant