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

Update setup.cfg #2088

Merged
merged 6 commits into from
Jul 4, 2023
Merged

Update setup.cfg #2088

merged 6 commits into from
Jul 4, 2023

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jul 4, 2023

copilot:all

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Updates to the Ultralytics YOLOv3 repository include performance improvements, code cleanup, and secure use of shell commands.

📊 Key Changes

  • Disabled check-yaml in the pre-commit hooks.
  • Replaced subprocess.run() with a safer list argument format to mitigate shell injection.
  • Moved from --py37-plus argument to more generic settings in pyupgrade pre-commit hook configuration.
  • Corrected code formatting and style issues, following PEP8 guidelines.
  • Introduced better handling for de-parallelizing models when logging or plotting.
  • Updated license file configuration in setup.cfg.
  • Streamlined TensorFlow model code and fixed torch utility redundancy.

🎯 Purpose & Impact

  • 🔒 Increasing security by avoiding potential shell injection vulnerabilities.
  • 🧹 Cleaning up code to adhere to style guidelines, leading to improved readability and maintainability.
  • ⚙️ Enhancing model utility functions, possibly improving development workflows and user experience with neater and more efficient code execution.

@glenn-jocher glenn-jocher merged commit eb99ba8 into master Jul 4, 2023
@glenn-jocher glenn-jocher deleted the precommit branch July 4, 2023 22:50
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