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

Single cache_version definition #4846

Merged
merged 1 commit into from
Sep 17, 2021
Merged

Single cache_version definition #4846

merged 1 commit into from
Sep 17, 2021

Conversation

glenn-jocher
Copy link
Member

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

Defines dataset labels *.cache version in a single place, fixing a bug introduced in #4845.

๐Ÿ› ๏ธ PR Summary

Made with โค๏ธ by Ultralytics Actions

๐ŸŒŸ Summary

Updated dataset caching logic in YOLOv5 for improved consistency and maintainability.

๐Ÿ“Š Key Changes

  • Introduced cache_version variable in LoadImagesAndLabels class.
  • Updated cache validation to check against the cache_version variable.
  • Ensured cache version and file hash checks match current dataset.

๐ŸŽฏ Purpose & Impact

  • Purpose: The changes simplify the process of ensuring the cache format remains consistent across versions, making maintenance easier and reducing potential errors caused by format discrepancies.
  • Impact: Users will benefit from more consistent cache behavior when training or evaluating models. It will minimize issues related to outdated or mismatched cache data, leading to a smoother user experience. ๐Ÿ”„๐Ÿ’พ

Defines dataset labels *.cache version in a single place, fixing a bug introduced in #4845.
@glenn-jocher glenn-jocher self-assigned this Sep 17, 2021
@glenn-jocher glenn-jocher merged commit 4fdaec0 into master Sep 17, 2021
@glenn-jocher glenn-jocher deleted the update/cache_version branch September 17, 2021 16:01
CesarBazanAV pushed a commit to CesarBazanAV/yolov5 that referenced this pull request Sep 29, 2021
Defines dataset labels *.cache version in a single place, fixing a bug introduced in ultralytics#4845.
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
Defines dataset labels *.cache version in a single place, fixing a bug introduced in ultralytics#4845.
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.

1 participant