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

--resume training from URL weights fix #8884

Merged
merged 1 commit into from
Aug 5, 2022
Merged

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Aug 5, 2022

@kalenmike should fix data error on HUB resume

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improvement in data path handling for resumed YOLOv5 training sessions.

πŸ“Š Key Changes

  • Fixed the reference to the opt_data variable, ensuring the correct data path is used.
  • Replaced a simple string conversion with the check_file function for validating the data path.

🎯 Purpose & Impact

  • πŸ›  Fixes a bug where the data path might not be properly set when resuming training, enhancing reliability.
  • βœ… Ensures that resumed training sessions start smoothly without authentication timeout issues, improving user experience for those training models with YOLOv5 over time.

@kalenmike should fix data error on HUB resume
@glenn-jocher glenn-jocher self-assigned this Aug 5, 2022
@glenn-jocher glenn-jocher merged commit daed7a8 into master Aug 5, 2022
@glenn-jocher glenn-jocher deleted the update/resume branch August 5, 2022 14:42
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
--resume training from URL weights fix

@kalenmike should fix data error on HUB resume
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