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

Update general.py #9454

Merged
merged 2 commits into from Sep 17, 2022
Merged

Update general.py #9454

merged 2 commits into from Sep 17, 2022

Conversation

glenn-jocher
Copy link
Member

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

Signed-off-by: Glenn Jocher glenn.jocher@ultralytics.com

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improved dataset checking and automatic download logic in YOLOv5 utilities.

πŸ“Š Key Changes

  • Simplified YAML file reading using a new yaml_load function.
  • Enhanced path resolution for dataset files, fixing potential path issues.
  • Dataset auto-download now places files in a predefined DATASETS_DIR.

🎯 Purpose & Impact

🎁 Purpose:

  • To streamline the loading of dataset configurations by using a custom function.
  • To ensure dataset paths are correctly resolved, regardless of their initial relative path format.
  • To standardize the location of automatically downloaded datasets for better organization and consistency.

πŸš€ Impact:

  • Users will experience more reliable dataset loading with fewer path-related errors.
  • Automatic downloads become more predictable and less prone to clutter the user's directories.
  • Overall improvements to dataset management may simplify both development and user experience.

Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
@glenn-jocher glenn-jocher self-assigned this Sep 17, 2022
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
@glenn-jocher glenn-jocher merged commit 6a9fffd into master Sep 17, 2022
@glenn-jocher glenn-jocher deleted the update/hub_datasets branch September 17, 2022 13:42
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.

None yet

1 participant