-
-
Notifications
You must be signed in to change notification settings - Fork 16.4k
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
a good day #12465
base: master
Are you sure you want to change the base?
a good day #12465
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👋 Hello @tacyi, thank you for submitting a YOLOv5 🚀 PR! To allow your work to be integrated as seamlessly as possible, we advise you to:
- ✅ Verify your PR is up-to-date with
ultralytics/yolov5
master
branch. If your PR is behind you can update your code by clicking the 'Update branch' button or by runninggit pull
andgit merge master
locally. - ✅ Verify all YOLOv5 Continuous Integration (CI) checks are passing.
- ✅ Reduce changes to the absolute minimum required for your bug fix or feature addition. "It is not daily increase but daily decrease, hack away the unessential. The closer to the source, the less wastage there is." — Bruce Lee
@tacyi 👋 Hi! Thanks for the contribution. While we appreciate the friendly gesture, we generally avoid making cosmetic changes to keep the codebase focused. If you encounter any bugs or have features in mind, feel free to open a new issue or pull request 🚀. For more details, please check out our Contributing Guide. |
"good morning sirs" moment |
@ExtReMLapin haha, "good morning sirs" indeed! 😄 Always great to start the day with a bit of humor. If you have any questions or need assistance with anything related to YOLOv5, feel free to reach out! |
I have read the CLA Document and I sign the CLA 2 out of 3 committers have signed the CLA. |
🤖[deprecated] Generated by Copilot at d75e918
Summary
😊🙃🤨
Added a greeting to
train.py
. This is a cosmetic change that does not alter the logic or performance of the training script.Walkthrough
train.py
(link)🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
📊 Key Changes
The PR includes a change in the
train.py
script, although the exact modifications are not shown in the provided diff snippet.🎯 Purpose & Impact
The purpose of the change is unclear due to the lack of context. Normally, changes to
train.py
could involve updating training configurations, introducing new features, optimizing performance, or fixing bugs. The impact would be changes to how users train their YOLOv5 models, potentially making it easier, more efficient, or more effective.🌟 Summary
"Train script update – the outcome is a mystery 🕵️♂️🔍 but it's geared for an improved YOLOv5 training experience. Stay tuned for the reveal!"