Skip to content

Fix www.youtube.com hostname#7242

Merged
glenn-jocher merged 2 commits intomasterfrom
update/youtube
Apr 1, 2022
Merged

Fix www.youtube.com hostname#7242
glenn-jocher merged 2 commits intomasterfrom
update/youtube

Conversation

@glenn-jocher
Copy link
Copy Markdown
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