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

Revert "make parameter ignore epochs" #5999

Closed
wants to merge 1 commit into from

Conversation

jinmc
Copy link
Contributor

@jinmc jinmc commented Dec 16, 2021

Reverts #5972

After training, I have noticed the parameter is not working as expected.
With further research, I figured out that best_fitness is not the parameter that affects EarlyStopping,
but fi parameter, so I should be editting that parameter.

I will make a PR after considering better code in terms of readability and cleanliness.

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Improvements and maintenance work continues on the ultralytics/yolov5 project, though this particular PR does not include code changes.

📊 Key Changes

  • No code changes were found in this PR.

🎯 Purpose & Impact

  • Since this PR does not contain any modifications, there is no direct impact on the user experience or the functionality of the project.
  • It's possible that this PR was created by accident, is a placeholder for further changes, or pertains to non-code-related updates such as documentation, which are not reflected in the diff.
  • Users and developers can expect that the actual codebase of the ultralytics/yolov5 project remains unchanged by this PR.

@github-actions
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions YOLOv5 🚀 and Vision AI ⭐.

@github-actions github-actions bot added the Stale label Jan 16, 2022
@github-actions github-actions bot closed this Jan 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant