Skip to content

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

Merged
glenn-jocher merged 1 commit intomasterfrom
save-period
May 30, 2022
Merged

Simplify and fix --save-period epoch 0#8042
glenn-jocher merged 1 commit intomasterfrom
save-period

Conversation

@glenn-jocher
Copy link
Copy Markdown
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.

1 participant