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

Do not save hyp.yaml and opt.yaml on evolve #5775

Merged
merged 2 commits into from Nov 24, 2021
Merged

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Nov 24, 2021

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improvements to configuration saving in training script and fix to generation indexing in hyperparameter evolution logging.

πŸ“Š Key Changes

  • Conditional saving of hyp.yaml and opt.yaml files to exclude them during hyperparameter evolution (evolve).
  • Correctly adjusted the index for displaying the last generation number in hyperparameter evolution results output.

🎯 Purpose & Impact

  • The conditional saving prevents unnecessary overwriting of hyperparameter files when performing evolution, which can protect evolved configuration sets from being replaced. πŸ›‘οΈ
  • The adjustment to the last generation index ensures the accurate representation of generations, enhancing user comprehension of the evolution process. 🧬

These changes aim to refine the training process and maintain clarity in the hyperparameter evolution procedure for users and developers alike.

@glenn-jocher glenn-jocher self-assigned this Nov 24, 2021
@glenn-jocher glenn-jocher merged commit 4ca4aec into master Nov 24, 2021
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch November 24, 2021 22:25
@glenn-jocher glenn-jocher linked an issue Nov 29, 2021 that may be closed by this pull request
1 task
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
* Do not save hyp.yaml and opt.yaml on evolve

* Update general.py
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.

Difference between hyp_evolve.yaml and hyp.yaml after evolve run
1 participant