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

Fix www.youtube.com hostname #7242

Merged
merged 2 commits into from
Apr 1, 2022
Merged

Fix www.youtube.com hostname #7242

merged 2 commits into from
Apr 1, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Apr 1, 2022

Fix for #7241

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Enhanced YouTube source handling in YOLOv5 dataset streaming.

πŸ“Š Key Changes

  • Expanded URL checks to include 'www.youtube.com' along with 'youtube.com' and 'youtu.be'.

🎯 Purpose & Impact

  • 🎯 Purpose: Ensures that YouTube video URLs with 'www' are recognized as valid sources for streaming.
  • πŸ” Impact: Users will experience improved flexibility and fewer errors when using YouTube URLs in their datasets. This will be particularly beneficial for applications relying on real-time video analysis or training models with varied YouTube video data.

@glenn-jocher glenn-jocher linked an issue Apr 1, 2022 that may be closed by this pull request
2 tasks
@glenn-jocher glenn-jocher self-assigned this Apr 1, 2022
@glenn-jocher glenn-jocher merged commit 37675e1 into master Apr 1, 2022
@glenn-jocher glenn-jocher deleted the update/youtube branch April 1, 2022 19:38
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
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.

Error using YouTube address in detect.py
1 participant