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

A minor correction in a comment #12782

Merged
merged 2 commits into from
Mar 4, 2024

Conversation

kumar-selvakumaran
Copy link
Contributor

@kumar-selvakumaran kumar-selvakumaran commented Mar 4, 2024

I added the 'h' in 'https' in the link to the label smoothing issue.

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Optimizations and clean-up in YOLOv5's codebase.

πŸ“Š Key Changes

  • Code Formatting: Improved readability and consistency by modifying string formatting in export.py.
  • Codebase Cleanliness: Added blank lines for better code separation in experimental.py, __init__.py in loggers, and clearml_utils.py.
  • Typo Correction: Fixed a missing 'h' in a URL within a comment in utils/loss.py to ensure the hyperlink is correct and accessible.

🎯 Purpose & Impact

  • Maintainability: Clearer, well-formatted code makes for easier maintenance and comprehension for developers working on the project.
  • Readability: These changes, while minor, contribute to the overall quality of the documentation and comments within the code.
  • Resource Accessibility: Correcting the typo in the URL ensures that developers can access the intended external information without confusion.

kumar-selvakumaran and others added 2 commits March 4, 2024 14:47
I added the 'h' in 'https' in the link to the label smoothing issue.

Signed-off-by: Kumar Selvakumaran <62794224+kumar-selvakumaran@users.noreply.github.com>
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 @kumar-selvakumaran, 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

@glenn-jocher
Copy link
Member

@kumar-selvakumaran looks good, thank you for the PR!

@glenn-jocher glenn-jocher merged commit b24177b into ultralytics:master Mar 4, 2024
kumar-selvakumaran added a commit to kumar-selvakumaran/yolov5 that referenced this pull request Mar 4, 2024
@kumar-selvakumaran kumar-selvakumaran deleted the patch-1 branch March 4, 2024 23:09
@kumar-selvakumaran
Copy link
Contributor Author

kumar-selvakumaran commented Mar 4, 2024

@kumar-selvakumaran looks good, thank you for the PR!

No problem! Can't wait to make bigger contributions moving forward.

@glenn-jocher
Copy link
Member

@kumar-selvakumaran that's fantastic to hear! We're excited to see your future contributions. The YOLOv5 community thrives because of the active involvement and dedication of contributors like you. Keep up the great work! πŸš€

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

3 participants