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

Fix typos #13049

Merged
merged 2 commits into from
May 29, 2024
Merged

Fix typos #13049

merged 2 commits into from
May 29, 2024

Conversation

RainRat
Copy link
Contributor

@RainRat RainRat commented May 28, 2024

no functional change

I have read the CLA Document and I sign the CLA

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improvements in typo fixes and clarifications in YOLOv5 code comments and documentation.

πŸ“Š Key Changes

  • Fixed a typo in train.py, changing "mport" to "import".
  • Corrected a spelling mistake in augmentations.py ("satuaration" to "saturation").
  • Tweaked phrasing in dataloaders.py for clarity ("the the" to "the").
  • Adjusted wording in hpo.py within clearml logger for readability ("do dont" to "don't").

🎯 Purpose & Impact

  • Enhances Documentation: Makes the YOLOv5 codebase clearer and more professional, leading to improved readability. πŸ“š
  • Reduces Confusion: Correcting typos and clarifying documentation helps avoid misunderstandings, making it easier for new users to utilize the software effectively. 🧭
  • Improves Developer Experience: Such improvements, although minor, contribute to a smoother development process by providing accurate instructions and comments. πŸ› οΈ

These changes, while seemingly small, collectively enhance the user and developer experience, ensuring the accuracy and clarity of the YOLOv5 documentation and examples.

no functional change
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

πŸ‘‹ Hello @RainRat, 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 running git pull and git 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

better fix for typo

no functional change
@glenn-jocher glenn-jocher changed the title fix typos Fix typos May 29, 2024
@glenn-jocher glenn-jocher merged commit 2b7bcf6 into ultralytics:master May 29, 2024
8 checks passed
@glenn-jocher
Copy link
Member

@RainRat PR merged! Thank you for your contributions :)

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

2 participants