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

[pre-commit.ci] pre-commit suggestions #7279

Merged
merged 9 commits into from
Apr 4, 2022

Conversation

pre-commit-ci[bot]
Copy link
Contributor

@pre-commit-ci pre-commit-ci bot commented Apr 4, 2022

updates:

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Improvements and updates in code documentation, dependencies, and activation functions for the YOLOv5 model.

📊 Key Changes

  • Updated versions of pyupgrade and mirrors-yapf in pre-commit configuration for enhanced code formatting and modernization.
  • Extended documentation in the TensorFlow and PyTorch components of the model, adding comments explaining the purpose of classes and methods, such as layers and activation functions.
  • Refactored and improved descriptive comments for activation functions like SiLU, Hardswish, Mish, FReLU, and AconC in utils/activations.py for better understanding and readability.
  • Minor code enhancements and additional comments have been integrated across several files to clarify functionality and maintain consistency.

🎯 Purpose & Impact

  • 🚀 The update of dependencies ensures compliance with the latest standards and improvements offered by those packages.
  • 📝 Enhanced in-code documentation aids developers in understanding the structure and functionality of the YOLOv5 model more quickly and thoroughly.
  • ‍💻 Improved clarity in activation function descriptions ensures better maintainability and easier debugging for developers working with activation functions.
  • 🛠️ These updates collectively constitute a refinement of the codebase, which benefits developers contributing to or modifying the YOLOv5 model while making the code more accessible for educational purposes or further research.

@glenn-jocher glenn-jocher merged commit 7882950 into master Apr 4, 2022
@glenn-jocher glenn-jocher deleted the pre-commit-ci-update-config branch April 4, 2022 20:47
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
* [pre-commit.ci] pre-commit suggestions

updates:
- [github.com/asottile/pyupgrade: v2.31.0 → v2.31.1](asottile/pyupgrade@v2.31.0...v2.31.1)
- [github.com/pre-commit/mirrors-yapf: v0.31.0 → v0.32.0](pre-commit/mirrors-yapf@v0.31.0...v0.32.0)

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update yolo.py

* Update activations.py

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update activations.py

* Update tf.py

* Update tf.py

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Glenn Jocher <glenn.jocher@ultralytics.com>
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