-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Update trainer.py #10160
base: main
Are you sure you want to change the base?
Update trainer.py #10160
Conversation
As it stands, the scheduler is not a function with values from 1 to lrf, but instead is a function with values from 1 to lrf + a small epsilon resulting from the fact that for the last epoch x / self.epochs < 1 (e.g. for 300 epochs, for the last epoch it is 299/300)
CLA Assistant Lite bot: I have read the CLA Document and I sign the CLA 1 out of 2 committers have signed the CLA. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #10160 +/- ##
===========================================
- Coverage 75.99% 36.03% -39.97%
===========================================
Files 121 121
Lines 15351 15351
===========================================
- Hits 11666 5531 -6135
- Misses 3685 9820 +6135
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@JacekMaksymiuk please submit a new comment on this PR to sign the CLA with this line: I have read the CLA Document and I sign the CLA |
@glenn-jocher I have added this information to the comment. |
recheck |
As it stands, the scheduler is not a function with values from 1 to lrf, but instead is a function with values from 1 to lrf + a small epsilon resulting from the fact that for the last epoch x / self.epochs < 1 (e.g. for 300 epochs, for the last epoch it is 299/300)
I have read the CLA Document and I sign the CLA
🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Enhanced Learning Rate Strategies in Model Training 🚀
📊 Key Changes
🎯 Purpose & Impact