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

Minor import and spelling updates #1133

Merged
merged 1 commit into from
Oct 13, 2020
Merged

Minor import and spelling updates #1133

merged 1 commit into from
Oct 13, 2020

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Oct 13, 2020

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Refinement of codebase with spelling corrections and minor code reorganization for clarity.

πŸ“Š Key Changes

  • Removed unused platform import from detect.py.
  • Moved math import from the top of train.py to where it's used.
  • Corrected spelling mistakes across multiple comments and warning messages in utils/general.py.
  • Improved the clarity of existing comments and docstrings.

🎯 Purpose & Impact

  • πŸš€ Code Hygiene: Elimination of unused imports and better placement of others improve the code maintainability and readability.
  • βœ… Clarity & Communication: Spelling corrections and clearer comments ensure that the meaning is accurately conveyed, which benefits both user understanding and further development.
  • 🧩 Non-Functional Refactoring: These changes are non-functional; they have no direct impact on the program's output or performance, but contribute to better software practices.

@glenn-jocher
Copy link
Member Author

/rebase

@glenn-jocher glenn-jocher merged commit 4d3680c into master Oct 13, 2020
@glenn-jocher glenn-jocher deleted the import_and_spelling branch October 13, 2020 15:24
glenn-jocher added a commit that referenced this pull request Oct 15, 2020
burglarhobbit pushed a commit to burglarhobbit/yolov5 that referenced this pull request Jan 1, 2021
KMint1819 pushed a commit to KMint1819/yolov5 that referenced this pull request May 12, 2021
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
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.

1 participant