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

Simplify and fix --save-period epoch 0 #8042

Merged
merged 1 commit into from
May 30, 2022
Merged

Simplify and fix --save-period epoch 0 #8042

merged 1 commit into from
May 30, 2022

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented May 30, 2022

Fix now allows for epoch 0 saving

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Simplified checkpoint saving condition during training in YOLOv5.

πŸ“Š Key Changes

  • Removed the redundant epoch check (epoch > 0) from the condition that determines when to save a checkpoint during training.

🎯 Purpose & Impact

  • πŸ’‘ Purpose: Streamlining the code to avoid unnecessary checks which can improve code readability and slight performance.
  • πŸ” Impact: Users will not notice a difference in how the model saves checkpoints; however, the codebase itself is minimally cleaner and more efficient. There is no change in functionality, just a refactor of the condition logic.

@glenn-jocher glenn-jocher changed the title Simplify and update --save-period epoch 0 Simplify and fix --save-period epoch 0 May 30, 2022
@glenn-jocher glenn-jocher self-assigned this May 30, 2022
@glenn-jocher glenn-jocher merged commit dcf8073 into master May 30, 2022
@glenn-jocher glenn-jocher deleted the save-period branch May 30, 2022 10:42
tdhooghe pushed a commit to tdhooghe/yolov5 that referenced this pull request Jun 10, 2022
Simplify and update `--save-period` epoch 0
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
Simplify and update `--save-period` epoch 0
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